warum crasht das nur?

[jupp (Gast)]

ich habe hier einen sehr seltsamen crash den ich mir nicht erklären 
kann...


jemand ne idee?

//dieser code funzt:
#define LED_ON(num_)       (PORTB &= ~(1 << (num_)))
volatile uint8_t key_state;
ISR(TIMER2_OVF_vect)                            
{
  uint8_t diff, b; b = 0;

  if(key_state != KEY_PIN){
    diff = key_state ^ KEY_PIN;
    while(b < 8){
      if(diff & (1 << b)){
        LED_ON(b);  //HIER!
      }
    b++;
    }    
    key_state = KEY_PIN;
  }
}

//dieser nicht:
#define LED_ON(num_)       (PORTB &= ~(1 << (num_)))
volatile uint8_t key_state;
ISR(TIMER2_OVF_vect)                            
{
  uint8_t diff, b; b = 0;

  if(key_state != KEY_PIN){
    diff = key_state ^ KEY_PIN;
    while(b < 8){
      if(diff & (1 << b++)){
        b = b - 1; //HIER!
        LED_ON(b); //HIER!
      }
    }    
    key_state = KEY_PIN;
  }
}

[Michael 9. (michael93)]

Hallo,

>       if(diff & (1 << b++)){

was soll denn das b++ in einer if Bedingung?

Michael

[Peter D. (peda)]

jupp wrote:
> ich habe hier einen sehr seltsamen crash den ich mir nicht erklären
> kann...

Du meinst, warum Deine Endlosschleife endlos läuft?

Weil sich b nicht ändert, sobald die Bedingung wahr ist:

b + 1 - 1 bleibt immer b


Peter

[jupp (Gast)]

die schleife sieht eigentlich so aus und das funktioniert auch.

while(b < 8)
  if(diff & (1 << b++))
    was_anderes(b);

das b++ steht im if weil ich es irgendwo hochzählen muss und so is am 
einfachsten.

ist doch das gleiche wie:

while(b < 8)
  if(diff & (1 << b)){
    b++;
    was_anderes(b); 
  }

oder nicht?
ich bin da nur zufällig draufgestossen und wundere mich.

[jupp (Gast)]

>Du meinst, warum Deine Endlosschleife endlos läuft?

böse falle =)

danke

[Michael 9. (michael93)]

hab mich nur gewundert, warum mann b++ in einem if schreibt. Ist ja wie 
b = b + 1. Ich würde sowas davrsetzen oder so. Weiß nicht ob das den 
Fehler verursachen kann ???

[gast (Gast)]

> if(diff & (1 << b++))
> [...]
> ist doch das gleiche wie:
> [...]
> if(diff & (1 << b)) {
>    b++;
> [...]


NEIN, ist es nicht. Im letzteren Fall wird b nur erhöht, wenn die 
Bedingung wahr ist!

[Simon K. (simon)]

Und deswegen sollte man es auch vermeiden diese unären Operatoren so 
sehr verschachtelt zu verwenden. Das gibt nur Ärger.

[Gast (Gast)]

Debugger starten und schauen was passiert.

[Karl H. (kbuchegg)]

jupp wrote:

> das b++ steht im if weil ich es irgendwo hochzählen muss und so is am
> einfachsten.

Am einfachsten ist es immer, wenn Schleifenabbruchbedingung und 
Verändern der Schleifenvariable nicht allzuweit voneinander entfernt 
sind.

  for( b = 0; b < 8; ++b )
  {
    if( diff & ( 1 << b ) )
      LED_ON( b );
  }

Moral von der Story: Wenn du eine Zählschleife willst, dann schreib auch 
eine Zählschleife. Dafür ist sie da