Distributeur de Billet en JAVA

Bonjour,

Apprenti en Java je cherche à recréer un distributeur de billet automatique. Jusque là tout va bien, ma classe doit passer des test unitaires, mais il en échoue a 2 sur 16. Je ne vois pas d'où peut venir l'erreur. Quelqu'un peut-il m'aider a déceler cela et m'aider eventuellement ?

Voici le code de mon DAB :

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
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
public class DABiP {
	int nbBillet100;
	int nbBillet50;
	int nbBillet20 ;
	int nbBillet10 ; 
	int nbPiece2; 
	int nbPiece1; 
 
	//Constructeur privÃ©
	private DABiP(int b10, int b5, int b2, int b1, int p2, int p1){
		nbBillet100	= b10;
		nbBillet50 	= b5;
		nbBillet20 	= b2;
		nbBillet10 	= b1;
		nbPiece2 	= p2;
		nbPiece1 	= p1;
	}
	//DABiP initialisÃ© avec 100 unitÃ©s de chaque, soit 18300euros
	public DABiP(){
		this(100, 100, 100, 100, 100, 100);
	}
 
	/************************************************************
        * SEULES LES METHODES QUI SUIVENT DOIVENT ETRE MODIFIEES
        *************************************************************/
 
	//Calculer le montant restant dans le DABiP
	public int calculerMontantRestantDansDABiP(){
		int a;
 
		a = nbBillet100 * 100 + nbBillet50 * 50 + nbBillet20 * 20 + nbBillet10 * 10 + nbPiece2 * 2 + nbPiece1 * 1;
 
		return a;
	}
	//Retirer une somme d'argent du DABiP
	public boolean retrait(int somme){
 
		int x = 0;
 
		if(somme > calculerMontantRestantDansDABiP()) {
			return false;	
		}
		else if(somme < 0) {
			return false;
		}
 
		else{
			if(somme/100 > 1){
				x = somme/100;
				if(nbBillet100 - x >= 0){
					nbBillet100 = nbBillet100 - x;
					somme = somme - 100 * x;
				}
				else {
					x = nbBillet100;
					nbBillet100 = 0;
					somme = somme - 100 * x;
				}
			}
 
			if(somme/50 > 1){
				x = somme/50;
				if(nbBillet50 - x >= 0){
					nbBillet50 = nbBillet50 - x;
					somme = somme - 50 * x;
				}
				else{
					x = nbBillet50;
					nbBillet50 = 0;
					somme = somme - 50 * x;
				}
			}
 
			if(somme/20 > 1){
				x  = somme/20;
				if(nbBillet20 - x >= 0){
					nbBillet20 = nbBillet20 - x;
					somme = somme - 20 * x;
				}
				else{
					x = nbBillet20;
					nbBillet20 = 0;
					somme = somme - 20 * x;
				}
			}
 
			if(somme/10 > 1){
				x = somme/10;
				if(nbBillet10 - x >= 0){
					nbBillet10 = nbBillet10 - x;
					somme = somme - 10 * x;
				}
				else{
					x = nbBillet10;
					nbBillet10 = 0;
					somme = somme - 10 * x;
				}
			}
 
			if(somme/2 > 1){
				x = somme/2;
				if(nbPiece2 - x >= 0){
					nbPiece2 = nbPiece2 - x;
					somme = somme - 2 * x;
				}
				else{
					x = nbPiece2;
					nbPiece2 = 0;
					somme = somme - 2 * x;
				}
			}
 
			if(somme >= 1){
				x = somme;
				if(nbPiece1 - x >= 0){
					nbPiece1 = nbPiece1 - x;
					somme = somme - x;
				}
				else{
					x = nbPiece1;
					nbPiece1 = 0;
					somme = somme - x;
				}
				}
		return true;
		}
 
	}
}

Voici les tests qu'ils doit passer

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
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
 
public class Notation {
	public static void main(String[] args) throws InterruptedException {
		DABiP dab;
		double note		= 0;
		double score 	= 0;
 
		//Test 1
		System.out.print("Test 01: ");
		dab 		= new DABiP();
		if(dab.calculerMontantRestantDansDABiP() == 18300){
			score++;
			System.out.println("ok");
		}
		else{
			System.out.println("non");
		}
 
		//Test 2
		System.out.print("Test 02: ");
		dab.nbBillet100 = 95;
		if(dab.calculerMontantRestantDansDABiP() == 17800){
			score++;
			System.out.println("ok");
		}
		else{
			System.out.println("non");
		}
 
		//Test 3
		System.out.print("Test 03: ");
		dab.nbBillet50	= 97;
		dab.nbPiece2 	= 96;
		dab.nbPiece1 	= 97;
		if(dab.calculerMontantRestantDansDABiP() == 17639){
			score++;
			System.out.println("ok");
		}
		else{
			System.out.println("non");
		}
 
		//Test 4
		System.out.print("Test 04: ");
		int tab[] = {100, 50, 20, 10, 2, 1};
		int p = 0;
		for(int i = 0; i < 10; i++){
			int rand = tab[(int)(Math.random()*100)%tab.length];
			int nb = (int)(Math.random()*100)%6;
			p += (rand==100?100*nb:0);
			p += (rand==50?50*nb:0);
			p += (rand==20?20*nb:0);
			p += (rand==10?10*nb:0);
			p += (rand==2?2*nb:0);
			p += (rand==1?1*nb:0);
			dab.nbBillet100-=(rand==100?nb:0);
			dab.nbBillet50-=(rand==50?nb:0);
			dab.nbBillet20-=(rand==20?nb:0);
			dab.nbBillet10-=(rand==10?nb:0);
			dab.nbPiece2-=(rand==2?nb:0);
			dab.nbPiece1-=(rand==1?nb:0);
		}
		if(dab.calculerMontantRestantDansDABiP() == 17639-p){
			score++;
			System.out.println("ok");
		}
		else{
			System.out.println("non");
		}
 
		//Test 5
		System.out.print("Test 05: ");
		dab.nbBillet100 = 0;
		dab.nbBillet50 	= 0;
		dab.nbBillet20 	= 0;
		dab.nbBillet10 	= 0;
		dab.nbPiece2 	= 0;
		dab.nbPiece1 	= 0;
		if(dab.calculerMontantRestantDansDABiP() == 0){
			score++;
			System.out.println("ok");
		}
		else{
			System.out.println("non");
		}
 
		//Test 6
		System.out.print("Test 06: ");
		dab = new DABiP();
		boolean r = dab.retrait(0);
		if(r && dab.nbBillet100 == 100 && dab.nbBillet50 == 100 && dab.nbBillet20 == 100 && dab.nbBillet10 == 100 && dab.nbPiece2 == 100 && dab.nbPiece1 == 100){
			score++;
			System.out.println("ok");
 
		}
		else{
			System.out.println("non");
		}
 
		//Test 7
		System.out.print("Test 07: ");
		dab = new DABiP();
		r = dab.retrait(40);
		if(r && dab.nbBillet100 == 100 && dab.nbBillet50 == 100 && dab.nbBillet20 == 98 && dab.nbBillet10 == 100 && dab.nbPiece2 == 100 && dab.nbPiece1 == 100){
			score++;
			System.out.println("ok");
 
		}
		else{
			System.out.println("non");
		}
 
		//Test 8
		System.out.print("Test 08: ");
		dab = new DABiP();
		r = dab.retrait(367);
		if(r && dab.nbBillet100 == 97 && dab.nbBillet50 == 99 && dab.nbBillet20 == 100 && dab.nbBillet10 == 99 && dab.nbPiece2 == 97 && dab.nbPiece1 == 99){
			score++;
			System.out.println("ok");
 
		}
		else{
			System.out.println("non");
		}
 
		//Test 9
		System.out.print("Test 09: ");
		dab = new DABiP();
		r = dab.retrait(18300);
		if(r && dab.nbBillet100 == 0 && dab.nbBillet50 == 0 && dab.nbBillet20 == 0 && dab.nbBillet10 == 0 && dab.nbPiece2 == 0 && dab.nbPiece1 == 0){
			score++;
			System.out.println("ok");
 
		}
		else{
			System.out.println("non");
		}
 
		//Test 10
		System.out.print("Test 10: ");
		dab = new DABiP();
		r = dab.retrait(18301);
		if(!r && dab.nbBillet100 == 100 && dab.nbBillet50 == 100 && dab.nbBillet20 == 100 && dab.nbBillet10 == 100 && dab.nbPiece2 == 100 && dab.nbPiece1 == 100){
			score++;
			System.out.println("ok");
 
		}
		else{
			System.out.println("non");
		}
 
		//Test 11
		System.out.print("Test 11: ");
		dab = new DABiP();
		r = dab.retrait(19634);
		if(!r && dab.nbBillet100 == 100 && dab.nbBillet50 == 100 && dab.nbBillet20 == 100 && dab.nbBillet10 == 100 && dab.nbPiece2 == 100 && dab.nbPiece1 == 100){
			score++;
			System.out.println("ok");
 
		}
		else{
			System.out.println("non");
		}
 
		//Test 12
		System.out.print("Test 12: ");
		dab = new DABiP();
		dab.nbBillet100 = 2;
		r = dab.retrait(300);
		if(r && dab.nbBillet100 == 0 && dab.nbBillet50 == 98 && dab.nbBillet20 == 100 && dab.nbBillet10 == 100 && dab.nbPiece2 == 100 && dab.nbPiece1 == 100){
			score+=2;
			System.out.println("ok");
 
		}
		else{
			System.out.println("non");
		}
 
		//Test 13
		System.out.print("Test 13: ");
		dab = new DABiP();
		dab.nbBillet100 = 2;
		dab.nbBillet50 	= 1;
		dab.nbBillet20 	= 5;
		dab.nbBillet10 	= 6;
		dab.nbPiece2 	= 50;
		r = dab.retrait(407);
		if(r && dab.nbBillet100 == 0 && dab.nbBillet50 == 0 && dab.nbBillet20 == 0 && dab.nbBillet10 == 1 && dab.nbPiece2 == 47 && dab.nbPiece1 == 99){
			score+=2;
			System.out.println("ok");
 
		}
		else{
			System.out.println("non");
		}
 
		//Test 14
		System.out.print("Test 14: ");
		dab = new DABiP();
		dab.nbBillet100 = 0;
		dab.nbBillet50 	= 0;
		dab.nbBillet20 	= 0;
		dab.nbBillet10 	= 0;
		dab.nbPiece2 	= 0;
 
		r = dab.retrait(57);
		if(r && dab.nbBillet100 == 0 && dab.nbBillet50 == 0 && dab.nbBillet20 == 0 && dab.nbBillet10 == 0 && dab.nbPiece2 == 0 && dab.nbPiece1 == 43){
			score+=2;
			System.out.println("ok");
 
		}
		else{
			System.out.println("non");
		}
 
		//Test 15
		System.out.print("Test 15: ");
		dab = new DABiP();
		r = dab.retrait(-1634);
		if(!r && dab.nbBillet100 == 100 && dab.nbBillet50 == 100 && dab.nbBillet20 == 100 && dab.nbBillet10 == 100 && dab.nbPiece2 == 100 && dab.nbPiece1 == 100){
			score++;
			System.out.println("ok");
 
		}
		else{
			System.out.println("non");
		}
 
		//Test 16
		System.out.print("Test 16: ");
		dab = new DABiP();
		dab.nbBillet100 = 5;
		dab.nbBillet50 = 0;
		dab.nbBillet20 = 0;
		dab.nbBillet10 = 0;
		dab.nbPiece2 = 0;
		dab.nbPiece1 = 5;
		r = dab.retrait(12);
		if(!r && dab.nbBillet100 == 5 && dab.nbBillet50 == 0 && dab.nbBillet20 == 0 && dab.nbBillet10 == 0 && dab.nbPiece2 == 0 && dab.nbPiece1 == 5) {
			score+=2;
			System.out.println("ok");
		}
		else {
			System.out.println("non");
		}
 
 
		/**
                 * NOTATION
                 */
 
		System.out.println("--------------------------");
		System.out.println("Note: "+score+"/20");
 
 
	}
}

Salut,

Premièrement, pour déboguer ton code, tu pourrais commenter tous les tests sauf l'un de ceux qui ne passent pas, en ajoutant pour commencer un simple affichage des différents nombres de billets (et de pièces, bizarre pour un DAB mais admettons) présents dans le DAB après le retrait et en face le nombre attendu dans le test. Cela te permettrait d'avoir un/des indices.
Par exemple (ici sur chaque ligne, dans l'ordre la valeur, le nombre présent dans le DAB puis le nombre attendu :
Test 08: 
100 97 97
50 100 99
20 97 100
10 100 99
2 97 97
1 99 99
non
On voit tout de suite qu'il y a un souci entre les billets de 50, de 20, et de 10. Donc on peut réduire le problème : on a 67 euros à retirer, on devrait obtenir 1 billet de 50, et on obtient 0. En résumé, ton DAB donne 20x3 alors que le test voudrait qu'on ait 50+10. Question à se poser : pourquoi ?

Entre parenthèses, ce serait pas mal d'avoir les spécifications pour savoir exactement ce que tu dois implémenter (les règles de gestion en somme).

Ensuite, regarde bien tes tests qui sont tous sous la forme (sauf le dernier, pour les pièces de 1, donc question, pourquoi ce cas différent des autres ?).
Code:

1 2 3 4 5 6 7 8 9 10 11 12 if( somme/V > 1){ // cas 1 x = somme/V; if(nbV - x >= 0){ // cas 2 nbV = nbBilletV- x; somme = somme - V * x; } else { // cas 3 x = nbV; nbV = 0; somme = somme - V * x; } }
cas 1
Le test dit que si somme/V est > 1, on prend des billets de valeur V (ou des pièces, peu importe). Si somme=V, somme/V==1. Donc si on demande par exemple 100 euros, on n'obtient pas de billet de 100 euros. Est-ce normal ? Sans les spécifications, cela m'est impossible de le savoir. Mais en tout cas, par rapport à la différence entre 0 et 1 billet de 50 pour une somme de 67, on a un indice intéressant.
cas 3
Pour que tu comprennes mieux ce qu'il se passe, ajoute à ton code une trace qui affiche les différentes valeurs (x, somme, nbV), avant et après chaque test (pour chaque billet/pièce de valeur V). Question : que se passe-t-il quand V vaut 0 (voir test 16, nbpieces2 vaut 0, regarde les traces que je t'ai fait ajoutées et tu devrais comprendre le souci).

Merci grace a vous j'ai reussi valider le test 08, mais je n'arrive pas a faire le 16

Citation:

Envoyé par Jika971

Merci grace a vous j'ai reussi valider le test 08, mais je n'arrive pas a faire le 16

Et tu as mis en place les traces que je t'ai indiquées ? Si oui, que remarques-tu (pour le test 16) ? Si non, fais-le...

SALUT

Vérifie d'abord la valeur de la boucle final lors du test je pense cela que doit venir le problème.