Discussion :

[RimaG]

Bonjour,
j'ai besoin de votre aide.

Je veux faire l'arrondi avec des conditions comme suit :

Au gramme le plus proche sans décimale si ≥10 g
Au décigramme le plus proche si < 10 g et > 0,5 g
«*0 g*» ou «*< 0,5 g*» autorisé si ≤ 0,5 g

Pour cela j'ai créé une méthode

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
private Double nearByValueEur (Double value, Double minValue){
		Double res = 0.0;
		if (value >=10){
			res= (double) Math.round(value);
		}
		else if (value >minValue && value <10){
 
        	BigDecimal bd = new BigDecimal(value);
        	bd= bd.setScale(2, RoundingMode.UP);
        	value = bd.doubleValue();
		}
		else if (value == minValue){
 
			res=minValue;
		}
		else{
			String msg= "Moins de 0,5 g sont autorisés";
			return res= Double.valueOf(msg);
		}
		return res;
	}

Quand je lance, le programme ne marche pas. J'ai des doutes par rapport à la boucle.
Si quelqu'un peut me corriger ou m'expliquer la faute.
Je vous en remercie par avance.

[Drowan]

le programme ne marche pas

C'est à dire ?
Le programme plante ? Tu as quel message d'erreur ? quelle ligne plante le programme ?
Tu n'as pas le résulat attendu ? Dans quel cas ? Tu as essayé avec quelles entrées ? Tu obtients quelles sorties ? tu attendais quoi comme sortie ?

[Gaillac]

Quelle boucle ?

Le premier problème c'est

Code :

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

1
2
String msg= "Moins de 0,5 g sont autorisés";
return res= Double.valueOf(msg);

Parce que

Code :

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

Double.valueOf("Moins de 0,5 g sont autorisés");

ça va coincer.
Mon avis perso ça serait d'afficher le message et renvoyer la valeur plancher, donc

Code :

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

1
2
System.out.println("Les valeurs de moins de " +minValue+  " g ne sont pas autorisées");
res= minValue;

Deuxième problème, dans la partie moins de 10g :
Tu n'affecte jamais ta valeur de retour res, mais une variable value non utilisée.

Troisième problème, pour être au décigramme près, il faut plutôt

Code :

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

bd= bd.setScale(2, RoundingMode.UP);

ce qui donne au final :

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
private Double nearByValueEur (Double value, Double minValue){
		Double res = 0.0;
		if (value >=10){
			res= (double) Math.round(value);
		}
		else if (value >minValue && value <10){
 
        	BigDecimal bd = new BigDecimal(value);
        	bd= bd.setScale(1, RoundingMode.UP);
        	res = bd.doubleValue();
		}
		else if (value == minValue){
 
			res=minValue;
		}
		else{
			System.out.println("Les valeurs de moins de " + minValue +  " g ne sont pas autorisées");
			res= minValue;
		}
		return res;
	}

et :
new dbl 12.49 =12.0
new dbl 12.50 =13.0
new dbl 2.45 =2.5
Les valeurs de moins de 0.5 g ne sont pas autorisées
new dbl 0.25 =0.5

NB : plutôt que

Code :

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

bd= bd.setScale(1, RoundingMode.UP);

, la méthode HALF_UP donne des résultats plus exacts :

Code :

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

bd= bd.setScale(1, RoundingMode.HALF_UP);

new dbl 2.45 =2.5
new dbl 2.44 =2.4
new dbl 2.50 =2.5
et

Code :

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

1
2
3
else if (value == minValue){
	res=minValue;
}

serait avantageusement remplacé par un >= dans la condition d'avant.

Enfin dans le même ordre d'idée, l'arrondi de 12.49 ou 12.15 en 12.0 n'est pas bon non plus, merci round...

[eulbobo]

J'ajouterai que c'est typiquement le genre de cas dans lequel le TDD permet d'avancer au fur et à mesure jusqu'à arriver au bon résultat.



Crée une classe de test unitaire (ce qui va sous-entendre de rendre cette méthode publique, et donc potentiellement de la mettre dans une autre classe) et procède étape par étape

Crée le test : je dois avoir un arrondi au gramme le plus proche sans décimale si c'est ≥10 g (donc tester pour la valeur 10g qui doit renvoyer 10, 12.55 qui doit renvoyer 13, et 13.2 qui doit renvoyer 13 aussi)
Implémente jusqu'à ce que le test soit vert

Passe au contrat suivant : Au décigramme le plus proche si < 10 g et > 0,5 g

ETC...

Tu verras que tu arriveras à trouver la solution par toi même au fur et à mesure en te concentrant sur une seule problématique à la fois, et avec l'assurance que ton code fonctionne ensuite !




Sinon, comme tes types en entrée sont des "Double", donc des objets java et pas des types de base, tu ne peux pas les comparer en utilisant '==' : il faut comparer les valeurs qui sont contenues (et comme c'est du double, ça risque de renvoyer faux même si c'est vrai) : utilise donc Bigdecimal.compareTo pour ça