Arduino Interrupt

Mal grundlegend: der strukturelle Fehler ist das delay im Interrupt.
Ein Interrupt ist eine " Unterbrechung" und Unterbrechungen sollten 
nicht allzu lang dauern...

Dan N. schrieb:
> Über einen Taster setze ich den Pin 3 auf High.
Wie ist der Taster angeschlossen?

> Leider reagiert der Interrupt...
Und was macht dann die andere LED?

im Anhang der Schaltplan

Lothar M. schrieb:
>> Leider reagiert der Interrupt...
> Und was macht dann die andere LED?

Die andere LED blinkt ganz normal weiter...

Dan N. schrieb:
> Leider reagiert der Interrupt entweder gar nicht, oder nur ganz
> kurz(d.h. die LED leuchtet nur ganz kurz auf, obwohl sie ja 3 sek.
> leuchten sollte).

Note
Inside the attached function, delay() won’t work

https://www.arduino.cc/reference/en/language/functions/external-interrupts/attachinterrupt/

Dan N. schrieb:
> im Anhang der Schaltplan
Da fehlt ein Pulldown für einen halbwegs definierten Low-Pegel...

Du könntest auch den Pullup im uC-Pin aktivieren und mit dem Taster nach 
GND schalten. So machen es alle Anderen.

Hi,
hab dir mal dein Code bisschen umgeschrieben das die Delays da raus 
sind. Sollte sich alles selbst erkären. So kann der µcontroller noch 
andere Dinge außer "warten" machen.

const int ledPin = 13;
const int ledPin2 = 7;
bool interrupt = 0;

void setup()
{
 pinMode(ledPin, OUTPUT);
 pinMode(ledPin2, OUTPUT);
 attachInterrupt(1, pin_ISR_1, RISING);
}

void loop()
{
static long timer1=0;
static long timer2=0;

if (millis()-timer1>=500){
 timer1+=500;
 digitalWrite(ledPin2, digitalRead(ledPin2)^1);  // wechselt ~alle 500ms
 }
if (interrupt == 1){
 digitalWrite(ledPin, HIGH);   // letzt Led nach Interrupt
 interrupt = 0;
 timer2=millis();
 }
if (abs(timer2-millis())>3000){ // Schaltet nach 3 Sec die Led wieder 
aus
 digitalWrite(ledPin, LOW);
 }

//
//Hier könnte noch viel viel mehr code stehen.
//

}

void pin_ISR_1()
{
interrupt = 1;
}

Hab den jetzt nicht getestet aber sollte so schon passen... und nicht 
den Pulldown Widerstand vergessen.

MfG
M.V

Lothar M. schrieb:
> Da fehlt ein Pulldown für einen halbwegs definierten Low-Pegel...

Oh, ja klar...den hab ich vergessen!

Georg M. schrieb:
> Note
> Inside the attached function, delay() won’t work

Dann hab ich sowieso ein größeres Problem. Diesen Code hab ich nur 
schnell geschrieben, um generell die Funktion des Interrupts zu testen.
In meinem eigentlichen Code für mein Projekt hab ich im Interrupt 
mehrere delays drin. Und der Code ist generell länger!


Code für Interrupt:

void showBATTERY()
{
  batteryVALUE = ((int32_t)analogRead(battery) * 1000l)/1024; // 
Umwandlung von Analog in Digital

 if (batteryVALUE < 750) // Bedingung
  {
    digitalWrite(ledRED , HIGH);  // Wenn Bedingung erfüllt, wird ledRED 
auf High gesetzt
    delay(3000);
    digitalWrite(ledRED , LOW);
  }
 else
  {
    digitalWrite(ledRED , LOW);  //Wenn Bedingung nicht erfüllt, wird 
ledRED auf Low gesetzt
  }
 if (batteryVALUE > 750) // Bedingung
  {
    digitalWrite(ledGREEN , HIGH); // Wenn Bedingung erfüllt, wird 
ledGREEN auf High gesetzt
    delay(3000);
    digitalWrite(ledGREEN , LOW);
  }
 else
  {
    digitalWrite(ledGREEN , LOW); // Wenn Bedingung nicht erfüllt, wird 
ledGREEN auf Low gesetzt
  }
}

ne so wird das auch nichts, aber mit mein Ansatz sollte dein Problem zu 
lösen sein.

MfG
M.V

Ich könnte mich auch damit anfreunden, dass ich die delays komplett 
rauswerfe...

void showBATTERY()
{
  batteryVALUE = ((int32_t)analogRead(battery) * 1000l)/1024; //
Umwandlung von Analog in Digital

 if (batteryVALUE < 750) // Bedingung
  {
    digitalWrite(ledRED , HIGH);  // Wenn Bedingung erfüllt, wird ledRED
auf High gesetzt
  }
 else
  {
    digitalWrite(ledRED , LOW);  //Wenn Bedingung nicht erfüllt, wird
ledRED auf Low gesetzt
  }
 if (batteryVALUE > 750) // Bedingung
  {
    digitalWrite(ledGREEN , HIGH); // Wenn Bedingung erfüllt, wird
ledGREEN auf High gesetzt
  }
 else
  {
    digitalWrite(ledGREEN , LOW); // Wenn Bedingung nicht erfüllt, wird
ledGREEN auf Low gesetzt
  }
}

Wenn das blinken unwichtig ist.

Michael V. schrieb:
> Wenn das blinken unwichtig ist.

es soll ja nicht blinken, sondern für eine gewisse zeit die led leuchten 
lassen und dann wieder ausgehen. aber ich kann damit leben, wenn dann 
die entsprechende led dauerhaft leuchtet bis zu einem reset.

das problem ist aber, dass es ohne die delays auch nicht so richtig 
funktioniert.
wenn ich am analogen eingang die spannung auf über 7,5 Volt drehe und 
den taster drücke, dann schaltet es von der roten led auf die grüne 
um...so weit so gut, aber wenn ich den taster bei der gleichen spannung 
erneut drücke, dann schaltet es wieder auf die rote led um

Michael V. schrieb:
> Hi,
> hab dir mal dein Code bisschen umgeschrieben das die Delays da raus
> sind. Sollte sich alles selbst erkären. So kann der µcontroller noch
> andere Dinge außer "warten" machen.

das ist gut gemeint, aber das ist bei meinem loop-programm leider nicht 
möglich, da dort nur mein hauptcode drin stehen darf, da es sich um eine 
zeitabhängige programmierung im ms-bereich handelt und da kann ich mir 
keine weitere zeitverzögerung erlauben.

Sei froh das der noch irgendwas macht !

Spannung vom Analogen Eingang sind 0-5V

Michael V. schrieb:
> Sei froh das der noch irgendwas macht !
>
> Spannung vom Analogen Eingang sind 0-5V

keine sorge, ich hab nen spannungsteiler drin. maximal liegen 4,5 Volt 
an!

kannst du mal dein ganzen Code und Schaltplan posten? Dann kann man dir 
besser helfen.

ich weiß, der schaltplan ist etwas unkonventionell gemacht...aber das 
sollte jetzt egal sein!


Gesamter Code:

const int impECU = A1;    // impECU den Pin A1 zuweisen
const int ledBLUE = 4;     // ledBLUE den Pin 4 zuweisen
int impVALUE = 0;  // den Wert von impVALUE auf Null setzen
const int REED = 2;  // REED den Pin 2 zuweisen
const int RELAY = 8; // RELAY den Pin 8 zuweisen
const int battery = A0;    // battery den Pin A0 zuweisen
const int ledRED = 13;      // ledRED den Pin 13 zuweisen
const int ledGREEN = 7;     // ledGREEN den Pin 7 zuweisen
int batteryVALUE = 0;  // den Wert von batteryVALUE auf Null setzen

void setup()
{
  pinMode(impECU, INPUT);  // impECU als INPUT deklarieren
  pinMode(ledBLUE, OUTPUT);  // ledBLUE als OUTPUT deklarieren
  pinMode(REED , INPUT);  //  REED als INPUT deklarieren
  pinMode(RELAY , OUTPUT);  // RELAY als OUTPUT deklarieren
  pinMode(ledRED, OUTPUT);  // ledRED als OUTPUT deklarieren
  pinMode(ledGREEN, OUTPUT);// ledGREEN als OUTPUT deklarieren
  pinMode(battery, INPUT);  // battery als INPUT deklarieren
  attachInterrupt(1, showBATTERY, RISING);
}

void loop()
{
  impVALUE = ((int32_t)analogRead(impECU) * 1000l)/1024; // Umwandlung 
von Analog in Digital

 if (impVALUE > 700) // Bedingung (größer als 7 Volt)
  {
    digitalWrite(ledBLUE , HIGH);  // Wenn Bedingung erfüllt, wird 
ledBLUE auf High gesetzt
  }
 else
    digitalWrite(ledBLUE , LOW);  // Wenn Bedingung nicht erfüllt, wird 
ledBLUE auf LOW gesetzt

 if (digitalRead(REED) == HIGH)  // Bedingung
  {
    digitalWrite(RELAY , HIGH);  // RELAY wird auf HIGH gesetzt
    delay(70);  // 70ms Pause
    digitalWrite(RELAY , LOW);  // RELAY wird auf LOW gesetzt

    do
    {  delay(10); // 10ms Pause
    }
    while(digitalRead(REED) == HIGH); // Abfrage, ob Taster wieder 
losgelassen wurde
  }
}
void showBATTERY()
{
  batteryVALUE = ((int32_t)analogRead(battery) * 1000l)/1024; // 
Umwandlung von Analog in Digital

 if (batteryVALUE < 750) // Bedingung
  {
    digitalWrite(ledRED , HIGH);  // Wenn Bedingung erfüllt, wird ledRED 
auf High gesetzt
  }
 else
  {
    digitalWrite(ledRED , LOW);  //Wenn Bedingung nicht erfüllt, wird 
ledRED auf Low gesetzt
  }
 if (batteryVALUE > 750) // Bedingung
  {
    digitalWrite(ledGREEN , HIGH); // Wenn Bedingung erfüllt, wird 
ledGREEN auf High gesetzt
  }
 else
  {
    digitalWrite(ledGREEN , LOW); // Wenn Bedingung nicht erfüllt, wird 
ledGREEN auf Low gesetzt
  }
}

bei A0 soll doch sicher +12V nicht +9 Volt gemessen werden ? hast du das 
so in der realität ?

Welche Relais benutzt du ?

zusätzlich sind da noch eine menge "Fehler" drin. Entweder in der 
Zeichnung und/auch von der Lokik her.

Das dürfte alles so nicht Funktionieren. Wenn die Betriebsspannung auf 
unter 9.5 V geht hört der Festspannungsregler auf zu arbeiten. Somit 
kein Strom für den µController somit keine Lampen Leuchten. Du kannst 
halt mit deiner Anordnung nur von 12-9.5 V messen.

Michael V. schrieb:
> bei A0 soll doch sicher +12V nicht +9 Volt gemessen werden ? hast
> du das
> so in der realität ?
>
> Welche Relais benutzt du ?

doch die 9 volt stimmen, da hängt zusätzlich eine 9V Bolckbatterie 
dran...diese soll eben überwacht werden.

War beim ersten blick nicht so klar -.-

ich muss jetzt los hab, aber dein Problem verstanden meld mich später 
oder morgen mal.

Michael V. schrieb:
> War beim ersten blick nicht so klar -.-
>
> ich muss jetzt los hab, aber dein Problem verstanden meld mich später
> oder morgen mal.

ich muss auch los und bin erst am wochenende wieder online

Zur Inspiration ;-)

(Edit: Hm, die erste Datei bitte ignorieren - habe ein bisschen 
nachgebessert. Kann aber die Datei im Beitrag nicht löschen)

Eric B. schrieb:
> Zur Inspiration ;-)
>
> (Edit: Hm, die erste Datei bitte ignorieren - habe ein bisschen
> nachgebessert. Kann aber die Datei im Beitrag nicht löschen)

Danke...das ist etwas zu hoch für mich ;-)
Ich bin ja ein Arduino-Neuling...