Discussion :

[cobra38]

Bonjour à tous,

j'essaie de compiler un programme trouvé sur le net
https://www.instructables.com/MQTT-S...ature-Monitor/

j'ai une erreur à la compilation dont je ne connais pas la signification
( le terme 'return' semble pourtant bien noté )

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
control reaches end of non-void function [-Werror=return-type]

dans ce sous-programme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
byte Check_AP_Switch() // returns value depending on how long the AP button is pressed  
{
  unsigned long Switch_Pressed_Time; 
  if(digitalRead(AP_Switch) == AP_SwitchOff )   // read & test the button status
  {       // if not pressed
      Time_Start = millis();   // get time marker in readiness for switch being pressed
      return 0;
  }
  else // switch is ON
  {
    while (digitalRead(AP_Switch) == AP_SwitchOn)  // wait until button is released
    {
      delay(10);
      Switch_Pressed_Time = millis() - Time_Start;  // calculate the time since button was pressed(mS)
    }
          // button released
    //if (EnableMessages) Serial.println(Switch_Pressed_Time);     // Diagnostic - button duration time (mS) 
    if(Switch_Pressed_Time >= 1000) return 1;   // Wake up Access point
    if(Switch_Pressed_Time <  1000)  return 0;   // button not pressed long enough
  }  // end of else // switch is ON
}     // end of byte Check_AP_Switch()

Vos conseils me seraient précieux
Mille mercis

[Jay M]

Ça veut dire que le compilateur a détecté un chemin qui emmène à la fin de la fonction et qu’il n’y a pas de return d’une valeur du type attendu ou pas de return du tout

Dans votre cas c’est si vous rentrez dans le else le compilateur ne voit pas que les 2 if couvrent toutes les possibilités, le second if ne sert à rien il suffit de l’enlever

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
else // switch is ON
  {
    while (digitalRead(AP_Switch) == AP_SwitchOn)  // wait until button is released
    {
      delay(10);
      Switch_Pressed_Time = millis() - Time_Start;  // calculate the time since button was pressed(mS)
    }
          // button released
    //if (EnableMessages) Serial.println(Switch_Pressed_Time);     // Diagnostic - button duration time (mS) 
    if(Switch_Pressed_Time >= 1000) return 1;   // Wake up Access point
    return 0;   // button not pressed long enough
  }  // end of else // switch is ON
}     // end of byte Check_AP_Switch()

[licardentaistor]

tu es censé retourner un byte dans ta fonction mais si

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 if(digitalRead(AP_Switch) == AP_SwitchOff )

cette condition n'est pas satisfaite que retourne tu?

[cobra38]

Merci à tous

Effectivement , la suppression de la ligne concernée résout le problème

Par contre, il faut que je change de lunettes

Merci encore et mille pardons pour le dérangement


Bonne Journée
pascal

[edgarjacobs]

Hello,

Une autre manière de faire: return Switch_Pressed_Time >= 1000 ? 1 : 0;

[cobra38]

Merci edgarjacobs
c'est bien noté et plus simple visiblement

[Jay M]

tu es censé retourner un byte dans ta fonction mais si

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 if(digitalRead(AP_Switch) == AP_SwitchOff )

cette condition n'est pas satisfaite que retourne tu?

Si cette condition n'est pas satisfaite, le code rentrait dans le else où on trouvait

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
    if(Switch_Pressed_Time >= 1000) return 1;   // Wake up Access point
    if(Switch_Pressed_Time <  1000)  return 0;   // button not pressed long enough

donc dans tous les cas il y avait un return et le code était réellement fonctionnel

cependant le compilateur n'a pas été assez malin pour voir que >= 1000 et < 1000 étaient complémentaires et donc que dans tous les cas il y avait bien un return, d'où le message.

en mettant

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
    if(Switch_Pressed_Time >= 1000) return 1;   // Wake up Access point
    return 0;   // button not pressed long enough

ou comme proposé par @edgarjacobs

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 return Switch_Pressed_Time >= 1000 ? 1 : 0;

ou encore juste

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 return Switch_Pressed_Time >= 1000; // le bool sera promu en entier 0 ou 1, et la fonction serait mieux si elle retournait un bool

le compilateur "retrouvait ses petits" et ne met plus l'erreur