LED mit Taster ein/aus schalten

[Daniel L. (daniel_l49)]

Hallo hab hier mal ein kleines Programm geschrieben, aber es tut nicht 
was ich will

Es soll per Tasten druck eine LED an bzw: ausgeschaltet werden.

Ich habe keinen Plan was ich noch falsch mache.

#include <asf.h>
#define F_CPU 3686400UL
#include <avr/io.h>
#include <interrupt.h>
#include <util/delay.h>
#include <stdio.h>

#define taste_gedruckt       1 
#define taste_nicht_gedruckt 0


int main (void)
{
  
  DDRB = 0xff;
  DDRD = 0x00;
  PORTD = 0xff;
  
  int i = 0;
  int taste_jetzt;
  int taste_vorher; 
  
    
  while (1)
  {
    
    if(!(PIND & (1<<PD2)))
    {
      taste_jetzt  = taste_gedruckt;  
      taste_vorher = taste_nicht_gedruckt;
    }
    else
    taste_jetzt  = taste_nicht_gedruckt; 
    taste_vorher = taste_gedruckt;
    
        
  
    if(taste_vorher != taste_jetzt)
    {
      if (taste_jetzt == taste_gedruckt)
      {
        PORTB ^= (1<<PB0);  
      }
    } taste_vorher = taste_jetzt
  }
}

mfg Daniel

[Daniel L. (daniel_l49)]

hab noch etwas vergessen

if(!(PIND & (1<<PD2))) // wenn Taste gedrückt

[Dark Maul (Gast)]

Daniel L. schrieb:
> }
>     else
>     taste_jetzt  = taste_nicht_gedruckt;
>     taste_vorher = taste_gedruckt;

lol... überleg dir mal ob da vielleicht Klammern fehlen.

[Karl H. (kbuchegg)]

Daniel L. schrieb:

>     if(!(PIND & (1<<PD2)))
>     {
>       taste_jetzt  = taste_gedruckt;
>       taste_vorher = taste_nicht_gedruckt;
>     }

Ho, ho, ho


Nur weil eine Taste jetzt gedrückt ist, bedeutet das nicht, dass die 
Taste vorher nicht gedrückt war.

'Schläft' dein Benutzer auf der Taste ein, dann stellst du viele tausend 
mal hintereinander fest, dass die Taste gedrückt ist und auch in der 
Abfrage zuvor gedrückt war.

Und 'by the way': egal wie schnell  dein menschlicher Benutzer ist, aus 
Sicht deines µC ist alles was er tut eine Super-Zeitlupe. Dein µC ist 
millionenfach schneller als selbst der schnellste menschliche Benutzer.

Wenn du also fragst, was du falsch gemacht hast: Du hast die 
Leistungsfähigkeit deines µC grob unterschätzt.

#include <asf.h>
#define F_CPU 3686400UL
#include <avr/io.h>
#include <interrupt.h>
#include <util/delay.h>
#include <stdio.h>

#define taste_gedruckt       1 
#define taste_nicht_gedruckt 0


int main (void)
{
  
  DDRB = 0xff;
  DDRD = 0x00;
  PORTD = 0xff;
  
  int i = 0;
  int taste_jetzt;
  int taste_vorher; 
  
    
  if(!(PIND & (1<<PD2)))
    taste_vorher = taste_gedruckt;
  else
    taste_vorher = taste_nicht_gedruckt; 

  while (1)
  {
    
    if(!(PIND & (1<<PD2)))
    {
      taste_jetzt  = taste_gedruckt;  
    }
    else
      taste_jetzt  = taste_nicht_gedruckt; 
    
    if(taste_vorher != taste_jetzt)
    {
      if (taste_jetzt == taste_gedruckt)
      {
        PORTB ^= (1<<PB0);  
      }
      taste_vorher = taste_jetzt
    }
  }
}

Allerdings:
je nachdem wie gut deine Tasten mechanisch beisammen sind, wirst du ein 
neues Problem feststellen: Taster prellen

Entprellung

[Max H. (hartl192)]

Siehe Entprellung
Ich mache Flankenerkennung (steigend) immer so:

while(1)
{
    PINDnew=PIND;
    flankeD=~PINDnew;
    flankeD=flankeD&PINDold;
    PINDold=PINDnew;
    
    if(flankeD&(1<<4))      // PD4

}
Die Variablen sind unsigend char
Zum Entprellen würde ein Delay in der while helfen, ich führe die 
Routine aber immer in einem Timer Interrupt aus.

[Daniel L. (daniel_l49)]

Oke danke für die Antworten

Es funktioniert schon