Was stimmt an diesem Code nicht?

[Simon N. (ardi)]

Hi,
ich habe an meinem Arduino einen Schlater und 3 LEDs angebunden. Wenn 
ich den Knopf drücke soll die erste LED starten, wenn ich den nochmal 
drücke die zweite und die dritte beim dritten drücken.
Wenn ich aber es teste und auf den Knopf drücke, dann geht die erste LED 
an und nach dem drücken wieder aus und nochmal drücken das gleiche.
Was ist hier falsch:

void setup(){
  pinMode(13, OUTPUT);
  pinMode(12, OUTPUT);
  pinMode(11, OUTPUT);
  pinMode(2, INPUT);
  Serial.begin(9600);
}

void loop(){
 int counter=0;

 int schalter=digitalRead(2);
 Serial.println(schalter, DEC);
 
 if(schalter==HIGH){
   counter++;
 }
 if(counter==0){
   digitalWrite(13, LOW);
   digitalWrite(12, LOW);
   digitalWrite(11, LOW);
 }
 if(counter==1){
   digitalWrite(13, HIGH);
   digitalWrite(12, LOW);
   digitalWrite(11, LOW);
 }
 if(counter==2){
   digitalWrite(13, HIGH);
   digitalWrite(12, HIGH);
   digitalWrite(11, LOW);
 }
 if(counter==3){
   digitalWrite(13, HIGH);
   digitalWrite(12, HIGH);
   digitalWrite(11, HIGH);
 }  
 if(counter==4){
   counter=0;
 }
 
  
}

Danke und Gruß

[Flo G. (phlo)]

Du setzt mit

int counter=0;

in jedem Schleifendurchlauf den counter wieder auf 0. D.h. er kann gar 
nie höher als 1 werden. Heißt: Die Initialisierung von counter ins setup 
verschieben!

Grüße

[MÖÖP (Gast)]

Der Counter in deiner Loop wird immer wieder auf "0" gesetzt ... immer 
und immer wieder ...

[Axel R. (Gast)]

Da ist kein Schleifendurchlauf. Nur weil die Routine "loop()" heisst?
Der macht das alles einmal und resettet evtl. Beim nächsten Druck ist er 
dann wieder bei eins.
Oder ist das beim aruninio anders? bestimmt doert der Name der Funktion 
deren Verhalten ;))

[Thomas H. (pcexperte)]

Simon Name schrieb:

> void loop(){
>  int counter=0; <--- !!!!!!!!
>

Weil bei jedem Durchlauf deiner Hauptschleife counter auf 0 gesetzt 
wird.

Schreibe das int counter=0; mal in die void setup().

Gruß
Thomas

[Flo G. (phlo)]

Axel R. schrieb:
> Oder ist das beim aruninio anders? bestimmt doert der Name der Funktion
> deren Verhalten ;))

Auch wenns für die Hardcore-C-Frickler meistens gewöhnungsbedürftig ist: 
Ja. Macht auch Sinn. Siehe auch 
http://de.wikipedia.org/wiki/Konvention_vor_Konfiguration

[Axel R. (Gast)]

Das "int counter=0;" dort falsch steht, kommt ja noch hinzu

[Simon N. (ardi)]

Ok danke, das hat schon geholfen.
Nur jetzt ist es so der Ablauf
1.Druck: LED 1 und 2 gehen an
2. Druck: während dem Drücken gehen alle LEDs an und nach dem Drücken 
sind alle aus
3.Druck: während dem Drücken gehen wieder alle an und danach ist die 
erste LED an
4. Druck: beim Drücken geht sind LED1 und 2 an und danach ist die erste 
an
5. Druck: alle sind an
6. Druck: dritte geht aus
7.Druck: alle gehen beim Drücken an und danch sind alle wieder aus

Und nun mein Gedanke:
WAS???
Kann mir jemand helfen?

[Axel R. (Gast)]

http://arduino.cc/de/Reference/Loop
Ach du kack..
Na - da bin ich raus. Mit sowas will ich nichts zu tun haben ;)

[Flo G. (phlo)]

Was sagt denn die Ausgabe deines Serial.print? Vielleicht auch mal den 
Counter damit ausgeben...

[Mike M. (mikeii)]

Bleib cool,

statt in der main()

while(1){

// schöne Sachen machen

}

ham die bei sich im Hintegrund wohl

while(1){

loop();

}

Soll den Einsteigern wohl ne Hilfe sein

[Axel R. (Gast)]

war ja nicht soo bierernst zu nehmen. Natürlich lese ich hier weiter mit 
:)
Die kommen aber auch auf Ideen. Wenn der geneigte arduino-User dann mal 
umsteigt, blickt er nicht mehr durch!"
Die Umgebung ( processing.org ) finde ich aber ganz nett, muss ich 
erlich zugeben. Ich habe da mit JAVA bissschen was veranstaltet. Geht 
1A!
Zurück zum Thema...

[Dietrich L. (dietrichl)]

Simon Name schrieb:
> WAS???

Ist die Taste entprellt? Sonst ist das Ergebnis eher zufällig, zumindest 
nicht genau reproduzierbar.

Simon Name schrieb:
> if(schalter==HIGH){
>    counter++;

Außerdem wird nicht gewartet, bis der Taster wieder losgelassen wird. 
Also müssten beim Halten der Taste die LEDs durchlaufen.

Aber vielleicht sind in den Arduino-Funktionen noch geheimnisvolle 
Eigenschaften versteckt, die das verhindern ...

Gruß Dietrich

[Helfender (Gast)]

Solange der Schalter gedrückt ist, zählt er immer wieder
und wieder hoch. Daher sind alle LEDs an.
Wenn du ihn los lässt, dann bleibt er zufällig auf dem letzten
Wert stehen.

Vielleicht solltest du nach dem Drücken und Setzen der LEDs
warten, bis der Schalter wieder losgelassen wird.

Nächstes Problem könnte dann das Prellen des Schalters sein...

[Axel R. (Gast)]

http://www.arduino.cc/en/Tutorial/Switch

[Charles. K (Gast)]

Normalerweise reicht ein static vor der Deklaration. Dann wird die 
Variable nur einmal initialisiert (zumindest in C).
Also:
[c]void loop(){
 static int counter=0;
...[c]

Trotzdem denke ich auch, dass etwas gegen das Tastenprellen gemacht 
werden muß.


Gruß,
Charles

[Mike M. (mikeii)]

Ist das beim Arduino nicht schon von der Bibliothek aus entprellt? Kann 
mir nicht vorstellen, dass ein Anfänger das kann/weiß

[Charles. K (Gast)]

Ich bin kein Arduino-Experte, aber da digitalRead nur den Zustand eines 
Ports zurück gibt, kann die Funktion nicht wissen, ob ein Taster oder 
etwas anderes angeschlossen ist. Ich habe auch ein Beispiel-Code zum 
entprellen gefunden: http://www.arduino.cc/en/Tutorial/Switch

[Axel R. (Gast)]

> Ich habe auch ein Beispiel-Code zum
entprellen gefunden: http://www.arduino.cc/en/Tutorial/Switch

Ach nee, Du auch?
Beitrag "Re: Was stimmt an diesem Code nicht?"

[Arduino (Gast)]

Es gibt auch eine Library mit der benötigten Funktionalität.

http://playground.arduino.cc//Code/Bounce