Quelltext bleibt immer stehen und geht nicht weiter

[Frank (Gast)]

Ich habe ein Problem der Quelltext bleibt immer an der stelle "1"

stehen und geht nicht weiter wenn ich das Programm in AVR studio 
laufenlassen möchte als Mikro. verwende ich einen atmega8.
brauch dringend HILFE

int main(void)
{
//lcd_init();            
//lcd_home();
DDRC  = 0x00;
PORTC = 0xff;
  
while(1)
{

  qwe = PINC;   // 1
if (qwe == 0b00000001);
{
//  lcd_string("1");  
}
....
}

[taucher (Gast)]

und wie ist PINC definiert?

[ozo (Gast)]

wo hast du denn die variable "qwe" deklariert?

[Frank (Gast)]

Die variable ist über int main und PINC ist 0x00.

[Roland P. (pram)]

ich tipp mal ins Blaue:
Compile und Zielplatform stimmen nicht überein.
Du musst im AVR-Studio unter Project-Options ATMega8 einstellen.

Gruß
Roland

[Frank (Gast)]

ist eingestellt und sorry ich habe ein atmega 16

[Kachel-Heinz (Gast)]

Mega16 und PortC???

Da klingeln sofort alle JTAG-Glocken...

KH

[Frank (Gast)]

Bitte was klingelt????

[Johannes M. (johnny-m)]

Kachel-Heinz wrote:
> Mega16 und PortC???
>
> Da klingeln sofort alle JTAG-Glocken...
Ist im Simulator aber egal, ob JTAG aktiv ist oder nicht. Und aufhängen 
darf sich das Programm deswegen überhaupt nicht.

[Stefan E. (sternst)]

> if (qwe == 0b00000001);

Entferne das Semikolon.

[Johannes M. (johnny-m)]

Stefan Ernst wrote:
>> if (qwe == 0b00000001);
>
> Entferne das Semikolon.
Richtig, das ist ratsam, wenn das Programm etwas einigermaßen Sinnvolles 
tun soll...

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Stefan Ernst wrote:
>> if (qwe == 0b00000001);
> Entferne das Semikolon.

Ja, aber mit

if (qwe == 0b00000001) /*nichts*/;
{
   lcd_string("1");  
}

dürfte er nicht stehenbleiben, sondern immer in die Klammer springen,
auch wenn z.B. qwe = 0x45 wäre. Dann führt er /nichts*/ eben nicht aus. 
Oder was übersehe ich?

[Stefan E. (sternst)]

Sorry, das war nicht als Antwort auf die eigentliche Frage gedacht, 
sondern als Zusatzhinweis. ;-)

Zur eigentlich Frage denke ich, dass die Zeile "qwe = PINC;" der einzige 
Inhalt der while-Schleife ist, weil der Rest wegoptimiert wurde. Der 
if-Block ist ja schon mal leer und wer weiß schon, was "...." ist. Dann 
scheint im Simulator die Ausführung an dieser Zeile hängenzubleiben, 
weil einfach nur immer wieder diese Zeile ausgeführt wird.

[Frank (Gast)]

Ich habe den Fehler gefunden es war das == zwischen qwe und 0b.. jetzt 
habe ich nur noch ein gleich und es geht.

[Stefan E. (sternst)]

Frank wrote:
> ... und es geht.

Vielleicht scheint die Ausführung jetzt nicht mehr hängen zu bleiben, 
aber der Code macht sicher nicht das, was du möchtest, denn ein "=" ist 
an dieser Stelle definitiv falsch.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

>ein "=" ist an dieser Stelle definitiv falsch
Aber das ist doch egal, denn
>>es geht
und was will man mehr?   ;-)

@ Frank
Ich habe ja auch schon mal aus Versehen eine Zuweisung (=) statt eines 
Vergleichs (==) in eine Abfrage gemacht, aber ich war hinterher nicht 
stolz drauf...

Mit einer Zuweisung (=) hier
:
if (qwe = 0b00000001)
{
   irgendwas();
:
wird "irgendwas" immer ausgeführt. Denn da wird qwe zuerst der Wert 1 
zugewiesen, und dann geprüft ob qwe Null ist. Ist es natürlich nicht, 
und damit ist die Bedingung erfüllt.

[P. S. (Gast)]

Ich wuerde mir mal ein C-Buch besorgen, das lesen und dann erst mit dem 
Programmieren anfangen. Wenn eine Posix-Umgebung vorhanden ist, finde 
ich immer noch das alte "Kernighan & Ritchie, Programmieren in ANSI-C" 
empfehlenswert.