if-else + while

[upaucc]

Hi,
habe mal was ganz einfaches geschrieben, aber es funktioniert nicht wie es sollte. Asuro soll zuerst checken ob er irgendwo dagegen steht, ist dies der Fall soll er stehen bleiben und eben eine Nachricht ausspucken, das macht er auch wunderbar.
Sollte keiner der Taster gedrückt sein, soll er solange geradeausfgahren bis er irgendwo dagegen stößt, dann soll er ca. 1 s still stehen und dann rückwärts fahren.
In wirklichkeit macht er jedoch das: Wird er eingeschaltet steht er ca 1s und fährt dann rückwärts.

#include "asuro.h"

int main(void)
{
   Init();
   if(PollSwitch()==0){
		while(PollSwitch()==0){
		MotorDir(FWD,FWD);
		MotorSpeed(150,150);
		StatusLED(GREEN);
		}
	MotorDir(BREAK,BREAK);
	MotorSpeed(0,0);
	Msleep(666);
	MotorDir(RWD,RWD);
	MotorSpeed(100,100);
    }
   else{
	MotorDir(BREAK,BREAK);
	MotorSpeed(0,0);
	StatusLED(RED);
	SerWrite("Hinderniss!\n",11);
   }
   
   
   
}

[EDH]

also die schleife wird einmal ausgeführt, un ddann eben nicht mehr.

möglicherweise hilft es , wenn du poillswitch öfters aufrufst

[EDH]

kleiner nachtrag:
schrieb vieleicht ans ende while(1)
und der form halber auch noch return 0

[damaltor]

soo wolln wir uns das mal ansehen.

#include "asuro.h"

int main(void)
{
Init();

if(PollSwitch()==0){ Warum die if schleife? die while schleife prüft doch schon ob PollSwitch gleich null ist...

while(PollSwitch()==0){ soweit in ordnung... solange kein taster gedrückt, fahre gerade aus
MotorDir(FWD,FWD);
MotorSpeed(150,150);
StatusLED(GREEN);
}

MotorDir(BREAK,BREAK); sooo... wenn eine taste gedrückt wurde (und die while schleife von oben abgebrochen wurde) dann halte an, warte 666millisekunden, und fahre rückwärts.
MotorSpeed(0,0);
Msleep(666);
MotorDir(RWD,RWD);
MotorSpeed(100,100);

ab hier...

}
else{
MotorDir(BREAK,BREAK);
MotorSpeed(0,0);
StatusLED(RED);
SerWrite("Hinderniss!\n",11);
}

bis hier, dieser code wird nicht gebraucht. schliesslich sind ja schon beide möglichkeiten geschreiben worden da oben.

} diese klammer markiert das ende der main funktion


also lässt sich der code auf folgendes reduzieren:

#include "asuro.h"

int main(void){

Init();
while(PollSwitch()==0){
 MotorDir(FWD,FWD);
 MotorSpeed(150,150);
 StatusLED(GREEN);
}

MotorDir(BREAK,BREAK);
MotorSpeed(0,0);
Msleep(666);
MotorDir(RWD,RWD);
MotorSpeed(100,100);

}

ausserdem gibt es noch ein problem:

die Funktion PollSwitch() gibt immer mal wieder falsche werte zurück, deshlab sollte sie immer mehrfach abgefragt werden, bevor du damit arbeitest. du könntest z.B. am beginn deines programms (hinter init() folgendes einbauen:

for(int k=0;k<10;k++){
PollSwitch();
}

einfügen.

gerade am beginn eines programms sind die werte von pollswitch meistens falsch.

[EDH]

dr hacken ist nur, das es einen fallunterscheidung zwischen "ist hinderniss am anfang?" und "ist hinder niss während dem fahren?" gibt
dafür braucht man dann die if abfrage schon...

[damaltor]

hmm.. also ich würd sagen das ist egal... er fährt gerade aus, bis er irgendwo gegenfährt. und wenn ein hindernis am start ist, fährt er gar nicht erst los...

[EDH]

wenn ein hinderniss am anfang ist, fährt er gleuih rückwärts.
auserdem gibt er nicht "hinderniss" aus.

[damaltor]

achso.. hmm...

öhm...

dann muss um den ganzen code (ab der while schleife bis zum ende von meinem code) eine if schleife, so wei du es auch gemacht hast... probier mal folgendes:



#include "asuro.h"

int main(void){

Init();
for(int k=0;k<10;k++){
PollSwitch();
}
if(PollSwitch==0){
while(PollSwitch()==0){
MotorDir(FWD,FWD);
MotorSpeed(150,150);
StatusLED(GREEN);
}

MotorDir(BREAK,BREAK);
MotorSpeed(0,0);
Msleep(666);
MotorDir(RWD,RWD);
MotorSpeed(100,100);


}else{
SerWrite("Hindernis",10);
}

}
[/code]

das sieht deinem code schon wieder verdächtig ähnlich... mmhh.. probiers mal reinzukopieren.

[EDH]

bei GCC 4.1.2 friist er den code nicht.
du darfs nicht for(int k=0; ... schrieben
es muss heise
int k;
for (k=0; ......

#include "asuro.h"

int main(void){

Init();
int k;
for(k=0;k<10;k++){
PollSwitch();
}
if(PollSwitch==0){
while(PollSwitch()==0){
MotorDir(FWD,FWD);
MotorSpeed(150,150);
StatusLED(GREEN);
}

MotorDir(BREAK,BREAK);
MotorSpeed(0,0);
Msleep(666);
MotorDir(RWD,RWD);
MotorSpeed(100,100);


}else{
SerWrite("Hindernis",10);
}

}

[EDH]

aber funktionieren tuts trotzdem nicht.
jetzt macht er (bei mir zumindest) gar nix ehr