Arduino - Loop Zähler

[Asdf A. (eizi)]

Hallo :)

Kaum hab ich das erste Problem überwunden ist auch schon das nächste 
da..:P
Hoffe mir kann wieder jemand so gut helfen.


Ich möchte dass eine LED genau einmal blinkt, egal wie oft der Loop 
durchläuft. Da dachte ich mir ich realisiere das einfach mit einem 
Zähler..klappt aber nicht und ich kann mir nicht erklären warum nicht.

Hier der Code

int ledoutput =9;
int var=0;


void setup() {
 
  pinMode(ledoutput, OUTPUT);
  
}


void loop() {
  
  var++;
  if(var <2) {
  digitalWrite(ledoutput, HIGH);
  delay(1000);
  digitalWrite(ledoutput, LOW);
  delay(1000); }
 
  }

Was ist falsch? ;)

[holger (Gast)]

void setup() {

  pinMode(ledoutput, OUTPUT);
  digitalWrite(ledoutput, HIGH);
  delay(1000);
  digitalWrite(ledoutput, LOW);
  delay(1000); }

}


void loop() {

  }

[Wolfgang (Gast)]

Max E. schrieb:
> Was ist falsch? ;)

var ist alle 256 loop-Durchläufe wieder 0

[Asdf A. (eizi)]

Okey das hätte ich wahrscheinlich noch erwähnen müssen, Setup ist keine 
Lösung.. ;)

Um das zu erläutern.

Ich plane meine Stereoanlage automatisch synchron zu meinem TV ein & 
auszuschalten. Wenn der Arduino also einen HIGH-Pegel bekommt soll 
dieser einfach einmal den IR-Code "einschalten" senden. Geht aber nur im 
Loop da der Arduino schon läuft.

Wenn der Arduino ein LOW vom TV bekommt soll dieser den IR-Befehl zum 
Ausschalten senden und den Zähler zurücksetzen :)

[Asdf A. (eizi)]

Wolfgang schrieb:
> Max E. schrieb:
>> Was ist falsch? ;)
>
> var ist alle 256 loop-Durchläufe wieder 0


Danke für die Info das wusste ich nicht, erklärt aber trotzdem nicht 
warum es nicht klappt. :P

[holger (Gast)]

>Okey das hätte ich wahrscheinlich noch erwähnen müssen, Setup ist keine
>Lösung.. ;)

void loop() {

  if(var == 0) {
  var = 1;
  digitalWrite(ledoutput, HIGH);
  delay(1000);
  digitalWrite(ledoutput, LOW);
  delay(1000); }

  }

[Wolfgang (Gast)]

Max E. schrieb:
> Danke für die Info das wusste ich nicht, erklärt aber trotzdem nicht
> warum es nicht klappt. :P

Doch, dein Zähler zählt
0, 1, 2, ..., 127, -128, -127, ... , -1, 0, 1, ...
und immer so weiter.

[Asdf A. (eizi)]

Klappt, vielen Dank :)


Trotzdem würde mich interessieren warum mein COde nicht funktioniert?

var++; 
  if(var <2) {
  digitalWrite(ledoutput, HIGH);
  delay(1000);
  digitalWrite(ledoutput, LOW);
  delay(1000); }
 
  }

Ich meine var wird auf eins erhöht, ist < als 2 & läuft durch und wird 
wieder erhöht. 2 = 2 also wird übersprungen

Wo ist der Denkfehler?

[Wolfgang (Gast)]

Wolfgang schrieb:
> 0, 1, 2, ..., 127, -128, -127, ... , -1, 0, 1, ...

Sorry, da du für var den Typ int verwendest (und nicht int8), läuft dein 
Zähler über 16 bit, also
> 0, 1, 2, ..., 32767, -32768, -32767, ... , -1, 0, 1, ...
und wiederholt sich erst nach 65536 Durchläufen.

[holger (Gast)]

>Trotzdem würde mich interessieren warum mein COde nicht funktioniert?

Ganz einfach:

var zählt so schnell durch das es nach einigen tausend CPU Zyklen
überläuft. Wurde oben ja schon beschrieben. Diese paar tausend
CPU Zyklen sind aber so schnell um das es für dich aussieht
als würde der Zähler nicht wirken und deine LED blinkt dauernd.

Mach mal das

void loop() {

  if(var == 0) {
  var = 1;
  digitalWrite(ledoutput, HIGH);
  delay(1000);
  digitalWrite(ledoutput, LOW);
  delay(1000); }

  delay(1000); }
  }

Dann wird es etwas dauern bis wieder geblinkt wird, aber
es wird wieder geblinkt;)

[holger (Gast)]

Verdammt zu früh abgesendet:(

So sollte das sein.

void loop() {
  var++;
  if(var <2) {
  digitalWrite(ledoutput, HIGH);
  delay(1000);
  digitalWrite(ledoutput, LOW);
  delay(1000); }

  delay(1000); }
  }

[holger (Gast)]

Und jetzt ne Klammer zu viel:(

Jetzt aber:

void loop() {
  var++;
  if(var <2) {
  digitalWrite(ledoutput, HIGH);
  delay(1000);
  digitalWrite(ledoutput, LOW);
  delay(1000); }

  delay(1000);
  }

[Amateur (Gast)]

>void loop() {
>  var++;
>  if(var <2) {
>  digitalWrite(ledoutput, HIGH);
>  delay(1000);
>  digitalWrite(ledoutput, LOW);
>  delay(1000); }
>
>  delay(1000);
>  }

Auch nicht!

Das ständige inkrementieren von var bewirkt, das diese irgendwann mal 
Kopeister schlägt. Dann geht‘s bei 0 wieder los.

einfacher: nix inkrement
void loop() {
  if (var == 0) {        // Nur einmal am Anfang zutreffend
    var = 1;
    ... an
    ... warte
    ... aus
    ... gut erhaltenes delay (1000) zu verkaufen
  }
}

[holger (Gast)]

>Auch nicht!

Ja, war schlecht das Beispiel weil der gesamte Kontext
im Post stehen sollte.

>Das ständige inkrementieren von var bewirkt, das diese irgendwann mal
>Kopeister schlägt. Dann geht‘s bei 0 wieder los.

Auch das wurde oben ja schon gesagt.

So, noch ein Beispiel damit er das ausprobieren kann

void loop() {
  var++;
  if(var <2) {
  digitalWrite(ledoutput, HIGH);
  delay(1000);
  digitalWrite(ledoutput, LOW);
  delay(1000); }

  delay(1);
  }

Wenn ich davon ausgehe das delay(1) eine Millisekunde dauert
wird in diesem Beispiel nach vermutlich 32s wieder dauernd geblinkt
weil der Zähler überläuft.

[Asdf A. (eizi)]

Okey, danke für deine Hilfe meine Stereosteuerung funktioniert & das mit 
der Deklarierung nicht geklappt hat hab ich auch verstanden:)

Ich zeig dir mal den Code :)

#include <IRremote.h>

int usbinput = 8;
int ledoutput =9;
int zaehler=0;
int zaehler1=0;

IRsend irsend;


void setup() {
 
  pinMode(usbinput, INPUT);
  pinMode(ledoutput, OUTPUT);
  
 }


void loop() {
  
  
  
   
  if (digitalRead(usbinput)==HIGH) {
      digitalWrite(ledoutput, HIGH); 
    
       if(zaehler == 0) {
        zaehler = 1;    
        zaehler1 = 0;    
      irsend.sendRC5(0x50C, 12); //Einschalten
      delay(6000); //Wartet bis Stero gestartet ist
      for(int i=0; i<=12; i++) { //Dreht Lautstärke auf Maximum
      irsend.sendRC5(0x410, 12);
      delay(400);  }
 
      }
  }
  
      
      if (digitalRead(usbinput)==LOW) {
      digitalWrite(ledoutput, LOW);
      zaehler=0;
      
      if(zaehler1 == 0) {
      zaehler1 = 1;
      irsend.sendRC5(0x50C, 12); //Ausschalten 
      delay(2000);

      }
    }
}