Tastenabfrage in C

[DerSchatten (Gast)]

Ich versuche gerade in C eine Tastenabfrage zu erstellen.
Bei kurzem Druck soll eine Variable hochgezählt werden. Bei langem Druck 
soll die aktuelle Variable ins EEPROM gespeichert werden:

while(1)
  {
    PORTD = pgm_read_byte(&Tabelle[nKeyPress]);
    if (bit_is_clear (PINB, PINB0))
    {
      if (bPortB) // wenn Taster gedrückt
      {
        dauer++;
          }
      else // wenn nicht gedrückt
      {
        if   (dauer > 10)
        {
          if (nKeyPress < 8)
          {
            nKeyPress++;
            bPortB = 0;
          }
          else
          {
            nKeyPress = 0;
            bPortB = 0;
          }
        }
        if   (dauer > 20) // wenn länger gedrückt
        {
          eeprom_write_byte(&eeFooByte, nKeyPress);
          dauer = 0;
          Warte (100);
        }
      }
      }
      else
    {
          bPortB = 1;
      }
  }
  return 0;

Ich komme nur einfach nicht klar damit.
Ist das der richtige Weg?

[Karl H. (kbuchegg)]

DerSchatten schrieb:

> Ich komme nur einfach nicht klar damit.
> Ist das der richtige Weg?

Dein Fragnment ist gar nicht so einfach zu analysieren.
Alles in allem erscheint es mir aber ünverhältnismässig kompliziert für 
die Funktion die es bereitstellt.

Benutze doch einfach die PeDa Entprellung. Die macht das was du vorhast, 
aus dem Stand.
http://www.mikrocontroller.net/articles/Entprellung#Komfortroutine_.28C_f.C3.BCr_AVR.29

[Karl H. (kbuchegg)]

Dein Hauptproblem wird wohl darin liegen, dass

     else // wenn nicht gedrückt
      {
        if   (dauer > 10)

dieser else Zweig nie betreten wird, da bPortB gar keine Chance hat 
jemals 0 zu werden. Dei einzigen Möglichkeiten dafür befinden sich 
allesamt im else Zweig selber.

[Ch D. (chrisu)]

So kann ich das empfehlen, für jeden einfacher(es geht nur um die 
Formatierung.)

while(1){
        PORTD = pgm_read_byte(&Tabelle[nKeyPress]);
        if (bit_is_clear (PINB, PINB0)){
                if (bPortB){ // wenn Taster gedrückt
                        dauer++;
                }
                else{ // wenn nicht gedrückt
                        if(dauer > 10){
                                if (nKeyPress < 8){
                                        nKeyPress++;
                                        bPortB = 0;
                                }
                                else{
                                        nKeyPress = 0;
                                        bPortB = 0;
                                }
                        }
                        if(dauer > 20){ // wenn länger gedrückt
                                eeprom_write_byte(&eeFooByte, nKeyPress);
                                dauer = 0;
                                Warte (100);
                        }
              }
        }
        else{
                bPortB = 1;
        }
}
return 0;

[DerSchatten (Gast)]

Hallo Karl,

hab mir das Beispiel angesehen und ausprobiert.
Erstmal 1:1 übernommen.

Das einzige was jedoch passiert ist, das die 3 LEDs sofort zu leuchten 
beginnen, mehr nicht.
So wie ich das verstanden habe sollen die ja erst leuchten wenn einer 
der 3 Taster gedrückt wird oder?
Was bewirkt hier ein langer Tastendruck?

[Tastenpreller (Gast)]

Hallo DerSchatten,

bitte stelle doch mal den "neuen" Code hinein, sonst kann man den Fehler 
nicht suchen.