Discussion :

[mike1313]

Bonjour, désolé si mon cas vous semble trop simple.

Voici mon problème j'ai voulu mettre en pratique un petit exercice consistant à associé un tarif à un poids d'une lettre.

Au delà de 3kg il s'agit d'un colissimo nous n'avons plus à afficher le poids.

Le problème est que cela ne marche pas.
La première partie fonctionne.
A la fin du code la condition if (poidsmax=true) ne tient pas compte de la valeur de poidsmax(booléen) des conditions précédentes.
J'ai ajouté une simple condition de test pour mettre en évidence mon problème dans la seconde partie.

Exemple quand je saisie :

Quelle est le poids de votre lettre ? 22
Le poids de votre lettre est de : 22g
Pour info : poids de la lettre = 22 et poidsmax = false
Dans la condition if poidsmax=true poidsmax=true


Dans cet exemple 22g, la valeur poidsmax = false.
Seulement à la fin de la condition if (poidsdelalettre<=20), la valeur poidsmax = true ....

J'ai dû oublier quelque chose quelque part.

Quelqu'un a une idée d'une solution ?

Merci pour votre aide

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
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
 
 
 
 
    public static void main(String args[]) {
 
 
 
    	Scanner poids =new Scanner (System.in);
 
 
    	System.out.print("Quelle est le poids de votre lettre ? ");
 
    	int poidsdelalettre = poids.nextInt();
    	double tarifenvoi=0;
    	boolean poidsmax=false;
 
 
    	System.out.println("Le poids de votre lettre est de : "+poidsdelalettre+"g");
 
 
    	if (poidsdelalettre<=20)
    		{tarifenvoi=0.61;
    		poidsmax=false;}
 
    		else if (poidsdelalettre<=50)
    	    		{tarifenvoi=1.02;
    	    		 poidsmax=false;}
 
    		else if (poidsdelalettre<=100)
	    			{tarifenvoi=1.55;
   	    		 	poidsmax=false;}
 
    		else if (poidsdelalettre<=250)
	    			{tarifenvoi=2.45;
   	    		 	poidsmax=false;}
 
    		else if (poidsdelalettre<=1000)
    				{tarifenvoi=4.35;
   	    		 	poidsmax=false;}
 
    		else if (poidsdelalettre<=2000)
					{tarifenvoi=5.65;
   	    		 	poidsmax=false;}
 
    		else if (poidsdelalettre<=3000)
					{tarifenvoi=6.55;
		 			poidsmax=false;}
 
    		else if (poidsdelalettre>3000)
			{poidsmax=true;};
 
			System.out.println("Pour info : poids de la lettre = "+poidsdelalettre+" et poidsmax = "+poidsmax);
 
 
 
 
    	if(poidsmax=true){System.out.println("Dans la condition if (poidsmax=true) poidsmax="+poidsmax);}
 
    	else {System.out.println("poidsmax=false valeur = "+poidsmax);}
 
 
    }
}

[joel.drigo]

Salut,

Le signe = est un signe d'affectation (pour donner une valeur à une variable). Le signe de comparaison d'égalité est ==.

Quand tu écris if(poidsmax=true), ça met true dans poidsmax, puis ça évalue poidsmax (qui vaut true donc) : donc ce test est toujours vrai.

Il faudrait écrire :

Code :

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

if ( poidsmax==true )

Mais comme l'évaluation d'une expression dans un if est toujours une comparaison à true, et que poidsmax est un booléen, tu peux écrire directement :

Code :

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

if ( poidsmax )

ce qui t'évitera à l'avenir de taper =, au lieu de ==, par erreur.

[mike1313]

Merci Joel !

C'est tout à fait ça.

Réponse claire et constructive.

Je post le code finalisé pour les utilisateurs qui rencontreraient le même problème.

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
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
 
 
class tarif1 {
 
 
    public static void main(String args[]) {
 
 
 
    	Scanner poids =new Scanner (System.in);
 
 
    	System.out.print("Quelle est le poids de votre lettre (en grammes) ? ");
 
    	int poidsdelalettre = poids.nextInt();
    	double tarifenvoi=0;
    	boolean poidsmax=false;
 
 
    	System.out.println("Le poids de votre lettre est de "+poidsdelalettre+"g");
 
 
    	if (poidsdelalettre<=20)
    		{tarifenvoi=0.61;
    		poidsmax=false;}
 
    		else if (poidsdelalettre<=50)
    	    		{tarifenvoi=1.02;
    	    		 poidsmax=false;}
 
    		else if (poidsdelalettre<=100)
	    			{tarifenvoi=1.55;
   	    		 	poidsmax=false;}
 
    		else if (poidsdelalettre<=250)
	    			{tarifenvoi=2.45;
   	    		 	poidsmax=false;}
 
    		else if (poidsdelalettre<=1000)
    				{tarifenvoi=4.35;
   	    		 	poidsmax=false;}
 
    		else if (poidsdelalettre<=2000)
					{tarifenvoi=5.65;
   	    		 	poidsmax=false;}
 
    		else if (poidsdelalettre<=3000)
					{tarifenvoi=6.55;
		 			poidsmax=false;}
 
    		else if (poidsdelalettre>3000)
			{poidsmax=true;};
 
 
 
    	if(poidsmax){System.out.println("Votre lettre dépasse 3 kg, merci de consulter nos tarifs de colis.");}
 
    	else {System.out.println("Voici le tarif de votre envoi est de "+tarifenvoi+" €");}
 
    }
}

[andry.aime]

Bonjour,

pour éviter ce genre de bug, prend l'habitude de mettre les constants à gauches dans le test.

Code :

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

1
2
3
4
if(true=poidsmax) // t'auras directement un erreur de compilation
if (true==poidsmax)// la ça passe
if(unString.equals("a")) // tu peux avoir un NullPointerException
if("a".equals(unString)) // la ça passe

A+.

[mike1313]

Merci andry.aime !