Tastenabfrage funktioniert nicht

[Andreas S. (xevious)]

Hi!

Bin nach wie vor Anfänger und hab eine kurze Frage:

Ich hab vor, die Blinker von meinem Motorrad mit Tastern ein- und 
auszuschalten (mit ATTiny 2313). Das Blinken hab ich noch nicht drin.

Einschalten der Blinker über die jeweiligen Richtungstasten (hier 
Eingang PB0 und PB1).
Ausschalten über einen Tastendruck einer beliebigen Taste (von den 2).
Ich hab 4 Zustände abgefragt (Blinker ein: Taste drücken, Taste 
loslassen, Blinker aus: Wieder Taste drücken, Taste loslassen)

Problem:

Einschalten geht mit beiden Tastern wie erwartet, aber ausschalten kann 
ich nur über den Eingang PB0.

Warum?

Danke!!



#include <avr/io.h>
volatile int i=0;
int main (void){
  DDRB=0x00;
  PORTB=0xfc;
  DDRD=0xff;
  PORTD=0x00;
while(1){

if (i==0){ //BLINKER EIN
  if (PINB & (1<<PB0)){
    PORTD |= (1<<DDD0);
    i=1;
    }
  if (PINB & (1<<PB1)){
    PORTD |= (1<<DDD1);
    i=1;
    }
}

if ((i==1)&(!(PINB&0x03))){//Taster los1
  i=2;
  }

if ((i==2)&(PINB&0x03)){//BLINKER AUS
  PORTD &= ~((1<<PD0)|(1<<PD1));
  i=3;
  }

if ((i==3)&(!(PINB&0x03))){//Taster los2
  i=0;
  }

}
}

[bitte löschen (Gast)]

Mach es doch mal hübsch, und ein paar Kommentare dazu. So z.B.:

#include <avr/io.h>

volatile int i=0;

int main (void)
{
  DDRB=0x00;
  PORTB=0xfc; // Keine Pullups an den beiden Eingängen
  DDRD=0xff;
  PORTD=0x00; // Blinker anfangs aus
  while(1)
  {
    if (i==0) // Blinker ist aus
    { // Prüfe auf BLINKER EIN
      if (PINB & (1<<PB0)) // Taste links gedrückt
      {
        PORTD |= (1<<DDD0); // Linken Blinker an
        i=1;        // An-Flag setzen
      }

      if (PINB & (1<<PB1)) // Taste rechts gedrückt
      {
        PORTD |= (1<<DDD1); // Rechten blinker (auch!) an
        i=1;
      }
    }

    // Wenn Ein Blinker an ist, und keine Blinker-Taste gedrückt ist, 
    // setze i = 2
    // if ((i==1) & (!(PINB&0x03))) // Resultat abhängig von Compiler 
    //  (TRUE immer gleicher Zahlenwert?)
    // Besser:
    if ((i==1) && (!(PINB&0x03)))
    { //Taster los1
      i=2;
    }

    // Wenn i == 2 ist (Blinker ist an, und alle Tasten wurden schon
    // wieder losgelassen) und wenn keine Taste mehr gedrückt ist, 
    // dann Schalte Blinker wieder aus und setze i = 3.
    // if ((i==2) & (PINB&0x03)) // Siehe oben
    if ((i==2) && (PINB&0x03))
    {//BLINKER AUS
      PORTD &= ~((1<<PD0)|(1<<PD1));
      i=3;
    }

    // Wenn i == 3 ist, und keine Taste gedrückt ist, 
    // wurde der Blinker ausgeschaltet, und es geht wieder von vorne los
    // if ((i==3) & (!(PINB&0x03))) // Siehe oben
    if ((i==3) && (!(PINB&0x03)))
    {//Taster los2
      i=0;
    }
  }
}

Was passiert, wenn Du die Änderungen bezüglich & und && übernimmst?

[Andreas S. (xevious)]

DANKE DANKE, jetzt gehts. Das && wars wirklich!

Ich hab mich nach der AVR-GCC Anleitung gehalten, aber da hab ich über 
&& nichts gelesen (ansonsten toll verständlich).

[bitte löschen (Gast)]

Gerne. :-)