State Machine Diagramm

[H. G. (ledi)]

Hallo,

ich habe eine Frage zu einem Diagramm für eine state machine.

Eine ISR wird alle 30ms ausgelöst und ein Taster wird abgefragt.
Bei einem langen Tastendruck soll LED1 toggeln
Bei einem kurzen Tastendruck soll LED2 toggeln

Ist das Diagramm so wie ich es gezeichnet habe korrekt?

[Mac (Gast)]

Was du da gezeichnet hast ist ein Flußdiagram und keine Statemachine.

[Falk B. (falk)]

Das ist kein statemachine Diagramm sondern ein Flußdiagramm, das ist 
was anderes.

Die erste Frage müsste eher lauten "Wurde ein Taster losgelassen?".

[NurEinGast (Gast)]

Ich finde - nein.

FlagLang FlagKurz muessen irgendwo wieder zurückgesetzt werden. Sonst 
flackern die LED's nur so vor sich hin.

Und den Rest hat MAC und Falk schon geschrieben.
z.B. -> Wo geht Dein Automati z.B. nach "Flag kurz setzen" oder "Led 1 
On" als nächstes hin ?

[Thomas (Gast)]

Hinter "Start ISR alle 32ms" darf es nicht heißen "Wurde eine Taste 
gedrückt?". Wenn die Taste inzwischen wieder losgelassen wurde, läßt 
sich das nämlich gar nicht feststellen. Sonst müßte der Tastendruck 
einen Interrupt auslösen. An der Stelle läßt sich also nur die Frage 
"Ist eine Taste gedrückt?" beantworten.

[Udo S. (urschmitt)]

Ist es eigentlich so schwer seinem Thread eine sinnvolle, auf das 
eigentliche Thema weisende Überschrift zu geben?

Offensichtlich schon
Angenervt
Udo

[Gunb (Gast)]

Eine state machine besteht aus Zuständen, die man z.B. durch Kreise 
symbolisiert, in die man diese dann hineinschreibt.

Der Übergang zwischen Zuständen, die transitions, werden durch Pfeile 
symbolisiert, die Bedingung für den Zustandswechsel ordnet man dann dem 
jeweiligen Pfeil zu.

Was du gezeichnet hast, ist tatsächlich eher ein Flussdiagramm, dass in 
dieser Form auch nicht besonders leicht lesbar ist.

Such hier in der Artikelübersicht mal nach State Machines oder 
Zustandsautomaten, da wird das schön anhang einer Ampelsteuerung 
erklärt.

Es gibt eine etwas andere Notation, wenn man nach UML arbeitet, wird 
gerne bei komplexeren meist objektorientierten Ansätzen genommen. Ist 
für deinen Fall aber sicher nicht nötig, also, hier suchen und neu 
anfangen.

[Karl H. (kbuchegg)]

Thomas schrieb:
> Hinter "Start ISR alle 32ms" darf es nicht heißen "Wurde eine Taste
> gedrückt?". Wenn die Taste inzwischen wieder losgelassen wurde, läßt
> sich das nämlich gar nicht feststellen. Sonst müßte der Tastendruck
> einen Interrupt auslösen. An der Stelle läßt sich also nur die Frage
> "Ist eine Taste gedrückt?" beantworten.

Langsam.
Wenn das zb auf den Tastenroutinen vom PeDa beruht, dann geht das 
durchaus.

Ansonsten ist natürlich der ganze Kreis "war es ein langer Tastendruck" 
recht sinnfrei. Denn genau da besteht ja die Problemtik am ganzen. Die 
PeDa Routinen erledigen das. Allerdings hat er nicht gesagt ob er da was 
benutzt, oder ob er selber die Tasten auswertet.

[H. G. (ledi)]

Also der Code dazu sieht so aus:

/* Tasterzustandserkennung Tastendruck kurz / lang
*  ATtiny2313 @ 8MHz
*  mittels Watchdogtimer Overflow Interrupt (alle 32ms)
*  Tastendruck = kurz --> LED am PB4 toggeln
*  Tastendruck = lang --> LED am PB3 toggeln
*/

#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/wdt.h>

#define button_down  !(PINB & (1<<PB0))  // Button = Low
#define button_up  (PINB & (1<<PB0))  // Button = High
#define debounce_time    2      // Entprellzeit = 32ms x 2
#define short_keypress_timeout  15  
#define long_keypress_time    15  

enum press_type {short_keypress, long_keypress, running};

int main(void)
{
  WDTCSR = (1<<WDIE) | (1<<WDP0);    // watchdog @32ms (4096 cycles) + interrupt enabled
  DDRB |= (1 << PB3)+(1 << PB4);     // PB3 und PB4 = Ausgang
  DDRB &= ~(1<<PB0);          // PB0 = Eingang
  sei();
  while(1)
  {
    asm ("NOP");
  }
}

ISR (WDT_OVERFLOW_vect)
{
  static unsigned int button_timer;  // timer to determine button state
  static enum press_type typ;
  static enum state led;
  
  typ = running;            // vorherigen Tasterzustand verlassen

  if (button_down)          // button wurde gedrückt (LOW)
  {
    if (button_timer < long_keypress_time)
      button_timer ++;      
  }
  
  else                // button wurde (ist) losgelassen (HIGH)
  {
    if (button_timer == long_keypress_time)    // Tastendruck = langer Tastendruck
    {
      typ = long_keypress;
    }      
      
    else if ((button_timer > debounce_time) && (button_timer < short_keypress_timeout))    // Tastendruck = kurzer Tastendruck
    {                
      typ = short_keypress;
    }
    
    if (button_timer)        // Button Timer nach Tastendruck zurücksetzen
    {
      button_timer = 0;
    }  
  }
  
  if (typ == long_keypress)      // Langer Tastendruck
  {
    PORTB ^= (1<<PB3);        // Portpin PB3 toggeln
  }    
  
  else if (typ == short_keypress)    // Kurzer Tastendruck
  {
    PORTB ^= (1<<PB4);        // Portpin PB4 toggeln
  }    
  
  WDTCSR = (1<<WDIE);          // Watchdog Interrupt aktivieren
}