Entprellung die 1000ste

[prellung (Gast)]

vorweg: BITTE NICHT ERSCHIESSEN!

Ich weiß es gibt etliche threads dazu. Also was macht einer mehr :O)

AAlso ich habe da mal was vorbereitet:
---

void CheckPinState(void)
{
  if ((!(PINB & (1<<PB5))) && (PinState == 0)) //wenn Taster gedrückt
  {
    TCNT1 = 0; // Timer starten      
    PinState = 1;
  }
  else if (PINB & (1<<PB5)) // wenn Taste losgelassen
  {    
    PinState = 0;
  }
  else if ((PinState == 1) && (TCNT1 >= 97)) //Wenn Taste nach 5ms noch immer gedrückt
  {
    counter++;
    PinState = 2;
  }
}

---

Worum gehts: Eine einfache (SEHR einfache) Prellroutine die ich in 
meiner Main starte um meine Tasten zu entprellen. Der Unterschied zu 
anderen Routinen ist das ich eben kein Delay sondern einen Timer 
benutze. Man könnte sicher hingehen und das auch noch über Interrupts 
machen aber so funktioniert es schon einmal. Nun zu der Frage aller 
Fragen: Funktioniert meine Routine grade zufällig oder ist irgendwo ein 
Fehler versteckt den ich garnicht durchschaue? Letztendlich wird für 
eine variable "Entprellzeit" über Timer geprüft ob ein Signal anliegt.

Ich frage deswegen weil ich in verschiedenen Threads von Herrn 
Danneger-EntprellGURU gelesen habe das viele Routinen eben zufällig oder 
nur projektbasiert funktionieren. Ist dies hier ebenfalls der Fall?



Danke und Gruß
prellung

[Karl H. (kbuchegg)]

prellung schrieb:

> Ich frage deswegen weil ich in verschiedenen Threads von Herrn
> Danneger-EntprellGURU gelesen habe das viele Routinen eben zufällig oder
> nur projektbasiert funktionieren. Ist dies hier ebenfalls der Fall?

projektbasiert würde ich unterschreiben.

Deine Entprellung benutzt den Timer1 und sie benutzt ihn so, dass er zu 
sonst fast nichts mehr benutzbar ist.
PeDas Entprellung ist von der Sorte: wenn du sowieso schon einen Timer 
im System hast, der regelmässig ISR Aufrufe auslöst, dann kann der die 
Enptrellung auch noch gleich mitmachen. Die rutscht praktisch überall 
noch mit rein.

Zum anderen entprellt PeDas Entprellung nicht nur das Niederdrücken 
sondern auch das Loslassen eines Tasters. Ein Punkt der hier überhaupt 
nicht vor kommt.

Ich wüsste keinen Grund, wenn ich sowieso schon einen Timer benutze, 
warum ich die PeDa Entprellung nicht nehmen sollte. Und umgekehrt: wenn 
ich noch keinen Timer benutze, dann wäre die PeDa Entprellung ein Grund 
das zu tun.

[Peter D. (peda)]

prellung schrieb:
> Funktioniert meine Routine grade zufällig oder ist irgendwo ein
> Fehler versteckt den ich garnicht durchschaue?

Wenn Du sie selber schon nicht durchschaust, wie soll das dann ein 
anderer?
Es stimmt, if-else Monster sind wirklich schlecht zu durchschauen.

Du versucht ja offensichtlich eine Statemaschine zu erstellen, dann 
schreibe es auch als eine:

enum{
      IRGENDWAS,
      IRGEDWIE,
      NOCHNSTATE,
};

  switch( PinState ){
    case IRGENDWAS:
// usw.

Eine Funktion funktioniert auch nicht im luftleeren Raum.
Wenn sie jemand verstehen soll, mußt Du auch den Kontext hinschreiben, 
in dem sie aufgerufen wird (Main-Loop).

Und wenn eine Funktion etwas an den Aufrufer zurückliefern soll, ist ein 
void denkbar ungeeignet.
Globale Variablen nimmt man nur, wenn es nicht anders geht.