Fehlersuche 2 Tasten - 2 LED / Anfängerfrage

[Randy T. (peroja)]

Hallo,

Ich hab ein kleines Taster Problem.
Ich möchte 2 LED's unabhängig voneinander mittels 2 versch. tasten an 
und aus machen. Also taste 1 drücken Led1 = an und taste 1 nochmal 
drücken und Led1 aus. Das selbe mit Led 2

Nun hab ich das problem dass nur LED1 an und aus geht?
Beide tasten sind entprellt und genau gleich verdrahtet auf m 
breadboard.
LED ist auch richtig drinne....
Wenn jemand lust hat sich schnell meinen code anzusehen?
Wäre echt lieb.

Vielen Dank
Randy

#include <avr/io.h>
#include <util/delay.h>
#include <inttypes.h>
 
#define LED_AN(LED)  (PORTD |=  (1<<(LED)))
#define LED_AUS(LED)  (PORTD &= ~(1<<(LED)))
#define LED_TOGGLE(LED)  (PORTD ^=  (1<<(LED)))

#define LED1  PD0
#define LED2  PD1


// tester 1 

#define TASTER1  PD2
#define TASTER1_GEDRUECKT()  (PIND & (1<<TASTER1))
#define TASTE1_AUF 0
#define TASTE1_ZU  1

// taster 2

#define TASTER2  PD3
#define TASTER2_GEDRUECKT()  (PIND & (1<<TASTER2))
#define TASTE2_AUF 0
#define TASTE2_ZU  1
 

int main(void)
{
  uint8_t alter_tastenzustand1 = TASTE1_AUF;
  uint8_t alter_tastenzustand2 = TASTE2_AUF;
  DDRD &= ~(1<<TASTER1) | (1<<TASTER2);
  DDRD |= (1<<LED1) | (1<<LED2);
  
  
  while(1)
  {

  if (TASTER1_GEDRUECKT() && (alter_tastenzustand1 == TASTE1_AUF))
      {
       LED_TOGGLE(LED1);
  
    alter_tastenzustand1 = TASTE1_ZU;
      }
  
  if (!TASTER1_GEDRUECKT())
      alter_tastenzustand1 = TASTE1_AUF;
  }

// led 2

  if (TASTER2_GEDRUECKT() && (alter_tastenzustand2 == TASTE2_AUF))
      {
     
   
    LED_TOGGLE(LED2);

    alter_tastenzustand2 = TASTE2_ZU;
      }
  
  if (!TASTER2_GEDRUECKT())
      alter_tastenzustand2 = TASTE2_AUF;
  



}

[Karl H. (kbuchegg)]

Rück mal deinen Code ordentlich ein, dann siehst du es selber.

Nach jeder { werden alle nachfolgenden Zeilen um 2 Stellen eingerückt. 
Die jeweils zugehörige } wird dann wieder 2 Stellen ausgerückt.
Und das ziehst du jetzt mal konsequent durch und siehst dir danach an, 
welche Programmteile innerhalb der zentralen while Schleife sind und 
welche nicht.

[Randy T. (peroja)]

Hallo Karl,

Ja Logisch! Wenn ich die Schlaufe zumache nach dem ersten LED innerhalb 
des whiles geht Led2 natürlich nicht mehr!
OK vielen Dank. Problem gelöst!

Randy

[Karl H. (kbuchegg)]

Was du lernen sollst:

Eine ordentliche konsequente Programmstruktur kann dir helfen Fehler zu 
vermeiden.
Also tu es! Nicht am Ende, wenn alles fertig ist, sondern von Anfang an. 
Dazu gehören unter anderem auch saubere Einrückungen. Sauberer Code, der 
auch optisch gut aufgebaut ist, ist ein Entwicklungswerkzeug und eine 
Hilfe. Und du willst doch jede Hilfe annehmen, die du kriegen kannst. 
Noch dazu wenn sie dir nichts kostet ausser ein paar Drücken auf die 
Leertaste.