Delay funktioniert nur in loop (Arduino)

[Johannes S. (jojo_hau)]

Hallo zusammen.

Ich baue aktuell eine Steuerung für eine Fräse. In meinem Code verwenden 
ich delay() und micros(). Das funktioniert alles in der loop-Schleife, 
ich möchte den code aber gerne mehrmals verwenden und wollte dafür den 
code in eine Funktion packen. In der Funktion funktionieren delay() und 
micros() dann aber nicht mehr.

Als Beispiel:
digitalWrite(LED, HIGH);
delay(5000);
digitalWrite(LED, LOW);
delay(5000);

funktionier in der loop-Schleife perfekt und in einer Funktion leuchtet 
die LED kurz auf und geht wieder direkt aus.

Funktionieren diese Zeit-Sachen nur in der Loop-Schleife? Kann ich mir 
nicht wirklich vorstellen.

Grüße,
Johannes

[Rufus Τ. F. (rufus)]

Johannes S. schrieb:
> In der Funktion funktionieren delay() und micros() dann aber nicht mehr.

Wie gelangst Du zu dieser Erkenntnis?

Wann wird die Funktion von wo aufgerufen?

Wo ist Dein Code (nein, das Fragment oben zählt nicht)?

[Wolfgang (Gast)]

Johannes S. schrieb:
> In der Funktion funktionieren delay() und micros() dann aber nicht mehr.

Du musst die Funktion dann auch in loop() aufrufen.

[Johannes S. (jojo_hau)]

Danke für die Antwort.
Die oben genannten "5000" sind aber ein fester Wert, oder?

Zusätzlich vielleicht noch wichtig das ich dass ganze von dem 
receive-Event des I2c-Bus ausführen möchte. Ist das relevant?

Grüße,
Johannes

[Rufus Τ. F. (rufus)]

Johannes S. schrieb:
> Die oben genannten "5000" sind aber ein fester Wert, oder?

Die hast Du selbst genannt.

Magst Du mit Dir noch mal ins Reine kommen, wer hier jetzt welche Frage 
stellt?

[Johannes S. (jojo_hau)]

>Magst Du mit Dir noch mal ins Reine kommen, wer hier jetzt welche Frage
stellt?

Gerade stand hier noch dass Delay einen festen Wert braucht, ist aber 
schon wider weg.

>Wie gelangst Du zu dieser Erkenntnis?

Von der loop ausgeführt leuchtet die LED ca. 5 Sekunden und von dem 
receive-Event ausgeführt ca. 0,5s.

>Wo ist Dein Code (nein, das Fragment oben zählt nicht)?

Dachte mit einem so einfachen code ist es leichter den Fehler zu finden. 
Werden an meinen Code Notizen machen und gleich posten.

Grüße,
Johannes

[Johannes S. (jojo_hau)]

Hier mein Code, hoffe alles ist verständlich. Das ganze ist eine 
Schrittmotoren-Steuerung für 3 Schrittmotoren.

void receiveEvent(int HowMany)
{

  //Hier ist Code der Parameter für die for-Schleife festlegt 

  
  for(long a = 0;a < SchritteMax; a++)
  {
    Micros1 = micros(); //Zeit messen für die Verzögerung am Ende der Schleife
    TimerX++;       
    TimerY++;         //Wichtig für die bestimmung Welcher pin eine Puls bekommt. Nicht jeder Pin bekommt bei einem Schleifen-durchlauf einen Impuls. Denke dass das nicht so wichtig ist.
    TimerZ++;         
    if(digitalRead(PinEndschalterX) == LOW || digitalRead(PinEndschalterY) == LOW || digitalRead(PinEndschalterZ) == LOW)
    {
      break;           //if abfrage ob ein Endschalter betätigt wurde. Denke dass ist unwichtig.
    }
    if(TimerX >= Switches[0])
    {
      digitalWrite(PinStepX, LOW);
      delayMicroseconds(Impulslange);  //Zeit die der Pin LOW wird. sind 32 MicroSekunden.
      digitalWrite(PinStepX, HIGH);
      TimerX -= Schritt[0];            //Relevant für die Auswahl der Pins
      XCoordinate += WertProSchrittX;  //Koordinaten werden aktualisiert
    }
    if(TimerY >= Switches[1])
    {
      digitalWrite(PinStepY, LOW);
      delayMicroseconds(Impulslange);
      digitalWrite(PinStepY, HIGH);
      TimerY -= Schritt[1];
      YCoordinate += WertProSchrittY;
    }
    if(TimerZ >= Switches[2])
    {
      digitalWrite(PinStepZ, LOW);
      delayMicroseconds(Impulslange);
      digitalWrite(PinStepZ, HIGH);
      TimerZ -= Schritt[2];
      ZCoordinate += WertProSchrittZ;
    }
    
    if(a < SchrittMaxBeschleunigung)
    {
      Delay -= int(Beschleunigung);  //veränderung des Delays am ende. Beschleunigung bzw. Bremsen am anfang und ende.
    }
    
    if(a > SchrittMinBremsen)
    {
      Delay += int(Beschleunigung);
    }
    
    if(Delay < GeschwindigkeitAchse)
    {
      Delay = int(GeschwindigkeitAchse); // kontrollieren der Endgeschwindigkeit/ stimmt nie so ganz
    }
    
    for(long l=0;l<15000;l++)
    {
      Micros2 = micros();
      if (Micros2 - Micros1 > Delay)  //Verzögerung bis Der delay erreicht ist. Delay hat einen Wert zwischen 150 und 9999 MicroSekunden.
      {
        break;
      }
    }

  }
}

[Dieter S. (Gast)]

Erstmal ist derCode unvollständig und so funktioniert delay nicht.

Das muss so (Beispiel) aussehen "delay(500)".

[Johannes S. (jojo_hau)]

Danke für die Antwort

Dieter S. schrieb:
> Erstmal ist derCode unvollständig und so funktioniert delay nicht.

Was fehlt die den? soll ich nochmal eine Version des Codes posten ohne 
alle unwichtigen Sachen?

Dieter S. schrieb:
> Das muss so (Beispiel) aussehen "delay(500)".

Danke für die info, mit einer Variable hat es bis jetzt bei mir noch nie 
Probleme gegeben. Hab das Ganze mit dem Wert 32 anstatt Impulslange 
probiert funktioniert aber trotzdem nicht.

[Dieter S. (Gast)]

Im Code fehlt die komplette Deklaration und das Setup sowie die Loop. 
Wir wissen nicht was da losgeht.

Und die Variable musst du in die Klammer setzen.

[Johannes S. (jojo_hau)]

Ich hab die Datei mal Angehängt, da das Programm aber kompliziert ist 
und es keine Kommentare gibt werde ich hier die wichtigsten Sachen 
auflisten:

In dieser Version funktioniert der gesamte Code. Hier befindet sich der 
Primäre teil noch in der loop. In der nächsten Verion befinden sich 
vieles in einer libary.

Der Teil der in der loop steht soll in eine Funktion.

Der Arduino empfängt Daten über die I2c-Schnittstelle und setzt die 
Variabel Motorfahren auf true, wodurch die "Funktion" in der loop 
ausgeführt wird.

primär geht es darum das eine Funktion mit delay() oder micros() die vom 
receive-Event ausgeführt wird nicht korrekt funktioniert.

sorry hab den Code 2mal angehängt

[Ulrich F. (Gast)]

Johannes S. schrieb:
> primär geht es darum das eine Funktion mit delay() oder micros() die vom
> receive-Event ausgeführt wird nicht korrekt funktioniert.

Das liegt daran, dass die Interrupts in einer Interruptroutine gesperrt 
sind.

[Albert M. (Firma: Bastler aus Mönchengladbach) (albertm)]

"Don't use the delay() function in an interrupt routine!
It relies on interrupts itself."

[Johannes S. (jojo_hau)]

Danke für die Antworten

Albert M. schrieb:
> "Don't use the delay() function in an interrupt routine!
> It relies on interrupts itself."

was könnte ich als alternative nehmen? Die micros() Funktion scheint 
auch nicht zu funktionieren.

Grüße,
Johannes

[Ulrich F. (Gast)]

Johannes S. schrieb:
> was könnte ich als alternative nehmen?

Du hast die Interruptroutine gnadenlos überladen.
Ändere das.
Schleifen, Delay() und seine Brüder haben da nichts drin verloren.
ISRs haben schlank zu sein.

[Johannes S. (jojo_hau)]

Ulrich F. schrieb:
> Du hast die Interruptroutine gnadenlos überladen.
> Ändere das.
> Schleifen, Delay() und seine Brüder haben da nichts drin verloren.
> ISRs haben schlank zu sein.


Was meinst du damit konkret?
Ich denke das die Auswertung der Daten des I2c-Busses nicht so ein 
Problem ist, dass ist zwar viel code, es wird davon aber nur ein kleiner 
teil aufgerufen.

Soll ich die Funktion mit for-schleife und delays in die loop packen?

Grüße,
Johannes

[Ulrich F. (Gast)]

> Was meinst du damit konkret?

Du beschwerst dich über das nicht funktionieren von delay() in einer 
ISR.
Ich sage: Dann ändere dein Programm so, dass das delay() nicht nötig 
ist.
Mehr nicht....
Der Ball liegt bei dir, spiele ihn.

[Johannes S. (jojo_hau)]

Ich hatte nicht verstanden dass das Event eine ISR ist. Jetzt ist mir 
das Klar.
Hab auch schon eine Idee, werde das Morgen testen.

Danke für die Erklärungen.

Grüße,
Johannes