[Débutant] Erreur "missing return statement"

Bonjour,

Je ne comprends pas pourquoi la fonction suivante case_vide() me donne l'erreur

Citation:

---

missing return statement

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

Bille case_vide(){ if((compte_case_vide_inferieur()==true)&&(compte_case_vide_superieur()==true)){ case_vide_inferieur(); } if(compte_case_vide_inferieur()==true){ case_vide_inferieur(); } else{ if(compte_case_vide_superieur()==true){   case_vide_superieur(); } } }

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27

Bille case_vide_inferieur(){ //compteur=0; Bille [] b=new Bille[1];   for(int i=0; i<numero_case; i++){ if(cas[i].nombre_bille==0){ compteur=compteur+1; b[0]=cas[i]; } }   // b=new Bille[compteur]; return b[0]; }   Bille case_vide_superieur(){   Bille [] b=new Bille[1]; for(int i=15; i<numero_case; i++){ if(cas[i].nombre_bille==0){ compteur=compteur+1; b[0]=cas[i]; } }   return b[0]; }

---

Quelqu'un saurait-il me l'expliquer ?

Merci d'avance pour votre aide.

Salut,

La signature de ta méthode est Bille case_vide(), ce qui signifie qu'elle doit retourner une instance de classe Bille (ou null, éventuellement), ce qui signifie que tu dois avoir un return quelquechose (avec quelquechose de classe Bille) au moins à la fin de la méthode... comme tu as fait dans la seconde méthode (à priori il suffit de méttre return devant les 2 appels à la méthode case_vide_inferieur() et case_vide_superieur(), mais également un return final pour le cas où aucune condition testée n'est vraie.

Pourquoi dans la méthode case_vide_inferieur, crées-tu un tableau, pour ne stocker qu'une instance dedans, alors qu'une simple variable de type Bille suffit ?

A noter que quand on écrit if ( machin==true ), on peut écrire if ( machin ), qui fait la même chose, mais est un peu plus lisible.

Accessoirement, voir les conventions de nommage Java.

Les 2 premiers tests peuvent être simplifiés également, car :

Code:

---

1 2 3 4 5

si A et B faire x sinon (donc si pas A ou pas B) si A (donc toujours pas B) faire x fin si

---

peut se simplifier en

Code:

---

1 2 3 4 5

si A et B faire x sinon si A et pas B faire x fin si

---

soit en

Code:

---

1 2 3

si A faire x fin si

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

public Bille getCaseVide() {   Bille caseVide;   if ( compteCaseVideInferieur() ) {   caseVide = getCaseVideInferieur();   } else if( compteCaseVideSuperieur() ) {   caseVide = getCaseVideSuperieur();   } else {   caseVide = null; // ou je ne sais quelle valeur tu voudrais pour faire ton programme...mais une valeur, obligatoirement   }   return caseVide; // on retourne la valeur de caseVide évaluée

---

C'est plus clair, non ?

Ou, si tu préfères :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

public Bille getCaseVide() {   Bille caseVide;   if ( compteCaseVideInferieur() ) {   if( compteCaseVideSuperieur() ) {   caseVide = getCaseVideSuperieur();   } else {   caseVide = getCaseVideInferieur();   }   } else {   caseVide = null; // ou je ne sais quelle valeur tu voudrais pour faire ton programme...mais une valeur, obligatoirement   }   return caseVide; // on retourne la valeur de caseVide évaluée

---

Citation:

---

Envoyé par joel.drigo

Salut,

La signature de ta méthode est Bille case_vide(), ce qui signifie qu'elle doit retourner une instance de classe Bille (ou null, éventuellement), ce qui signifie que tu dois avoir un return quelquechose (avec quelquechose de classe Bille) au moins à la fin de la méthode... comme tu as fait dans la seconde méthode (à priori il suffit de méttre return devant les 2 appels à la méthode case_vide_inferieur() et case_vide_superieur(), mais également un return final pour le cas où aucune condition testée n'est vraie.

Pourquoi dans la méthode case_vide_inferieur, crées-tu un tableau, pour ne stocker qu'une instance dedans, alors qu'une simple variable de type Bille suffit ?

A noter que quand on écrit if ( machin==true ), on peut écrire if ( machin ), qui fait la même chose, mais est un peu plus lisible.

Accessoirement, voir les conventions de nommage Java.

---

Merci beaucoup Joel.drigo pour ton aide combien précieuse sur le déblocage de ce problème et sur la lisibilité des méthodes. J'ai appris beaucoup de choses.

Comme tu as su le voir c'est le return final qui posait problème. Quand je mets return final pour le cas où aucune condition testée n'est vraie, ça marche. Par contre, je n'ai pas mis return devant les methodes case_vide_inferieur() et case_vide_superieur() car celles-ci contiennent déjà des return qui renvoient des objets Bille.

C'est vrai pas besoin d'un tableau. Une simple variable b de type Bille suffit.

Citation:

---

Envoyé par joel.drigo

Salut,
Accessoirement, voir les conventions de nommage Java.

Les 2 premiers tests peuvent être simplifiés également, car :

Code:

---

1 2 3 4 5

si A et B faire x sinon (donc si pas A ou pas B) si A (donc toujours pas B) faire x fin si

---

peut se simplifier en

Code:

---

1 2 3 4 5

si A et B faire x sinon si A et pas B faire x fin si

---

soit en

Code:

---

1 2 3

si A faire x fin si

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

public Bille getCaseVide() {   Bille caseVide;   if ( compteCaseVideInferieur() ) {   caseVide = getCaseVideInferieur();   } else if( compteCaseVideSuperieur() ) {   caseVide = getCaseVideSuperieur();   } else {   caseVide = null; // ou je ne sais quelle valeur tu voudrais pour faire ton programme...mais une valeur, obligatoirement   }   return caseVide; // on retourne la valeur de caseVide évaluée

---

C'est plus clair, non ?

Ou, si tu préfères :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

public Bille getCaseVide() {   Bille caseVide;   if ( compteCaseVideInferieur() ) {   if( compteCaseVideSuperieur() ) {   caseVide = getCaseVideSuperieur();   } else {   caseVide = getCaseVideInferieur();   }   } else {   caseVide = null; // ou je ne sais quelle valeur tu voudrais pour faire ton programme...mais une valeur, obligatoirement   }   return caseVide; // on retourne la valeur de caseVide évaluée

---

---

le premier code me paraît beaucoup plus lisible que ce que j'ai fait. par contre pour le deuxième j'ai un petit doute. Je suis en train de regarder ça.