Tastenentprellung +++ Fehlerhaft

[frabe]

Hallo zusammen.
Einige Schritte weiter, aber noch nicht perfekt - mein universeller "Tastenentpreller ohne delay".
Folgender code klappt manchmal, aber nicht immer.

const int EntprellZeit = 500;   // [ms], Entprellzeit (Testhalber 500ms)

void loop() {
  AusgSekTakt();  //Sek-Takt ohne delay

//Tastenabfrage
  int TasteZustandAlt = 0;        // Tastenzustand einmalig initialisieren

  int TasteZustandNeu = digitalRead(TasteLiPin);  // akt. Zustand der Taste auslesen
  if(TasteZustandNeu != TasteZustandAlt)  {       // wenn Tastenzustand verändert
    if(TasteLiEntprell() >= EntprellZeit)  {      // wenn Taste nach Entprellzeit entprellt
      TasteZustandAlt = TasteZustandNeu;          // neuen Tastenzustand übernommen
      
      TextBlock("Taste im AKT. Zustand entprellt!");
    }
  }
}

//Zeitpunkt der ersten Tasten-Zustandsänderung
unsigned long TasteLiEntprell()  {
  static unsigned long StartZeitEntprell = 0; // [ms] Startzeit
  static int TasteZustand = 0;                // akt Zustand des Tasters
  TZ(1);//Testanzeige

  return Entprellen(TasteLiPin, TasteZustand, StartZeitEntprell);
}

//prüft Eingang auf AusgZustand und Zeit [ms]
unsigned long Entprellen(int Pin, int &TasteZustand, unsigned long &StartZeitEntprell)  { 
  if (digitalRead(Pin) != TasteZustand)  {  // Zustandsänderung des Tasters
    TasteZustand = ! TasteZustand;          // Zustand invertieren
    StartZeitEntprell = millis();           // Startzeit speichern
  }
  if (TasteZustand == 1)                    // wenn Taste gedrückt ist
    return millis() - StartZeitEntprell;    // Zeit zurück geben, seit Taste gedrückt wurde
  else
    return 0;                               // 0 zurück geben, wenn Taste nicht gedrückt, oder noch nicht entprellt
}

Fehler:
1. Sobald erstmalig korrekt 500ms-entprellt, zeigt das Prog bei jedem kleinen (<500ms) Tastendruck "Taste im AKT. Zustand entprellt!" an
2. Wenn die Taste länger als >Entprellzeit gedrückt wird, kommt dauernd "Taste im AKT. Zustand entprellt!",
obwohl folgende Zeile nur die Veränderung reagieren sollte.

"if(TasteZustandNeu != TasteZustandAlt)"

Ich bin mir nicht sicher, ob fälschlicher weise eine dauernde Neuinisialisierung mit "0" erfolgt - sollte eigentlich nur beim Ersten mal init.

 static unsigned long StartZeitEntprell = 0;
 static int TasteZustand = 0;

[wkrug]

Ich vermute mal, der Fehler liegt darin, das der Entprelltimer bei einem als gültig erkannten Tastendruck nicht zurück gesetzt wird.
Die Initialisierung des Timers gehört üblicherweise nicht in den Loop!
Du drückst eine Taste.
Dein Vergleich Neu Alt merkt das, der Entprelltimer ist auch abgelaufen, also wird der Tastendruck erkannt und übernommen.
Die Taste prellt nun - Es wird eine offene Taste erkannt und da der Timer nicht zurück gesetzt wurde auch akzeptiert.
So kann da ein paar mal hin und her gehen.

Würdest Du bei einem als gültig erkannten Tastendruck den Entprelltimer zurücksetzen würde für die nächste Zeit, bis zum Ablauf des Timers kein weiterer Tastendruck als gültig erkannt.

[frabe]

Die Initialisierung des Timers gehört üblicherweise nicht in den Loop!

" AusgSekTakt(); " ist ein einfacher Sek-Takt für einen Summer

und da der Timer nicht zurück gesetzt wurde

tuh ich das nicht mit?

StartZeitEntprell = millis();

- - - Aktualisiert - - -

...oder wo steht der Wald, bei all den Bäumen ...

[inka]

ich meine, ich stelle mir manchmal auch durchaus sportliche aufgaben (siehe mein outdoor-projekt) - aber warum verwendest du nicht die bounce-library?

[Holomino]

//Tastenabfrage
int TasteZustandAlt = 0; // Tastenzustand einmalig initialisieren

Bei der Zeile stimmt ohne "static" schon der Kommentar nicht.

[frabe]

Hallo Inka, danke für den Hinweis
Ich würde den Prozess gerne verstehen... wenn ich diese Logik verstehe, bin ich schon mal einen großen Schritt weiter...
.. und nach Möglichkeit keine zusätzliche Lib einfügen; #include "OneButton.h" ... und mehrer Tastereingänge unabhängig abfragen

(vielleicht befinde ich mich auch auf dem Holzweg

Bei der Zeile stimmt ohne "static" schon der Kommentar nicht.

Hallo Holomio.
"static" innerhalb des loop?
Ist doch eine einmalige Initialisierung - oder was meinst du?

[HaWe]

die loop() ist eine "verkappte" while() / for( ;; ) -Schleife in einer unsichtbaren main() Funktion, in der eine Funktion loop() wiederholt aufgerufen wird - wenn hier Variablen nicht static sind, werden alle Variablenwerte beim erneuten loop()-Durchlauf vergessen.

void setup() { }

void loop() { }


int main() {
   setup();
   for( ;; ) {
      loop();
   }

}

[frabe]

ahaaa! Danke!
Dachte loop() = main()

Mit
static int TasteZustandAlt = 0;
funktioniert es überhaupt nicht.

[HaWe]

dann liegt der Fehler evtl woanders.
Aber deklariere es mal testweise global:
static int TasteZustandAlt = 0;

[frabe]

Klappt leider auch nicht.
Bleibt dann in einer Dauerschleife mit der Serial.print-Ausgabe: TZ(1); im us-Takt hängen.