Arduino: Finde den Fehler im Sketch nicht, bitte um Hilfe!

[Bastler (Gast)]

Hi,

ich versuche ein Menü mit drei Tasten und einem Nokia 5110-Display zu 
schreiben, es funktioniert nocht nicht viel so wie es soll, aber das 
muss ich alles noch austüfteln, deswegen machen ich das auch, will mich 
mal an was komplexeres auf Anfänger-Niveau wagen.

Erstmal hier der letzte Versuch und danach die eigentliche Frage:

#include <SPI.h>
#include <Adafruit_GFX.h>
#include <Adafruit_PCD8544.h>

Adafruit_PCD8544 display = Adafruit_PCD8544(2, 3, 4, 5, 6);

const int btnPin = 0;
const int ledPin =  13;

const int btnUp = 1;
const int btnOk = 2;
const int btnDwn = 3;

const int btnUpL = 928;
const int btnUpH = 932;
const int btnOkL = 985;
const int btnOkH = 1005;
const int btnDwnL = 970;
const int btnDwnH = 980;

int ledState = HIGH;
int btnState;
int lBtnState = LOW;

long lDbncTime = 0;
long dbncDelay = 500;

//int subMenus = 1;

void setup() {

  pinMode(btnPin, INPUT);
  pinMode(ledPin, OUTPUT);

  Serial.begin(9600);
  display.begin();
  display.setContrast(45);
  display.setTextSize(1);
  display.setTextColor(BLACK);
  display.clearDisplay();

}

void loop() {
  buttons();
}

void buttons() {
  int btnData = analogRead(btnPin);
  Serial.println(btnData);
  int tmpBtnState = LOW;
  if (btnData > btnDwnL && btnData < btnDwnH) tmpBtnState = btnDwn;
  else if (btnData > btnOkL && btnData < btnOkH) tmpBtnState = btnOk;
  else if (btnData > btnUpL && btnData < btnUpH) tmpBtnState = btnUp;
  else tmpBtnState = LOW;
  if (tmpBtnState != lBtnState) lDbncTime = millis();
  if ((millis() - lDbncTime) > dbncDelay) btnState = tmpBtnState;
  lBtnState = tmpBtnState;

  display.setCursor(10, 10);
  display.println("Hauptbild");

  if (lBtnState == btnOk) mainMenu(1);

  display.display();
}

void mainMenu(int subMenus) {

  display.clearDisplay();

  //buttons();

  display.setCursor(2, 2);
  display.println("SubMenu1");
  display.setCursor(13, 2);
  display.println("SubMenu2");
  display.setCursor(24, 2);
  display.println("SubMenu3");
  display.setCursor(35, 2);
  display.println("Back");
  
  switch (subMenus) {

    case 1:
      display.clearDisplay();
      display.drawRect(0, 0, 84, 11, BLACK);
      if (btnState == btnOk) subMenu_0();
      break;

    case 2:
      display.clearDisplay();
      display.drawRect(0, 11, 84, 11, BLACK);
      //if (btnState = btnOk) subMenu_1();
      break;

    case 3:
      display.clearDisplay();
      display.drawRect(0, 22, 84, 11, BLACK);
      //if (btnState = btnOk) subMenu_2();
      break;

    case 4:
      display.clearDisplay();
      display.drawRect(0, 33, 84, 11, BLACK);
      if(lBtnState == btnOk) return;
      break;
  }

  switch (lBtnState) {

    case btnUp:
      subMenus = subMenus - 1;
      if (subMenus < 1) subMenus = 4;
      break;
    case btnDwn:
      subMenus = subMenus + 1;
      if (subMenus > 4) subMenus = 1;
      break;
  }

  display.display();
}

void subMenu_0() {

  display.clearDisplay();
  display.setCursor(2, 2);
  display.println("Ende!");

}

(Haupt)Problem: Die buttons-Funktion funktioniert ansich funktionell, 
auch durch die cases im switch(subMenus) zu blättern funktioniert, aber 
seltsamerweise nur solange sie im Loop stehen, das Aufrufen der 
mainMenu-Funktion geht von überall, aber sobald die Funktion läuft, 
passiert nichts beim Tastendruck. Wie man im Code sehen kann, habe ich 
die buttons-Funktion auch nochmal versucht in der mainMenu-Funktion zu 
starten, was theoretisch und praktisch keinen Sinn macht.
Könnt ihr den Fehler erkennen? Ich stehe total auf dem Schlauch.

Nebenproblem: Wenn das switch-statement im Loop ist und funktioniert, 
reagieren die Tasten viel zu schnell, ich kapier nicht wie ich das 
ausbremsen soll, hatte vor und nach dem Speichern vom temporären in den 
letzten Zustand eine Verzögerung eingebaut, aber das hat nicht 
funktioniert.

Ich würde mich sehr freuen, wenn ihr mich ein wenig unterstützen könnt, 
ich weiss, dass hier einige auf solche Fragen sehr allergisch reagieren, 
daher möchte ich extra erwähnen, dass es mir im Grunde reicht, wenn ihr 
mir die Fehler aufzeigt, bin aber klar nicht unglücklich, wenn ihr mir 
zumindest in ganz normalen Worten eine alternative Lösung anbietet.

Vielen Dank im Voraus,

Bastler

[Stefan F. (Gast)]

> Die buttons-Funktion funktioniert ansich funktionell,
> auch durch die cases im switch(subMenus) zu blättern funktioniert,
> aber seltsamerweise nur solange sie im Loop stehen, das Aufrufen der
> mainMenu-Funktion geht von überall, aber sobald die Funktion läuft,
> passiert nichts beim Tastendruck.

Kannst du dein Problem nochmal auf deutsch beschreiben? Das ist für mich 
total unverständlich. Versuche es mal mit viel kürzeren Sätzen.

[Flip B. (frickelfreak)]

Bastler schrieb:
> aber sobald die Funktion läuft,
> passiert nichts beim Tastendruck

> if (lBtnState == btnOk) mainMenu(1);

die fkt und alle abfragen in dieser funktion werden nur währen des 
drückens von ok ausgeführt. Mal dir mal deine ganze verschachtelung auf. 
Trenne zwischen status, eingabenverarbeitung  und anzeige, alles muss 
regelmässig gemacht werden.

[Peter D. (peda)]

Du brauchst eine Flankenerkennung, da Du ja nur eine Aktion bei jedem 
Drücken ausführen willst und nicht solange Du drückst.
Und Entprellen ist auch sinnvoll.
Es scheint, daß Du die Tasten analog mit Spannungsteiler einliest, hier 
mal ein Beispeil dafür:
Beitrag "Tastenmatrix auslesen über nur 2 Leitungen"

[Bastler (Gast)]

Stefan U. schrieb:
> Kannst du dein Problem nochmal auf deutsch beschreiben? Das ist für mich
> total unverständlich. Versuche es mal mit viel kürzeren Sätzen.

Dafür sind doch die Kommas da, das ist halt das Problem. Wenn ich es im 
Kontext kürzer beschreiben könnte, würde ich das. Die buttons-Funktion 
funktioniert nur im loop, in jeder anderen Funktion nicht.

Flip B. schrieb:
> die fkt und alle abfragen in dieser funktion werden nur währen des
> drückens von ok ausgeführt.

Ne, lBtnState ist nachdem der Taster gedrückt wurde, die Funktion läuft 
dann, dabei wird aber immernoch "Hauptbild" angezeigt und darüber dann 
die Einträge aus der mainMenu-Funktion. Innerhalb der cases reagiert die 
Taste btnOk, aber auch sehr seltsam, wenn ich sie nur kurz drücke, dann 
ruft es die mainMenu-Funktion erneut auf, wenn ich sie länger halte, 
dann das, was im angewählten case steht, die subMenu_0-Funktion. Aber 
ich kann nicht mehr durch die cases switchen.

> Mal dir mal deine ganze verschachtelung auf.

Das habe ich vorher schon, sozusagen, also mir ein Schema aufgestellt, 
deswegen ja, für mich scheint es alles logisch, aber mein Verständnis 
dafür reicht wohl noch nicht aus, um zu erkennen warum.

> Trenne zwischen status, eingabenverarbeitung  und anzeige, alles muss
> regelmässig gemacht werden.

Das verstehe ich nicht.

[Bastler (Gast)]

Peter D. schrieb:
> Du brauchst eine Flankenerkennung, da Du ja nur eine Aktion bei
> jedem Drücken ausführen willst und nicht solange Du drückst.
> Und Entprellen ist auch sinnvoll.
> Es scheint, daß Du die Tasten analog mit Spannungsteiler einliest, hier
> mal ein Beispeil dafür:
> Beitrag "Tastenmatrix auslesen über nur 2 Leitungen"


Ja, es funktioniert ja nach demselben Prinzip, ein Widerstand hängt 
zwischen Gnd und I/O und zwischen +5V und den Taster-Eingängen jeweils 
einer mit anderem Wert, der Ausgang der Taster hängt wiederum auch am 
I/O bzw. Gnd über den ersten Widerstand. Ich habe doch ein btnState und 
l(ast)BtnState, also während und danach und der Sinn der 
buttons-Funktion ist doch Entprellen.

Dein verlinkter Beitrag ist mir noch zu hoch, soweit bin ich noch nicht, 
aber der Link dazu ist gespeichert, danke dafür.