Ausschalten von LED nach dreimaligem Taster drücken

[Svenja S. (lavy7)]

Hallöchen alle miteinander :)

Ich möchte gerne eine LED "steuern", indem ich drei mal auf den Taster 
drücke und diese an geht. Nach erneutem dreimaligen Drücken soll sie 
wieder ausgehen und immer so weiter.
Das Einschalten funktioniert soweit ganz gut, jedoch weiß ich nicht, wie 
ich in dieser Art die LED wieder ausschalten soll. Ich hab es damit 
probiert, die Variable var wieder zu dekrementieren, jedoch kommt dabei 
nicht wirklich etwas gescheites raus.

Hier mein Code um die LED einzuschalten:

#include <avr/io.h>
#include <avr/interrupt.h>
int main(void)
{  
  PORTD = (1<<PD2); 
  DDRB |= (1<<PB0);
  uint8_t var = 0;
  uint8_t taster_vorher = 0;
  uint8_t taster_jetzt = 0;

  while(1)
  {
    if ( (PIND & (1<<PD2)) )
    {
      taster_jetzt = 1;
    }
    else
    {
      taster_jetzt = 0;
    }
    if ( taster_vorher == 0  &&  taster_jetzt == 1 )
    { 
      var++;
    }  
    if(var == 3)
    {
      PORTB |= (1<<PB0);
    }
     taster_vorher = taster_jetzt;  
  }
 }

Entprellt habe ich mittels Kondensatoren.

Liebe Grüße,
Lavy!

[Udo (Gast)]

Du hast keinen Befehl die LED auszuschalten:
PORTB &= ~(1<<PB0);

Führe eine Variable ein, die den Status der LED widerspiegelt, ändere 
die abfrage auf 3 durch eine Modulo-Divisions-Abfrage auf 0 und toggle 
dann die LED anhand deren Status.


Nebenbei: Willst du einen Triple-Klick erkennen, oder nur die Anzahl der 
Klicks zählen und dann reagieren? Weil für ersteres müsstest du die 
Klicks
nach einer gewissen Zeit resetten...

[fop (Gast)]

Svenja S. schrieb:
> if(var == 3)
>     {
>       PORTB |= (1<<PB0);
>     }

hier schaltest Du immer ein, wenn 3 Mal drücken erreicht wurde. Wie wäre 
es, wenn Du den Zustand der Led umkehrst, wenn 3 Mal drücken erreicht 
wurde ?
Ok, da lauert noch ein kleiner Fallstrick : die Led fängt an zu glimmen 
nach 3 Mal drücken. Wenn man erst mal weiss warum, ist es klar. Der 
Zähler steht noch auf 3 Mal gedrückt, und die Led wird in jedem 
Durchlauf an oder aus geschaltet. Bisher hast Du sie immer wieder 
eingeschaltet, was bei einer bereits eingeschalteten Led nichts macht, 
also nicht auffällt.

Die Idee von Udo ist deshalb nicht so prickelnd, weil jede Variable 
einen maximalen Wert hat. Bei C geht das Zählen danach ohne weitere 
Warnungen wieder von vorne los. Wenn also jemand es schafft 256 mal zu 
drücken, bevor Dein Programm neu gestartet wird, passiert was 
Unerwartetes weil 256 nunmal nicht ohne Rest durch 3 teilbar ist. 
Überhaupt will ja niemand wissen, wie oft der Taster seit Programmstart 
gedrückt wurde. Das macht die Sache nur unnötig kompliziert so mit 
Modulo und so. Einfach jedesmal bis 3 zählen und gut. Klappt schon im 
Kindergarten...

Achtung Spoiler :

.

.

.

.

    if(var == 3)
    {
      PORTB ^= (1<<PB0);
      var = 0;
    }

[Svenja S. (lavy7)]

Hey super, vielen Dank!
Manchmal ist die Lösung zu einfach um darauf zu kommen ^^'

Mal aus Interesse: Ich verwende einen ATMega8. Gibt es dort vielleicht 
eine interne Funktion, die man dafür ausnutzen könnte?
Also um dann aus dem Code quasi einen Dreizeiler zu machen?

Liebe Grüße
Lavy :)

[EAF (Gast)]

fop schrieb:
> if(var == 3)
>     {
>       PORTB ^= (1<<PB0);
>       var = 0;
>     }

Für die meisten/modernen AVR reicht hier  PINB = (1<<PB0); statt  PORTB 
^= (1<<PB0);

[fop (Gast)]

Du kannst dafür den Timer/Counter 2 im CTC Modus mit externem 
Takteingang nutzen. Die Initialisierung passt eventuell nicht in 3 
Zeilen, wenn es dann mal läuft, läuft es ohne eine einzige Codezeile.

[LostInMusic (Gast)]

>Also um dann aus dem Code quasi einen Dreizeiler zu machen?

Dahin kommst Du, wenn Du in Deinem Programm die Zeilen 21 bis 
(einschließlich) 28 änderst in...

    if ( taster_vorher == 0  &&  taster_jetzt == 1 )
    { 
    KeyPress
    }

...und in die Funktion "KeyPress" den entsprechenden Code auslagerst:

    var++;
    if(var == 3)
    {
    LEDState = !LEDState;
    var = 0;
    }

Plus noch an geeigneter Stelle eine Zeile, die sich darum kümmert, dass 
die boolesche Variable LEDState mit Pin PB0 auf dem Port PORTB verknüpft 
wird. Bekommst Du sicher selber hin.

Den Inhalt von "KeyPress" könnte man nun als "Anwendung" interpretieren 
und alles andere als "Betriebssystem" (beide Begriffe in einem sehr 
weiten Sinn). Wie Du siehst, ist die "Anwendung" dann tatsächlich nur 
ein ganz einfacher Vierzeiler.

[Frank M. (ukw) (Moderator)]

Svenja S. schrieb:
> Ich verwende einen ATMega8

EAF schrieb:
> Für die meisten/modernen AVR reicht hier  PINB = (1<<PB0); statt  PORTB
> ^= (1<<PB0);

Meiner Erinnerung nach ist der ATMega8 zu alt dafür. Ein ATMega88 
hingegen könnte das.

[EAF (Gast)]

Frank M. schrieb:
> Meiner Erinnerung nach ist der ATMega8 zu alt dafür.
Durchaus!
Allerdings, kam diese Salamischeibe quasi gleichzeitig mit meinem 
Posting.

Ansonsten, würde ich mich mit solchen Antiquitäten nur unter Zwang 
beschäftigen

[Svenja S. (lavy7)]

Nochmal vielen Dank an alle :)

Hab es jetzt auch mit Interupts probiert und funktioniert alles wie es 
soll!

Ich wünsche euch noch eine schöne Woche :)

LG
Lavy

[Guido K. (Firma: Code Mercenaries GmbH) (thebug)]

So was programmiert sich komfortabler als State-Machine (finiter 
Automat).

Also kein linearer Algorithmus, der hin und her hüpft, sondern eine 
Funktion, die periodisch aufgerufen wird und dann abhängig vom Status 
die Eingangswerte auswertet und ggf. den Status wechselt.

Wenn man nur ein/aus hätte, dann würde der "aus" Status, also State 0, 
prüfen, ob die Taste gedrückt wurde. Wenn das so ist, schaltet er die 
Lampe ein und wechselt zum nächsten Status (1), der dann jedes mal beim 
Aufruf prüft, ob die Taste wieder losgelassen wurde, dann wechselt er 
zum Status 2, der guckt, ob die Taste wieder gedrückt wurde, dann die 
Lampe ausschaltet, zum Status 3 wechselt, der auf das Loslassen wartet 
und zum Status 0 zurück kehrt.

Solche State-Machines kann man beliebig komplex machen. Am Besten ruft 
man die in einem Interrupt auf, wenn etwas passiert ist, oder periodisch 
über einen Timer. Die einzelnen States sind in einem switch/case was die 
Funktionen klein, handlich und übersichtlich macht.

ggf. mal was über finite Automaten lesen. Das ist ein sehr praktisches 
Werkzeug.

[Peter D. (peda)]

Ich würde zum Einschalten fünfmal drücken, dann reicht zum Ausschalten 
einmal drücken.

[EAF (Gast)]

Da fehlt noch ein "Zeitfenster".
Oder darf das drei mal drücken auch mit 1 Wochen Abstand zwischen den 
Drücken funktionieren?

[Egonwalter M. (heiner1234)]

Peter D. schrieb:
> Ich würde zum Einschalten fünfmal drücken, dann reicht zum Ausschalten
> einmal drücken.

+1 ;-)

Du musst Dir aber dann merken (oder aufschreiben), dass Du schon 5mal 
gedrückt hast ;-)

[LostInMusic (Gast)]

>Solche State-Machines kann man beliebig komplex machen.

Oh ja! Besser ist es aber, sie so einfach wie möglich zu machen :-)

[Dirk (Gast)]

Hallo Svenja,

hat dieses 3 malige betätigen einen Hintergrund?
Es macht nach nur dann Sinn wenn Du die Taste 3x innerhalb eines 
Zeitraumes mit Timeout betätigst.

Alternativ wäre die Länge eines Tastendrucks eine Lösung.