Problémes de conditions imbriquées

Version imprimable

Bonjour,

j'ai du mal avec mon code, c'est un exercice qui permet de deviner les champignons, en répondant par true et false, il ne respecte pas les conditions if et les exécute quand même donc j'ai plusieurs champignon notamment le coprin qui sors à tout les coups
merci de m'aider

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
 
import java.util.Scanner;
 
public class Champi{
 
    public static void main(String[] agrs) {
 
       boolean forest=false;
       boolean lamelles=false;
       boolean convexePres=false;
       boolean convexeforest=false;
       boolean agaric=false;
       boolean coprin=false;
       boolean anneau=false;
       boolean piedBlue=false;
       boolean amanite=false;
       boolean girolle=false;
       boolean cepe=false;
       int continued=1;
       System.out.println("Pensez a un champignon : amanite tue mouches, pied bleu, girolle,");
       System.out.println("cÃ¨pe de Bordeaux, coprin chevelu ou agaric jaunissant.");
 
       Scanner clavier = new Scanner(System.in);
       System.out.print("Est-ce que votre champignon vit en forÃªt (true : oui, false : non) ? ");
       forest = clavier.nextBoolean();
           if  (forest==false){
            System.out.print("Est-ce que votre champignon a un chapeau convexe 1 (true : oui, false : non) ? ");  
            convexePres= clavier.nextBoolean();
            }
        if ((convexePres==true) && (continued==1)){
            agaric=true;
            continued=0;
            }else{
            coprin=true;
            continued=0; }
 
 
        if (forest==true) {
            System.out.print("Est-ce que votre champignon a un chapeau convexe 2 (true : oui, false : non) ? ");  
            convexeforest= clavier.nextBoolean(); 
            continued=2;
 
            }
        if ((convexeforest==true) && (continued==2)){
            System.out.print("Est-ce que votre champignon a des lamelles (true : oui, false : non) ? ");
            lamelles= clavier.nextBoolean();}
        else{
            System.out.print("Est-ce que votre champignon a un anneau (true : oui, false : non) ? ");
            anneau=clavier.nextBoolean();}
 
        if ((anneau==true) && (continued==2)){
            amanite=true;
            continued=0;
            }
        if ((anneau=false) && (continued==2)){
            piedBlue=true;
            continued=0;
            }
        if ((lamelles==true) && (continued==2)){
            girolle=true;
            continued=0;
            }
        if ((lamelles==false) && (continued==2)){
            cepe=true;
            continued=0;
            }
 
 
        System.out.print("==> Le champignon auquel vous pensez est ");
        if (agaric==true) {
            System.out.print("l'agaric jaunissant");
        }
          if (coprin==true){
              System.out.print(coprin);
            System.out.print("le coprin chevelu");
        }
          if (piedBlue==true){
            System.out.print("le pied bleu");}
 
        if (amanite==true)
            System.out.print("l'amanite tue-mouches");
 
        if (cepe==true){        
        System.out.print("le cÃ¨pe de Bordeaux");
        }
        if (girolle==true){
        System.out.print("la girolle");}
        }
    }

Déjà avec un indentation correcte on peut lire le code

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
 
import java.util.Scanner;
 
public class Champi{
 
    public static void main(String[] agrs) {
 
        boolean forest=false;
        boolean lamelles=false;
        boolean convexePres=false;
        boolean convexeforest=false;
        boolean agaric=false;
        boolean coprin=false;
        boolean anneau=false;
        boolean piedBlue=false;
        boolean amanite=false;
        boolean girolle=false;
        boolean cepe=false;
        int continued=1;
        System.out.println("Pensez a un champignon : amanite tue mouches, pied bleu, girolle,");
        System.out.println("cÃ¨pe de Bordeaux, coprin chevelu ou agaric jaunissant.");
 
        Scanner clavier = new Scanner(System.in);
        System.out.print("Est-ce que votre champignon vit en forÃªt (true : oui, false : non) ? ");
        forest = clavier.nextBoolean();
        if  (forest==false){
            System.out.print("Est-ce que votre champignon a un chapeau convexe 1 (true : oui, false : non) ? ");
            convexePres= clavier.nextBoolean();
        }
        if ((convexePres==true) && (continued==1)){
            agaric=true;
            continued=0;
        }else{
            coprin=true;
            continued=0;
        }
 
 
        if (forest==true) {
            System.out.print("Est-ce que votre champignon a un chapeau convexe 2 (true : oui, false : non) ? ");
            convexeforest= clavier.nextBoolean();
            continued=2;
 
        }
        if ((convexeforest==true) && (continued==2)){
            System.out.print("Est-ce que votre champignon a des lamelles (true : oui, false : non) ? ");
            lamelles= clavier.nextBoolean();
        }
        else{
            System.out.print("Est-ce que votre champignon a un anneau (true : oui, false : non) ? ");
            anneau=clavier.nextBoolean();
        }
 
        if ((anneau==true) && (continued==2)){
            amanite=true;
            continued=0;
        }
        if ((anneau=false) && (continued==2)){
            piedBlue=true;
            continued=0;
        }
        if ((lamelles==true) && (continued==2)){
            girolle=true;
            continued=0;
        }
        if ((lamelles==false) && (continued==2)){
            cepe=true;
            continued=0;
        }
 
 
        System.out.print("==> Le champignon auquel vous pensez est ");
        if (agaric==true) {
            System.out.print("l'agaric jaunissant");
        }
        if (coprin==true){
            System.out.print(coprin);
            System.out.print("le coprin chevelu");
        }
        if (piedBlue==true){
            System.out.print("le pied bleu");
        }
 
        if (amanite==true)
            System.out.print("l'amanite tue-mouches");
 
        if (cepe==true){
            System.out.print("le cÃ¨pe de Bordeaux");
        }
        if (girolle==true){
            System.out.print("la girolle");
        }
    }
}

Ensuite,

Code:

1
2
3
4
5
6
7
        if ((convexePres==true) && (continued==1)){
            agaric=true;
            continued=0;
        }else{
            coprin=true;
            continued=0;
        }

Le test sur continued dans ce bloc est inutile, car il vaut d'office 1 avant ce bloc et 0 après ce bloc. De plus ce code implique que dès que c'est en forêt ou que ce n'est pas convexe, c'est un coprin...

Code:

if ((convexeforest==true) && (continued==2))

test sur continued inutile encore une fois. Si convexeforest alors continued =2 puisque les deux dont définis en même temps.

Tu dois surtout commencer par prendre une feuille de papier et noter toutes les conditions à tester et dans quelles conditions c'est quel champignon. Fais toi une table de vérité, met tous tes tests verticalement, tes champignons horizontalement, et met des oui / non / on s'en fou sur chaque case pour voir tes critères.
Ce n'est pas ton code le problème, c'est ta logique pour déterminer un champignon.

le code me parait correct et ça marche pas, je pense que c'est le type de boolean qui pose probleme

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
 
import java.util.Scanner;
 
public class Champi{
 
	public static void main(String[] agrs) {
 
	   boolean forest=false;
	   boolean lamelles=false;
	   boolean convexePres=false;
	   boolean convexeforest=false;
	   boolean agaric=false;
	   boolean coprin=false;
	   boolean anneau=false;
	   boolean piedBlue=false;
	   boolean amanite=false;
	   boolean girolle=false;
	   boolean cepe=false;
 
 
	   System.out.println("Pensez a un champignon : amanite tue mouches, pied bleu, girolle,");
       System.out.println("cÃ¨pe de Bordeaux, coprin chevelu ou agaric jaunissant.");
 
       Scanner clavier = new Scanner(System.in);
       System.out.print("Est-ce que votre champignon vit en forÃªt (true : oui, false : non) ? ");
       forest = clavier.nextBoolean();
       	if (forest==true) {
        	System.out.print("Est-ce que votre champignon a un chapeau convexe 2 (true : oui, false : non) ? ");  
    	    convexeforest= clavier.nextBoolean(); 
    	    }if (convexeforest==true){
    	    	System.out.print("Est-ce que votre champignon a des lamelles (true : oui, false : non) ? ");
    	    	lamelles= clavier.nextBoolean();
    	    	}if (lamelles==true){
    	    		girolle=true;
    	    	}else{
            		cepe=true;
            }if (convexeforest==false){
            	System.out.print("Est-ce que votre champignon a un anneau (true : oui, false : non) ? ");
            	anneau=clavier.nextBoolean();
            	}if (anneau==true){
            		amanite=true;
            	}else{
            		piedBlue=true;}       
 
        if (forest==false){
        	System.out.print("Est-ce que votre champignon a un chapeau convexe 1 (true : oui, false : non) ? ");  
    	    convexePres= clavier.nextBoolean();
    	    }if (convexePres==true) {
        	agaric=true;
        	}else{
        	coprin=true;}
 
 
 
        System.out.print("==> Le champignon auquel vous pensez est ");
        if (agaric==true) {
        	System.out.print("l'agaric jaunissant");}
      	if (coprin==true){
      		System.out.print("le coprin chevelu");}
      	if (piedBlue==true){
        	System.out.print("le pied bleu");}
        if (amanite==true){
        	System.out.print("l'amanite tue-mouches");}
        if (cepe==true){        
        System.out.print("le cÃ¨pe de Bordeaux");}
        if (girolle==true){
        System.out.print("la girolle");}
        }
    	    }

Ben ton code il dit

Code:

1
2
3
4
5
 		if (lamelles == true) {
 			girolle = true;
 		} else {
 			cepe = true;
 		}

Ce qui se traduit par "un champignon est d'office une girolle ou un cepe"
Ensuite

Code:

1
2
3
4
5
 		if (anneau == true) {
 			amanite = true;
 		} else {
 			piedBlue = true;
 		}

Un campignon est d'office une amanite ou un piedBlue

Code:

1
2
3
4
5
 		if (convexePres == true) {
 			agaric = true;
 		} else {
 			coprin = true;
 		}

Et toujours dans la même veinne: un champignon est d'office un agaric ou un coprin.
Bref, non, ton code n'est pas correct. Et si tu fais un effort sur l'indentation, tu verra tout de suite le problème.

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
 import java.util.Scanner;
 
 public class Champi {
 
 	public static void main(String[] agrs) {
 
 		boolean forest = false;
 		boolean lamelles = false;
 		boolean convexePres = false;
 		boolean convexeforest = false;
 		boolean agaric = false;
 		boolean coprin = false;
 		boolean anneau = false;
 		boolean piedBlue = false;
 		boolean amanite = false;
 		boolean girolle = false;
 		boolean cepe = false;
 
 
 		System.out.println("Pensez a un champignon : amanite tue mouches, pied bleu, girolle,");
 		System.out.println("cÃ¨pe de Bordeaux, coprin chevelu ou agaric jaunissant.");
 
 		Scanner clavier = new Scanner(System. in );
 		System.out.print("Est-ce que votre champignon vit en forÃªt (true : oui, false : non) ? ");
 		forest = clavier.nextBoolean();
 		if (forest == true) {
 			System.out.print("Est-ce que votre champignon a un chapeau convexe 2 (true : oui, false : non) ? ");
 			convexeforest = clavier.nextBoolean();
 		}
 		if (convexeforest == true) {
 			System.out.print("Est-ce que votre champignon a des lamelles (true : oui, false : non) ? ");
 			lamelles = clavier.nextBoolean();
 		}
 		if (lamelles == true) {
 			girolle = true;
 		} else {
 			cepe = true;
 		}
 		if (convexeforest == false) {
 			System.out.print("Est-ce que votre champignon a un anneau (true : oui, false : non) ? ");
 			anneau = clavier.nextBoolean();
 		}
 		if (anneau == true) {
 			amanite = true;
 		} else {
 			piedBlue = true;
 		}
 
 		if (forest == false) {
 			System.out.print("Est-ce que votre champignon a un chapeau convexe 1 (true : oui, false : non) ? ");
 			convexePres = clavier.nextBoolean();
 		}
 		if (convexePres == true) {
 			agaric = true;
 		} else {
 			coprin = true;
 		}
 
 
 
 		System.out.print("==> Le champignon auquel vous pensez est ");
 		if (agaric == true) {
 			System.out.print("l'agaric jaunissant");
 		}
 		if (coprin == true) {
 			System.out.print("le coprin chevelu");
 		}
 		if (piedBlue == true) {
 			System.out.print("le pied bleu");
 		}
 		if (amanite == true) {
 			System.out.print("l'amanite tue-mouches");
 		}
 		if (cepe == true) {
 			System.out.print("le cÃ¨pe de Bordeaux");
 		}
 		if (girolle == true) {
 			System.out.print("la girolle");
 		}
 	}
 }

28/09/2015, 20h06
johdev

1 pièce(s) jointe(s)

voici mon schéma
Pièce jointe 189003
pourquoi quand on réponds true oui (oui, à la question Est-ce que votre champignon vit en forÃªt (true : oui, false : non) ?)
(forest==true)
il va chercher aussi false? (forest==false)? je comprends pas. pourtant j'ai répondu true à la question (forest = clavier.nextBoolean();
28/09/2015, 20h44
tchize_

par ce que dans ton code le test sur le non de convexe ne dépend pas du test sur la foret. Donc peux importe ce que tu a répondu à foret, il rentrera dans tous les cas dans le convexe = false. Bref ton schema ne correspond pas à ton code.
28/09/2015, 22h27
johdev

comment faire alors pour ça fonctionne? pour l'inclure dans le code ?
28/09/2015, 22h46
tchize_
ben regarde ton schéma, il est à plusieurs niveaux.

Si foret, on part en haut, dedans on teste si convexe, tu dois retrouver la même chose dans ton code. Ce qui signifie en termes de code des ifs imbriqués
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 if (foret){ if (convexe){ agaric } else { coprin } } else { if (convexe){ if (anneau){ } //etc } //etc }
29/09/2015, 01h05
johdev

merci ça marche parfaitement. J'avais des difficultés à employer les imbriqués