While Schleife klappt nicht

[Peter (Gast)]

Hallo ich hab eine einfache Funktion für ein Selektion geschrieben, 
jedoch funktioniert diese nur wenn
while(1)... verwendet wird, der nachteil ist bei diesem, dass ich nie 
mehr rauskomme deshalb hab ich eine variable und die while schleife soll 
auf das ändern dieser reagieren

void select(void)
{
unsigned int ready;
ready = 0;
out("Waehle Animation\n");
lcd_puts(">Animation ");
index = 0;
char text[15];


  while((ready = 0))
  {  
    if ( !(PIND & (1<<PIND2)) ) 
    {
      out("Waehle Animation\n");
      lcd_puts(">Animation ");
      utoa(index, text, 10);
      lcd_puts(text);
      index++;
      delay_ms(1500);
    }
    if ( !(PIND & (1<<PIND3)) ) 
    {
      ready = 1;
      delay_ms(1500);
    }
  }
}

das problem ist aber sie wird einfach übersprungen ohne jegliche 
reaktion
lg

[Falk S. (falkschilling)]

Peter schrieb:

while((ready = 0))

Gewöhn dir Meister-Yoda-Style an:

while( NULL != ready )

[(prx) A. K. (prx)]

Kennst du den Unterschied zwischen = und == ?

[Falk S. (falkschilling)]

Sorry, meinte 0, nicht NULL.

[Peter (Gast)]

Der unterschied zwischen = und == ist der schlüssel :@ immer diese 
syntax!
ja mit = weißt man einen wert zu und mit == vergleicht man.
ich schwenke gerade von basic auf c um und das ist ziemlich verwirrend, 
aber vielen dank für euere hilfe!!
klappt jetzt auch
--closed--

[und gut isses (Gast)]

Peter schrieb:
> while((ready = 0))

while (ready == 0)

[Tom K. (ez81)]

Falk Schilling schrieb:
> Peter schrieb:while((ready = 0))
>
> Gewöhn dir Meister-Yoda-Style an:
> while( NULL != ready )

Nein, lies stattdessen die Compilerwarnungen (wer ohne -Wall compiliert, 
ist selbst schuld ;) ):

'warning: suggest parentheses around assignment used as truth value'

[Yalu X. (yalu) (Moderator)]

Warum hast du

  while((ready = 0))

mit doppelten Klammern geschrieben?

Aber nicht etwa, weil dir der Compiler das empfohlen hat:

warning: suggest parentheses around assignment used as truth value

Wenn doch: Schau dir die Meldung noch einmal genau an: Da ist die Rede
von "assignment", was ein wichtiger Hinweise auf einen mögliche Fehler
ist.

Tipp: Schreibe in Zukunft die If-Bedingungen grundsätzlich in einfache
Klammern (wobei natürlich innerhalb des Ausdrucks weitere Klammern
vorkommen dürfen) und aktiviere die Warnungen des GCC mit -Wall. Lässt
du dann versehentlich das zweite Gleichheitszeichen weg, liefert der
GCC die obige Warnung.

In den ganz seltenen Fällen, wo du das Ergebnis einer Zuweisung auf null
abprüfen möchtest, schreibst du das explizit hin wie in diesem Beispiel:

int countchar(char *str, char ch) {
  int n = 0;

  while((str = strchr(str, ch)) != NULL) {
    str++;
    n++;
  }
  return n;
}

Damit teilst du deine Absicht nicht nur dem Compiler, sondern auch einem
späteren Leser des Quellcodes mit.

Du kannst aber auch Zuweisungen innerhalb von Bedingungen auch gleich
vermeiden, auch wenn dadurch der Code etwas länger wird.

[Johann L. (gjlayde)]

Tom K. schrieb:
> Falk Schilling schrieb:
>> Peter schrieb:while((ready = 0))
>>
>> Gewöhn dir Meister-Yoda-Style an:
>> while( NULL != ready )
>
> Nein, lies stattdessen die Compilerwarnungen (wer ohne -Wall compiliert,
> ist selbst schuld ;) ):
>

'warning: suggest parentheses around assignment used as truth value'

Und genau das hat er auch gemacht: Doppelte Klammern gesetzt

--> Warnung erfolgreich beseitigt :-)