Logikfehler in Tasterauswertung für Menue

[Amper (Gast)]

Hallo Zusammen,

ich versuche gerade ein Menue zu Programmieren. Um in das Menue zu 
kommen soll eine Taste eine Secunde lang gedrückt werden. Danach soll 
mit jedem weiteren druck der Taste in den nächsten Menue Punkte 
gesprungen werden. Das klapt auch so weit sehr gut.
Allerdings soll wenn ich im lezten Menue Punkt bin mit einem weiteren 
Druck der Taste das Menue verlassen werden.
Leider spring ich dann immer zurück in den ersten Menuepunkt.

int menuePunkt = 0;

unsigned long timemils, timesec, timesecalt, menueDelay;
unsigned long timeMenueTaster1, timeMenueTaster2, timePlusTaster, timeMinusTaster;

bool statusMenueTaster, statusPlusTaster, statusMinusTaster;
bool statusMenue = false, statusPlus = false, statusMinus = false;
bool minusOn = false, plusOn = false, menueOn = false;
bool timerabgelaufen = false;

void loop () {  
  timemils = millis();
  
  //Start Tasterabfrage plus entprellung und Zeitverzögerung für Menue
  statusMenueTaster = digitalRead(menueTaster);
  statusPlusTaster  = digitalRead(plusTaster );
  statusMinusTaster = digitalRead(minusTaster);


  if(statusMenueTaster == LOW){
    if(timeMenueTaster1 < timemils && statusMenue == false){
      statusMenue = true;
      menueOn = true;
      if (menuePunkt==0){
        menueDelay = timeMenueTaster1+950;
      }
    }
  }else{
    timeMenueTaster1 = timemils+50;
    statusMenue = false;
  }

  if(statusMenue == true && menuePunkt == 0){
    if(menueDelay < timemils){
      menuePunkt = 1;
      menueOn = false;
    }
  }
  
  if(statusPlusTaster == LOW){
    if(timePlusTaster < timemils && statusPlus == false){
      statusPlus = true;
      plusOn= true;
    }
  }
  else{
    timePlusTaster = timemils+50;
    statusPlus = false;
  }

  if(statusMinusTaster == LOW){
    if(timeMinusTaster < timemils && statusMinus == false){
        statusMinus = true;
        minusOn = true;
      }
  }
  else{
    timeMinusTaster = timemils+50;
    statusMinus = false;
  }
  //Ende Tasterabfrage plus entprellung und Zeitverzögerung für Menue

  //Gehe durch das Menue.
  if (menuePunkt != 0 && menueOn){
    menueOn = false;
    menuePunkt++;
    }

  if (menuePunkt== 1){
      analogWrite(5 ,255 );
    } else if (menuePunkt== 2){
      analogWrite(6 ,255 );
    } else if (menuePunkt== 3){
      analogWrite(11 ,255 );
    } else if (menuePunkt== 4){
      analogWrite(5 ,0 );
      analogWrite(6 ,0 );
      analogWrite(11 ,0 );
      menuePunkt =0;
    }
}

Ich komm leider nicht drauf wo mein logischer Fehler liegt.

Danke schon mal für eure hilfe und noch mehr danke an die jenigen die es 
sich verkneifen können mich dumm anzumachen weil ich erst mal 
Programmierun lernen soll oder das der Code völliger mist ist oder, 
oder, oder...
Ist ja leider in nicht ganz unüblich hier.


bis dann und Danke
Amper

[grundschüler (Gast)]

Ohne debuggen

z.B. per LCD, uart, notfalls LEDs ist das ganz schwierig, den Fehler zu 
finden.

Du kannst mal versuchen menuePunkt auf 255 zu setzen. Wenn dann 1 
addiert wird ist er bei 0.

[A. S. (Gast)]

Du setzt doch menuPunkt hier auf 0
Amper schrieb:
> } else if (menuePunkt== 4){
>       analogWrite(5 ,0 );
>       analogWrite(6 ,0 );
>       analogWrite(11 ,0 );
>       menuePunkt =0;
>     }

und im nächsten Durchlauf auf 1
>  if(statusMenue == true && menuePunkt == 0){
>    if(menueDelay < timemils){
>      menuePunkt = 1;
>      menueOn = false;
>    }
>  }

[Amper (Gast)]

Hallo Grunschüler,

ich hab die ganzen Serial.print `s rausgenommen da der Code sonst 
ziemlich durcheinander ist. Ein paar LED´s hab ich auch drann um sehen 
zu was der yC gerade so macht.

@Achim S. ja das hab ich auch gesehen ich brauch das (if(statusMenue == 
true && menuePunkt == 0){) aber um beim ersten Druchlauf überhaubt ins 
menue zu kommen.

Wenn ich jede menge Serial.print eingebaut habe kann ich sehen das ich 
den statusMenue wider auf false setzte bevor die abfrage kommt 
allerdings muss ich dazu den taster ziemlich schnell wider loslassen. 
Was keine Lösung ist.

[Amper (Gast)]

Ich habs:

Ich hab noch eine Bool eingesetzt die gesetzt wird wenn ich das Menue 
verlasse und erst wider rückgesetzt wird wenn ich den Tasterloslasse.
Wahrscheinlich gibt es noch eine elegantere Lösung aber mir fällt gerade 
keine ein.

if(statusMenueTaster == LOW){
  if(timeMenueTaster1 < timemils && statusMenue == false){
    statusMenue = true;
    menueOn = true;
    if (menuePunkt==0){
      menueDelay = timeMenueTaster1+950;
    }
  }
}else{
  timeMenueTaster1 = timemils+50;
  statusMenue = false;
  menueEnde = false;
}

...

if(statusMenue == true && menuePunkt == 0 && menueEnde == false){
    if(menueDelay < timemils){
      menuePunkt = 1;
      menueOn = false;
    }
  }

...

if (menuePunkt== 1){
    analogWrite(5 ,255 );
  } else if (menuePunkt== 2){
    analogWrite(6 ,255 );
  } else if (menuePunkt== 3){
    analogWrite(11 ,255 );
  } else if (menuePunkt== 4){
    analogWrite(5 ,0 );
    analogWrite(6 ,0 );
    analogWrite(11 ,0 );
    menuePunkt =0;
    menueEnde = true;
}

[Amper (Gast)]

Manchmal hilft es einfach über ein Problem zu Sprechen um weiter zu 
kommen.

Danke und Grüße
Amper

PS: Kann ich den Beitrag schließen? da erledigt!

[Peter D. (peda)]

Amper schrieb:
> Ich hab noch eine Bool eingesetzt die gesetzt wird wenn ich das Menue
> verlasse und erst wider rückgesetzt wird wenn ich den Tasterloslasse.

Natürlich darf man nicht nur das Drücken auswerten, da man ja auf die 
Flanke reagieren will. Das Loslassen zu prüfen ist also essentiell.

Amper schrieb:
> Wahrscheinlich gibt es noch eine elegantere Lösung aber mir fällt gerade
> keine ein.

Tasten entprellen und Flanke erkennen braucht man sehr oft. Daher lohnt 
es sich, dafür eine extra Task zu implementieren, die dann nur noch die 
fertigen Ereignisse liefert.
Das macht dann auch die Auswertung in der Mainloop deutlich einfacher 
und übersichtlicher.

Z.B.:
Beitrag "Universelle Tastenabfrage"

[Amper (Gast)]

Hallo Peter Dannegger,

vielen Dank für dein Hinweis, hat mir sehr geholfen.

Grüße

[grundschüler (Gast)]

Ich hab auf Taster nahezu ganz verzichtet und mache Eingaben per 
ir-Fernbedienung. Einmal programmiert ist das universell einsetzbar. Nur 
ein Pin und nahezu unbegrenzte Möglichkeiten für Menus.

[Amper (Gast)]

Wenn ich irgendwann mal fertig werde poste ich nen Bild von der Uhr, da 
passt ne Ferhnbedinung einfach nicht zu.
Aber ja hab ich bei meine Lichtsteuerung auch bietet auch mehr 
möglichkeiten wie drei Taster :)

bis dann