warum crasht das nur?

jupp Gast #1206577 
ich habe hier einen sehr seltsamen crash den ich mir nicht erklären 
kann...


jemand ne idee?

1
//dieser code funzt:

2
#define LED_ON(num_)       (PORTB &= ~(1 << (num_)))

3
volatile uint8_t key_state;

4
ISR(TIMER2_OVF_vect)                            

5
{

6
  uint8_t diff, b; b = 0;

7


8
  if(key_state != KEY_PIN){

9
    diff = key_state ^ KEY_PIN;

10
    while(b < 8){

11
      if(diff & (1 << b)){

12
        LED_ON(b);  //HIER!

13
      }

14
    b++;

15
    }    

16
    key_state = KEY_PIN;

17
  }

18
}

19


20
//dieser nicht:

21
#define LED_ON(num_)       (PORTB &= ~(1 << (num_)))

22
volatile uint8_t key_state;

23
ISR(TIMER2_OVF_vect)                            

24
{

25
  uint8_t diff, b; b = 0;

26


27
  if(key_state != KEY_PIN){

28
    diff = key_state ^ KEY_PIN;

29
    while(b < 8){

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

31
        b = b - 1; //HIER!

32
        LED_ON(b); //HIER!

33
      }

34
    }    

35
    key_state = KEY_PIN;

36
  }

37
}
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Langsamer Modus Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Melden Danke
Peter D. #1206604 
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
Antwort Zitieren Bearbeiten Löschen Melden Danke
jupp Gast #1206615 
die schleife sieht eigentlich so aus und das funktioniert auch.

1
while(b < 8)

2
  if(diff & (1 << b++))

3
    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:

1
while(b < 8)

2
  if(diff & (1 << b)){

3
    b++;

4
    was_anderes(b); 

5
  }


oder nicht?
ich bin da nur zufällig draufgestossen und wundere mich.
Antwort Zitieren Bearbeiten Löschen Melden Danke
gast Gast #1206730 
> 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!
Antwort Zitieren Bearbeiten Löschen Melden Danke
Karl H. #1207169 
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.

1
  for( b = 0; b < 8; ++b )

2
  {

3
    if( diff & ( 1 << b ) )

4
      LED_ON( b );

5
  }


Moral von der Story: Wenn du eine Zählschleife willst, dann schreib auch 
eine Zählschleife. Dafür ist sie da
Antwort Zitieren Bearbeiten Löschen Melden Danke
Thread beobachten

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 17 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Dein Name und deine E-Mail-Adresse

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.