Tastendruck detektieren

[Gecko (Gast)]

Guten Abend

Ich habe ein kleines Projekt am laufen und steh seit längerem auf der 
Leitung.
Und zwar habe ich an meinem uC zwei Taster angeschlossen. Ich will nun 
detektieren, ob einer der beiden oder beide zusammen gedrückt werden.

Dafür habe ich erst mal einen Flankendetektor geschrieben. Nur kann man 
so nur einen Tastendruck detektieren.
Kann mir jemand sagen, wie ich das "eleganter" realisieren kann?

Hier mein Code:

unsigned char button1_pressed = 1;
unsigned char button2_pressed = 1;

unsigned char button1(void)        
{
  unsigned char return_value = 0;            
  
  if(GPIO_PinInGet(Button1_PORT,Button1_BIT) == 0) 
  {      
    if(button1_pressed == 0)                        
    {
      button1_pressed = 1;                          
      return_value = 1;
    }  
  }
  else                                                         
  {
    button1_pressed = 0;                          
  }
     
  return return_value;                 
}

unsigned char button2(void)        
{
  unsigned char return_value = 0;            
  
  if(GPIO_PinInGet(Button2_PORT,Button2_BIT) == 0) 
  {      
    if(button2_pressed == 0)                        
    {
      button2_pressed = 1;                          
      return_value = 1;
    }  
  }
  else                                                         
  {
    button2_pressed = 0;                          
  }
     
  return return_value;                 
}

Vielen Dank
MFG

[Karl H. (kbuchegg)]

Gecko schrieb:

> Dafür habe ich erst mal einen Flankendetektor geschrieben. Nur kann man
> so nur einen Tastendruck detektieren.

Das Problem ist reichlich knifflig. Denn als Mensch wirst du es niemals 
schaffen, die beiden Tasten 100% exakt gleichzeitig zu drücken.

Was du aber tun kannst:
Wenn du einen Tastendruck feststellst, kannst du kurz einen Blick auf 
den anderen Portpin werfen, ob dort die Taste gerade gedrückt ist. Wenn 
ja, dann sind somit beide Tasten gedrückt.

[Lutz H. (luhe)]

Gecko schrieb:
> unsigned char button1_pressed = 1;

Warum nicht
bool button1_pressed = false;

[Gecko (Gast)]

Ok jetzt habe ich es geschaft. Scheint mir aber ziemlich aufwendig 3 
Unterprogramme zu schreiben, um 3 Fälle zu überprüfen. Mir fällt 
allerdings keine einfachere Alternative ein. Tipp? :)

unsigned char button12(void)        
{
  unsigned char return_value = 0;            
  
  if((GPIO_PinInGet(Button1_PORT,Button1_BIT) == 0)&&(GPIO_PinInGet(Button2_PORT,Button2_BIT) == 0))
  {      
    if(button12_pressed == 0)                        
    {
      button12_pressed = 1;                          
      return_value = 1;
    }  
  }
  else                                                         
  {
    button12_pressed = 0;                          
  }
     
  return return_value;                 
}

unsigned char button1(void)        
{
  unsigned char return_value = 0;            
  
  if(GPIO_PinInGet(Button1_PORT,Button1_BIT) == 0) 
  {      
    if(button1_pressed == 0)                        
    {
      button1_pressed = 1;                          
      return_value = 1;
    }  
  }
  else                                                         
  {
    button1_pressed = 0;                          
  }
     
  return return_value;                 
}

unsigned char button2(void)        
{
  unsigned char return_value = 0;            
  
  if(GPIO_PinInGet(Button2_PORT,Button2_BIT) == 0) 
  {      
    if(button2_pressed == 0)                        
    {
      button2_pressed = 1;                          
      return_value = 1;
    }  
  }
  else                                                         
  {
    button2_pressed = 0;                          
  }
     
  return return_value;                 
}

[Karl H. (kbuchegg)]

Hast du das schon in einem Testprogramm verifiziert?

> Scheint mir aber ziemlich aufwendig 3 Unterprogramme zu schreiben
Na, ja.
Wenn du dir die Funktionen ansiehst, dann ist das immer die im Prinzip 
gleiche Funktion, nur die Variablen sind andere. Genau dafür gibt es 
Funktionsargumente, damit man einer Funktion Werte übergeben kann, mit 
denen sie dann operiert.

Dein C-Buch weiß zu dem Thema näheres.

[Lutz H. (luhe)]

Was soll Funktion zurückgeben: wenn kein Taster gedrückt 0
sonst 1?

 gibt es 4 Zustände:  kein Taster, Taster 1, Taster 2 , Taster 1 und 2
gedrückt.

[F. F. (foldi)]

lutz h. schrieb:
> Was soll Funktion zurückgeben: wenn kein Taster gedrückt 0
> sonst 1?
>
>  gibt es 4 Zustände:  kein Taster, Taster 1, Taster 2 , Taster 1 und 2
> gedrückt.

Ist das nicht schon die Antwort auf deine Frage?

[Lutz H. (luhe)]

unsigned char button12(void)
{
  unsigned char return_value = 0;

//  if((GPIO_PinInGet(Button1_PORT,Button1_BIT) == 0)&& 
//(GPIO_PinInGet(Button2_PORT,Button2_BIT) == 0)) {return_value = 0;}
  if((GPIO_PinInGet(Button1_PORT,Button1_BIT) == 1)&& 
(GPIO_PinInGet(Button2_PORT,Button2_BIT) == 0)) {return_value = 1;}
  if((GPIO_PinInGet(Button1_PORT,Button1_BIT) == 0)&& 
(GPIO_PinInGet(Button2_PORT,Button2_BIT) == 1)) {return_value = 2;}
  if((GPIO_PinInGet(Button1_PORT,Button1_BIT) == 1)&& 
(GPIO_PinInGet(Button2_PORT,Button2_BIT) == 1)) {return_value = 3;}
  return return_value;
}

[Max H. (hartl192)]

So würde eine Flankenerkennung (fallen) für einen Gnazen Port gehen. Ist 
zwar für PIC, dürfte sich aber leicht auf einen AVR übertragen lassen.

PORTBnew=PORTB;
flankeB=~PORTBnew;
flankeB=flankeB&PORTBold;
PORTBold=PORTBnew;

[F. F. (foldi)]

M. H. schrieb:
> So würde eine Flankenerkennung (fallen) für einen Gnazen Port gehen. Ist
> zwar für PIC, dürfte sich aber leicht auf einen AVR übertragen lassen.
>
>

> PORTBnew=PORTB;
> flankeB=~PORTBnew;
> flankeB=flankeB&PORTBold;
> PORTBold=PORTBnew;
>

Sicher schön kurz, aber wenn er nur die zwei Taster auswerten will und 
muss, den Rest des Ports noch braucht, dann muss er eh noch was 
anhängen.

Der letze Code war doch schon recht kurz.

[F. F. (foldi)]

lutz h. schrieb:
> unsigned char button12(void)
> {
>   unsigned char return_value = 0;
>
> //  if((GPIO_PinInGet(Button1_PORT,Button1_BIT) == 0)&&
> //(GPIO_PinInGet(Button2_PORT,Button2_BIT) == 0)) {return_value = 0;}
>   if((GPIO_PinInGet(Button1_PORT,Button1_BIT) == 1)&&
> (GPIO_PinInGet(Button2_PORT,Button2_BIT) == 0)) {return_value = 1;}
>   if((GPIO_PinInGet(Button1_PORT,Button1_BIT) == 0)&&
> (GPIO_PinInGet(Button2_PORT,Button2_BIT) == 1)) {return_value = 2;}
>   if((GPIO_PinInGet(Button1_PORT,Button1_BIT) == 1)&&
> (GPIO_PinInGet(Button2_PORT,Button2_BIT) == 1)) {return_value = 3;}
>   return return_value;
> }

Jetzt noch ein kurzes delay dazwischen und du kannst wirklich erkennen, 
ob da jemand beide Tasten gedrückt hat.

[Karl H. (kbuchegg)]

F. Fo schrieb:

> Sicher schön kurz, aber wenn er nur die zwei Taster auswerten will

schlimmer.

Er hat 2 physikalische Taster, will aber 3 logische Taster.

[Max H. (hartl192)]

Die Pins werden nur eingelesen und nicht beschreiben. Wenn man wie ich 
nur 3 Tasten auswerten will werden die andern Bits der Variable flanke 
einfach ignoriert.

if(flankeB&(1<<1))      
{

}  
if(flankeB&(1<<2))      
{

}    
if(flankeB&(1<<4))      
{

}

[Karl H. (kbuchegg)]

M. H. schrieb:
> Die Pins werden nur eingelesen und nicht beschreiben. Wenn man wie ich
> nur 3 Tasten auswerten will

Nochmal.
Er hat keine 3 Taster.
Er hat nur 2 real vorhandene.

Er will aber im Programm 3 Tastenzustände unterscheiden:
A gedrückt
B gedrückt
A + B gedrückt.

[Max H. (hartl192)]

Karl Heinz schrieb:
> M. H. schrieb:
>> Die Pins werden nur eingelesen und nicht beschreiben. Wenn man wie ich
>> nur 3 Tasten auswerten will
>
> Nochmal.
> Er hat keine 3 Taster.

Ich habe die 3 Tasten nur als Beispiel genommen weil ich das in meinem 
Code so hatte.
Und um das mit den 2 gleichzeitigen Tasten zu erkennen muss man ein 
längeres delay in die schleife einbauen und das 3. if so machen:

if(flankeB&(1<<1)&&flankeB&(1<<2))

Oder man merkt sich die Flanke für ein paar Schleifendurchläufe.

[F. F. (foldi)]

Karl Heinz schrieb:
> M. H. schrieb:
>> Die Pins werden nur eingelesen und nicht beschreiben. Wenn man wie ich
>> nur 3 Tasten auswerten will
>
> Nochmal.
> Er hat keine 3 Taster.
> Er hat nur 2 real vorhandene.
>
> Er will aber im Programm 3 Tastenzustände unterscheiden:
> A gedrückt
> B gedrückt
> A + B gedrückt.

Das geht nur, wie du schon angedeutet hast, wenn er die "menschliche 
Komponente" mit einbezieht. Also muss er den Zeitrahmen vorgeben und 
auch in der Abfrage erfassen, dazu noch ein Sicherheitsmechanismus, der 
alle Abweichungen als ungültig erklärt.
Da ist die länge des Codes eher Nebensache.

[Max H. (hartl192)]

while(1)
{
  PORTBnew=PORTB;
  flankeB=~PORTBnew;
  flankeB=flankeB&PORTBold;
  PORTBold=PORTBnew;

  if(flankeB&(1<<1))      
  {
     zaeler1=20;
  }  
  if(flankeB&(1<<2))      
  {
     zaeler2=20;
  }
  if(zaeler1&&zaeler2)
  {
     //beide gedrückt
  }
  if(zaeler1>0)
     zaeler1--;
  if(zaeler2>0)
     zaeler2--;
  
  Delay_ms(10);  
}

Das wäre eine mögliche Lösung. Wenn man die schleife z.B. alle 10ms 
ausführt dann kann man erkennen ob innerhalb von 200ms beide Tasten 
gedrückt wurden. Ich persönlich würde den Code nicht als schleife 
sondern in einem Timer Interrupt ausführen.
Die variable zaeler merkt sich 200ms lang ob eine Taste gedrückt wurde. 
Wurden beide Tasten einmal in den letzten 200ms gedrückt ist die 
Bedingung "zaeler1&&zaeler2" wahr.

[Martin V. (oldmax)]

Hi
Immer, wenn's etwas kompliziert erscheint, hilft eine Skizze. In diesem 
Fall ein Signalbild. Ich denke, das sagt mehr als tausend Worte.
Gruß oldmax

[Peter D. (peda)]

Beitrag "Universelle Tastenabfrage mit 2 Tastenerkennung"