Discussion :

[cherche_java]

Bonjour,

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

missing return statement

Code :

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

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 :

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
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.

[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.

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

Code pseudocode :

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

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 pseudocode :

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

1
2
3
4
5
si A et B 
    faire x
sinon si A et pas B
    faire x
fin si

soit en

Code pseudocode :

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

1
2
3
si A  
    faire x
fin si

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
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 :

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
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

[cherche_java]

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.

[cherche_java]

Salut,
Accessoirement, voir les conventions de nommage Java.

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

Code pseudocode :

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

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 pseudocode :

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

1
2
3
4
5
si A et B 
    faire x
sinon si A et pas B
    faire x
fin si

soit en

Code pseudocode :

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

1
2
3
si A  
    faire x
fin si

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
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 :

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
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.