Arduino Beste Lösung meinen Code um einen Taster zu erweitern

[Ardu (Gast)]

Hi,

ich will einen weiteren Taster für "timeset - 1" nutzen, bin aber 
unsicher, wie man das am besten löst, wie es funktonieren kann ist mir 
klar, aber eben nicht, obs dann so ordentlich ist, würde den gesammten 
digitalRead-Prozess einfach kopieren, den Taster-Pin ändern und aus dem 
+ ein - machen. Denke das müsste irgendwie mit nem else kürzer gehen, 
komm aber nicht dahinter wie. Hoffe ihr könnt mir da mal eben helfen.

Mfg

      case 1: //Hour setting
        if (flash) {
          display.drawRect(15, 0, 13, 9, BLACK); //Flash cursor (5Hz)
        }
        if (!digitalRead(8) && (!flash)) { //Update setting at 5Hz rate if button held down
          timeset = t.hour; //Get the current hour and save in temporary variable
          timeset = timeset + 1; //Increment the hour at 5Hz rate
          if (timeset > 23) {
            timeset = 0; //Roll over hour after 23rd hour (setting done in 24-hour mode)
          }
          t.hour = timeset; //After each update, write the hour to the time structure
          set_rtc_field(t, hourset); //Write the set field only to the RTC after each update
        }
        break;

[Sascha (Gast)]

Da das ganze ja scheinbar in 200ms Intervallen aufgerufen wird: Mach das 
so.

Begründung: Kostet praktisch keine Performance.

Und für Code der nur die Uhrzeit einstellen soll, muss man jetzt auch 
kein riesiges Fass aufmachen.

Wenn du mal was anderes hast: Normal liest man regelmäßig oder per 
Pin-INT alle Taster ein, speichert die in ne Variable und verwendet die 
dann im Code.

Hat den Vorteil dass Taster entprellen und einlesen zeitlich entkoppelt 
sind und dass man Programmlogik einfach von der Hardware abstrahieren 
kann.

Gerade bei OO Programmierung wichtig.

Bei kleineren Projekten kann man sich das aber auch mal ausnahmsweise 
schenken.

[Michael U. (amiga)]

Hallo,

denke aber daran, daß Du das auch anpassen mußt:

          if (timeset > 23) {
            timeset = 0; //Roll over hour after 23rd hour (setting done in 24-hour mode)
          }

Gruß aus Berlin
Michael

[Ardu (Gast)]

Danke.
Ich lerne noch, das hat nicht wirklich alles einen Sinn, noch nicht, 
will halt erstmal ein paar Funktionen und Methoden lernen.
Also die übliche Methode, die man zu Tastern allgemein so findet... ist 
wohl doch besser. Hab da noch ne "flash"-Funktion im Code, bin noch 
nicht sicher, wie wichtig das ist und wie ich eine saubere 
Aktualisierung sonst hinkriege, ohne dass das Display beim flackert, die 
flash-Funktion gehört zum Code-Teil, den ich irgendwo gefunden habe.

[Ardu (Gast)]

Ja Michael, habs nur vergessen zu erwähnen, danke.

[Sascha (Gast)]

!flash ist aber das Abprüfen einer Variablen. Das ist keine Methode, die 
da aufgerufen wird.