Programm reagiert nicht auf return 1 oder return 0

[student (Gast)]

Hey ich habe folgendes Probelm,

ich habe folgenden code:

//main.c
void main(void)
{
  bool vtest = false;
  int ret = 0;

  init();   

  if (!vtest){
    ret = test();
  }

  while (ret){

    switch_output_K3_ON;
    _delay_ms(1000);
    switch_output_K3_OFF;  
    _delay_ms(1000);  
  }    
}


//switch.c
int test(void)
{

//.... CAN definitionen
  
bool test_ON = true;

while (test_ON){
  if(can_rcv && can_rcv_message.id == 0x255){
     switch (can_rcv_message.data[0])
     {        
     case 0xFF:
           switch_output_K1_ON;
           _delay_ms(1000);
           switch_output_K1_OFF;  
           _delay_ms(1000);
           test_ON = false;
           return 1;
           break;

     case 0x00:
           switch_output_K2_ON;
           _delay_ms(1000);
           switch_output_K2_OFF;  
           _delay_ms(1000);
           test_ON = false;
           return 0;
           break;
     default:
           test_ON = false;
           break;  
     }
   }
}      
}

Der Code funktioniert eigentlich super. Ich empfange CAN Nachrichten und 
lasse die Ausgaenge K1 und K2 blinken, jedoch reagiert mein Programm 
nicht auf return Befehl und somit gehe ich nicht in die while Schleife 
in main.c um den Ausgang K3 zu schaltem. Kann mir einer sagen was ich 
falsch mache? Auch wenn die Variable test_ON auf falsch setze, dann gehe 
ich auch nicht aus der while Schleife von switch.c. Was ich auch 
irgendwie nicht ganz verstehe.

Help & Danke

[Marcus O. (marcus6100)]

innerhalb int test(void) fehlt am ende der Funktion eine return 
Anweisung (die dann im default Fall ausgeführt wird, wenn die Schleife 
verlassen wird). Du solltest auf die Compilerwarnung hören. Und 
irgendwie ist das sehr hässlich programmiert...

[Helge O. (_elko_)]

Mein Prof hat uns immer die Ohren lang gezogen, wenn wir mehr als ein 
einziges RETURN hatten.

was soll das programm zurückgeben, falls es in den default Zweig geht? 
Da gibts keine Rückgabe....
Glaube, dass man das nicht darf, und der Compiler die beiden anderen 
Returns deswegen wegcompiliert? Ist gerade nur so geraten, wäre ggf. nen 
Ansatzpunkt.

[student (Gast)]

Hey,

danke fuer den Tipp. Habe gerade ausprobiert (siehe unten), leider 
erfolglos, vielleicht habe ich was falsch verstanden?

@Marcus
Frage was meinst du hässlich programmiert? Meinst du die delays? Wenn ja 
das ist nur test.

//main.c
void main(void)
{
  bool vtest = false;
  int ret = 0;

  init();   

  if (!vtest){
    ret = test();
  }

  while (ret){
    switch_output_K3_ON;
    _delay_ms(1000);
    switch_output_K3_OFF;  
    _delay_ms(1000);  
  }    
}


//switch.c
int test(void)
{

//.... CAN definitionen
  
bool test_ON = true;

while (test_ON){
  if(can_rcv && can_rcv_message.id == 0x255){
     switch (can_rcv_message.data[0])
     {        
     case 0xFF:
           switch_output_K1_ON;
           _delay_ms(1000);
           switch_output_K1_OFF;  
           _delay_ms(1000);
           test_ON = false;
           return 1;
           break;

     case 0x00:
           switch_output_K2_ON;
           _delay_ms(1000);
           switch_output_K2_OFF;  
           _delay_ms(1000);
           test_ON = false;
           return 0;
           break; 
     }
   }
} 
return 0;     
}

[student (Gast)]

Interessant ist selbst wenn ich return nur am ende als default benutze 
und test_ON auf false gesetzt habe dann geht die schleife weiter.

//main.c
void main(void)
{
  bool vtest = false;
  int ret = 0;

  init();   

  if (!vtest){
    ret = test();
  }

  while (ret){
    switch_output_K3_ON;
    _delay_ms(1000);
    switch_output_K3_OFF;  
    _delay_ms(1000);  
  }    
}


//switch.c
int test(void)
{

//.... CAN definitionen
  
bool test_ON = true;

while (test_ON){
  if(can_rcv && can_rcv_message.id == 0x255){
     switch (can_rcv_message.data[0])
     {        
     case 0xFF:
           switch_output_K1_ON;
           _delay_ms(1000);
           switch_output_K1_OFF;  
           _delay_ms(1000);
           test_ON = false;
           break;

     case 0x00:
           switch_output_K2_ON;
           _delay_ms(1000);
           switch_output_K2_OFF;  
           _delay_ms(1000);
           test_ON = false;
           break; 
     }
   }
} 
return 1;     
}

[student (Gast)]

Also es funktioniert. Es lag total nicht an dem Code, sondern an einer 
anderen Variable, die nichts damit zu tun hat. ??????????? Verstehe 
nicht, aber es funktioniert. ??????
Was ich gemacht haben ist, ich habe alle Warungen im Compiler elimiert 
habe und es ging auf einmal.

DANKE JUNGS.

[Haudrauf ! (Gast)]

Sonst gibt es auch noch einen Simulator...