Probleme Interrupt Arduino Uno

[Simon K. (simon_k96)]

Hallo Community,

Ich bin gerade dabei eine Entfernungsmessung mit einem Ultraschallsensor 
(HC-SR04) und einem Interrupt in Arduino zu programmieren. Die 
Entfernungsmessung ansich funktioniert mithilfe eines "delay()", aber 
sobald ich das mit einem Interrupt austausche, kommt beim Compillieren 
folgende Fehlermeldung:

"sketch_okt23a.ino: In function 'void loop()':
sketch_okt23a:24: error: 'Distance' was not declared in this scope
sketch_okt23a:34: error: a function-definition is not allowed here 
before '{' token"

Ich würde mich sehr über Hilfe oder Lösungsvorschlage dazu freuen.
mfg Simon K.

Code:

const int trigPin =12;      //Trig Pin
const int echoPin = 13;     //Echo Pin
unsigned long previousMillis = 0; // speichert wie viele Sekunden seit derletzten Änderung vergangen sind
unsigned long interval = 1000;    // Interval zwischen zwei Änderungen//Echo Pin
long Duration = 0;

void setup(){
  pinMode(trigPin,OUTPUT);
  pinMode(echoPin,INPUT);
  Serial.begin(9600);
}

void loop()
{
  digitalWrite(trigPin, LOW);
  delayMicroseconds(2);
  digitalWrite(trigPin, HIGH);
  delayMicroseconds(10);
  digitalWrite(trigPin, LOW);

  Duration = pulseIn(echoPin,HIGH);

  long Distance_mm = Distance(Duration);

  Serial.print("Distance = ");
  Serial.print(Distance_mm);
  Serial.println(" mm");

  if (millis() - previousMillis > interval)
  {
    previousMillis = millis();   // aktuelle Zeit abspeichern
    long Distance(long time)
    {
      long DistanceCalc;
      DistanceCalc = ((time /2.9) / 2);
      return DistanceCalc;
    }
  }
}

[Helmut H. (helmuth)]

Sehe nichts was mit interrupts zu tun hat...
Damit das kompiliert du musst die Funktion Distance  aus der loop() 
herausnehmen.

Würde das klarer strukturieren (ohne Gewähr)

...
long messen() {
    // gibt dauer des impulses zurück
    digitalWrite(trigPin, LOW);
    delayMicroseconds(2);
    digitalWrite(trigPin, HIGH);
    delayMicroseconds(10);
    digitalWrite(trigPin, LOW);
    return pulseIn(echoPin,HIGH);
}

void zeige_mm(long time)       {
    // rechnet zeit in mm um und gibt aus
    long Distance_mm= ((time /2.9) / 2);
    Serial.print("Distance = ");
    Serial.print(Distance_mm);
    Serial.println(" mm");
}


void loop() {

    if (millis() - previousMillis > interval)
    {
        previousMillis = millis();   // aktuelle Zeit abspeichern
        zeige_mm(messen());
    }
}

[Karl H. (kbuchegg)]

Simon K. schrieb:

> sketch_okt23a:24: error: 'Distance' was not declared in this scope

Es gibt als keine Funktion oder Variable names Distance. Zumindest hat 
der Compiler in Zeile 24 noch nie etwas davon gehört

> sketch_okt23a:34: error: a function-definition is not allowed here
> before '{' token"

Deine Einrücktiefe zeigt es ohnehin recht deutlich
>

>   if (millis() - previousMillis > interval)
>   {
>     previousMillis = millis();   // aktuelle Zeit abspeichern
>     long Distance(long time)
>     {
>       long DistanceCalc;
>       DistanceCalc = ((time /2.9) / 2);
>       return DistanceCalc;
>     }
>   }
> }
>

Du kannst nicht innerhalb einer Funktion eine weitere 
Funktiondefinieren. Funktionen stehen immer für sich alleine.
Also nicht

void foo()
{
  ...
  void bar()
  {
    ...
  }
}

sondern

void foo()
{
 ...
}

void bar()
{
 ...
}

UNd wenn du dann beim Schreiben des Textes auch noch berücksichtigst, 
dass der COmpiler den Text genauso wie du auch von oben nach unten liest 
(und zwar nur ein einziges mal), dann ist es auch logisch, dass es eine 
gute Idee wäre, eine Funktionsdefinition vor ihre erste Verwendung (also 
den Aufruf) zu verschieben.

Also nicht

void foo()
{
  bar();
}

void bar()
{
  ...
}

sondern

void bar()
{
  ...
}

void foo()
{
  bar();
}

[Simon K. (simon_k96)]

Dankeschön für die schnellen Antworten!
Ich muss dazu sagen, dass ich erst seit ein paar Wochen mich mit Arduino 
beschäftige, aber ich werde es gleich mal testen.
mfg Simon K.