PIN-Abfrage in ISR -> Unterschied zwischen if(PIN)-else anders als if(PIN)-else if(!PIN)

[Michael (Gast)]

Hallo zusammen,

ich habe gerade mein DCF1-Modul von Pollin angeschlossen und wollte ein 
bisschen damit rumexperimentieren. Dazu habe ich einfach das Signal vom 
Modul (PA0) alle 0,01 s mit einem ATmega16 auf einen Pin (PD5) 
ausgegeben.

ISR (TIMER0_OVF_vect)
{
  // alle 0,01 Sekunden
  TCNT0 = (uint8_t)(-156);
  
  if (!(PINA & (1 << PINA0)))
  {
    PORTD |= (1 << PD5);
  }
  else if ((PINA & (1 << PINA0)))
  {
    PORTD &= ~(1 << PD5);
  }
}

Dabei blinkt die LED zwar ungefähr 1/s aber nach einigen Wiederholungen 
gab es dann eine mehrsekündige Pause. Ich dachte meine Beschaltung wäre 
nicht gut genug entstört, aber nachdem ich die else if-Bedingung durch 
ein einfaches else ersetzt habe klappt jetzt alles wunderbar.

ISR (TIMER0_OVF_vect)
{
  // alle 0,01 Sekunden
  TCNT0 = (uint8_t)(-156);
  
  if (!(PINA & (1 << PINA0)))
  {
    PORTD |= (1 << PD5);
  }
  else
  {
    PORTD &= ~(1 << PD5);
  }
}

Für mich ist es nachvollziehbar, dass zwischen den beiden if-Abfragen 
der Zustand des Pins wechselt, aber spätestens im nächsten Durchlauf 
müsste dann doch der gleiche Zustand anliegen und die LED wie gewohnt 
blinken (also ein maximaler Unterschied von 0,01 s). Natürlich kann ich 
hier die else Bedingung weglassen, aber ich verstehe nicht, wieso ich 
das muss.

Kann mir das einer erklären?

Danke und Gruß
Michael

[Nutzer (Gast)]

andere Frage: warum castet man eine vorzeichenbehaftete Zahl die über 8 
Bit lang ist nach uint8_t?

[Michael (Gast)]

Hallo,

damit man (im Kopf) einfacher rechnen kann.
Ich brauche mit meinem Quarz (16MHz) und dem Vorteiler (1024) 156 Takte 
für ~0,01s. Würde ich den Counter direkt mit positiven Integerwerten 
füttern so wäre die Rechnung/Überlegung:
Ich brauche 156 Takte, der Timer/Counter hat 8 bit -> 256 Takte, daher 
müsste ich TCNT0 auf 256 - 156 = 100 stellen.
Einfacher ist es jedoch, wenn ich mir durch den Cast einfach diesen Wert 
berechnen lasse -> (uint8_t)(-156) = 100
Ich könnte natürlich auch direkt TCNT0 = 100 setzen, aber so brauche ich 
keinen Zwischenschritt.

Gruß
Michael