un if plus que embettant!

Version imprimable

Bonjour à tous,

dans mon programme j'ai un if, qui apparement ne soulève pas de problème à eclipse mais quand j'execute, il plante et me renvoit à cette ligne :

Code:

if(Math.abs((t[15].subComplex(Resultat)).mod())<Math.abs((tbest_res[15].subComplex(Resultat)).mod())){

je précise que t est :

Code:

complex t[]=new complex [16];

et qu'il est rempli (avant le if) par l'intermediaire d'une autre méthode.

Resultat :

Code:

complex Resultat= ChiffreC[0];

tbest_res:

Code:

complex tbest_res[]=new complex [16];

subComplex:
Code:

1 2 3 4 5 //************************************************ //* Soustraction * //************************************************ public complex subComplex(complex z) {return new complex(re-z.re, im-z.im);}
Voila à votre avis pourquoi la ligne de code engendre une exception pour eclipse?

merci à tous
bon aprem
aud-

05/01/2008, 16h45
Chatbour

Salut,

C'est quoi l'exception engendrée ?
05/01/2008, 16h49
canary

Salut Chatbour!

Alors en fait c'est une suite de bugs:

Citation:

Exception in thread "main" java.lang.NullPointerException
at ChiffreBidouille.calculC(ChiffreBidouille.java:330)
at ChiffreBidouille.Imaginaire(ChiffreBidouille.java:248)
at menu.Choix(menu.java:48)
at menu.main(menu.java:9)

je pense qu'il vous faut la mehode calculC sinon ça va etre dur de comprendre ?
05/01/2008, 17h05
Chatbour

Citation:

Envoyé par canary

je précise que t est :

Code:

complex t[]=new complex [16];

et qu'il est rempli (avant le if) par l'intermediaire d'une autre méthode.

tu peux nous montrer le corps de cette méthode et comment tu fais l'appel à elle ?

voici la methode ou se trouve le 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
70
71
72
73
74
75
76
77
78
public static void calculC (complex ChiffreC[]) {
 
	Vector value = new Vector();
 
	value.addElement(ChiffreC[1]);
	value.addElement(ChiffreC[2]);
	value.addElement(ChiffreC[3]);
	value.addElement(ChiffreC[4]);
	value.addElement(ChiffreC[5]);
	value.addElement(ChiffreC[6]);
 
	boolean trouve = false;
	complex Resultat= ChiffreC[0];
	complex t[]=new complex [16];
	complex tbest_res[]=new complex [16];
 
	for(int i=0; i<16; i++){
		tbest_res[i]= new complex(0,0);		
	}
 
	for (int a=3;a>=0 ;a--){
		int compt=0;
		if(trouve)
			break;
		for (int b=3;b>=0;b--){
			if(trouve)
				break;
			for (int c=3;c>=0;c--){
				if(trouve)
					break;
				for (int d=3;d>=0;d--){
					if(trouve)
						break;
					for (int e=3;e>=0;e--){
						if(trouve)
							break;
 
					/** 0='+' 1='-' 2='*' 3='/' **/
 
					int tab[]={0,1,2,3};
 
					ArbreNaire noeud1 = new ArbreNaire(tab[a]);
					noeud1.setSons(value);
 
					ArbreNaire noeud2 = new ArbreNaire(tab[b], noeud1, value);
					ArbreNaire noeud3 = new ArbreNaire(tab[c], noeud2, value);
					ArbreNaire noeud4 = new ArbreNaire(tab[d], noeud3, value);
					ArbreNaire noeud5 = new ArbreNaire(tab[e], noeud4, value);
 
					//avant c'était : t=operation_noeudC(noeud1,noeud2,noeud3,noeud4,noeud5,Resultat);
					t=operation_noeudC(noeud1,noeud2,noeud3,noeud4,noeud5,ChiffreC);
 
					if(t[15]==(Resultat)) 
						trouve=true;
 
 
					//else{
 
					if(Math.abs((t[15].subComplex(Resultat)).mod())<Math.abs((tbest_res[15].subComplex(Resultat)).mod())){
					//if((t[15].subComplex(Resultat)).mod()<(tbest_res[15].subComplex(Resultat)).mod()){
						for(int i=0; i<16; i++){
							tbest_res[i]=t[i];
						out.println("tbest_res:"+tbest_res[i]);
						}
					}
 
					if(trouve)
						break;
 
					}
				}
			}
		}
	}
 
	afficheC(tbest_res[1],tbest_res[4],tbest_res[7],tbest_res[10],tbest_res[13],tbest_res);
 
}

et voici la methode qui traite le tableau t:

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
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
/**
 * Meme fonction que la precedente avec recherche de solution la plus proche
 * avec les complexes
 */
public static complex[] operation_noeudC(ArbreNaire noeud1,ArbreNaire noeud2,ArbreNaire noeud3,ArbreNaire noeud4,ArbreNaire noeud5,complex []ChiffreC){
 
	int i=0;
	complex res=new complex();
	complex res1=new complex();
	complex res2=new complex();
	complex res3=new complex();
	complex res4=new complex();
	complex Resultat= new complex(0,0);
	complex bestResult=new complex(1,1);
	complex Result[]=new complex[16];
	complex tabVal[]={new complex(7,7), new complex(6,6), new complex(5,5), new complex(4,4), new complex(3,3), new complex(2,2)};
	boolean trouve=false;
 
	while(i<6 && !trouve){
		for(int j=0; j<6; j++){
			if(trouve)
				break;
 
			complex val1=noeud5.getSonAtC(j);
 
			switch (noeud5.getValue()){
 
			case 0:
				if(tabVal[i].mod()!=val1.mod())
					res=tabVal[i].addComplex(val1);
 
				break;
 
			case 1:
				if(tabVal[i].mod()!=val1.mod()){
					if((tabVal[i].mod())>(val1.mod()))
						res=tabVal[i].subComplex(val1);
 
						else res=val1.subComplex(tabVal[i]);
				}
 
				break;
 
			case 2: 
				if((tabVal[i].mod())!=(val1.mod()))
					res=tabVal[i].multComplex(val1);
 
				break;
 
			case 3: 
				if((tabVal[i].mod())!=(val1.mod())){
					if((tabVal[i].mod())!=0 && (val1.mod()!=0)){
						if((tabVal[i].mod())>val1.mod())
							res=tabVal[i].divComplex(val1);
 
							else
								res=val1.divComplex(tabVal[i]);
					}
				}
 
				break;
 
			}//fin du switch
 
			for(int k=0;k<6;k++){
				if(trouve)
					break;
 
				complex val2=(complex)noeud4.getSonAtC(k);
 
				if(res.mod()!=0){
					switch(noeud4.getValue()){
 
					case 0: 
						if((tabVal[i].mod())!=val2.mod() && val1.mod()!=val2.mod())
							res1=res.addComplex(val2);
 
						break;
 
					case 1:
						if(tabVal[i].mod()!=val2.mod() && val1.mod()!=val2.mod()){
							if(val2.mod()>res.mod())
								res1=val2.subComplex(res);
 
								else res1=res.subComplex(val2);
						}
 
						break;
 
					case 2: 
						if(tabVal[i].mod()!=val2.mod() && val1.mod()!=val2.mod())
							res1=res.multComplex(val2);
 
						break;
 
					case 3:
						if(tabVal[i]!=val2 && val1!=val2){
							if((res.mod()!=0)&& val2.mod()!=0){
								if(res.mod()>val2.mod())
									res1=res.divComplex(val2);
 
									else
										res1=val2.divComplex(res);
							}
						}
 
						break;
 
					}//fin du switch
				}//fin du si
 
				for(int q=0; q<6;q++){
					if(trouve)
						break;
 
					complex val3=noeud3.getSonAtC(q);
 
					if(res1.mod()!=0){
						switch(noeud3.getValue()){
 
						case 0:
							if(tabVal[i].mod()!=val3.mod() &&val1.mod()!=val3.mod() && val2.mod()!=val3.mod())
								res2=res1.addComplex(val3);
 
							break;
 
						case 1:
							if(tabVal[i].mod()!=val3.mod() &&val1.mod()!=val3.mod() && val2.mod()!=val3.mod()){
								if(val3.mod()>res1.mod())
									res2=val3.subComplex(res1);
 
									else
										res2=res1.subComplex(val3);
							}
 
							break;
 
						case 2: 
							if(tabVal[i].mod()!=val3.mod() &&val1.mod()!=val3.mod() && val2.mod()!=val3.mod())
								res2=res1.multComplex(val3);
 
							break;
 
						case 3: 
							if(tabVal[i].mod()!=val3.mod() &&val1.mod()!=val3.mod() && val2.mod()!=val3.mod()){
								if((res.mod()!=0)&& val3.mod()!=0)
									res2=res1.divComplex(val3);
 
									else
										res2=val3.divComplex(res1);
							}
 
							break;
 
						}//fin du switch
					}//fin du si
 
					for(int x=0; x<6; x++){
						if(trouve)
							break;
 
						complex val4=(complex)noeud2.getSonAtC(x);
 
						if(res2.mod()!=0){
							switch(noeud2.getValue()){
 
							case 0: 
								if(tabVal[i].mod()!=val4.mod() && val1.mod()!=val4.mod() && val2.mod()!=val4.mod() && val3.mod()!=val4.mod())
									res3=res2.addComplex(val4);
 
								break;
 
							case 1: 
								if(tabVal[i].mod()!=val4.mod() && val1.mod()!=val4.mod() && val2.mod()!=val4.mod() && val3.mod()!=val4.mod()){
									if(val4.mod()>res2.mod())
										res3=val4.subComplex(res2);
 
										else res3=res2.subComplex(val4);
								}
 
								break;
 
							case 2: 
								if(tabVal[i].mod()!=val4.mod() && val1.mod()!=val4.mod() && val2.mod()!=val4.mod() && val3.mod()!=val4.mod())
									res3=res2.multComplex(val4);
 
								break;
 
							case 3:
								if(tabVal[i].mod()!=val4.mod() && val1.mod()!=val4.mod() && val2.mod()!=val4.mod() && val3.mod()!=val4.mod()){
									if((res2.mod()!=0) && val4.mod()!=0) {
										if(val4.mod()<res2.mod())
											res3=res2.divComplex(val4);
 
											else{ res3=val4.divComplex(res2); }
									}
								}
 
								break;
 
							}//fin du switch
						}//fin du si
 
						for(int z=0;z<6; z++){
							if(trouve)
								break;
 
							complex val5=noeud1.getSonAtC(z);
 
							if(res3.mod()!=0){
								switch(noeud1.getValue()){
 
								case 0: 
									if(tabVal[i].mod()!=val4.mod() && val1.mod()!=val5.mod() && val2.mod()!=val5.mod() && val3.mod()!=val5.mod() && val4.mod()!=val5.mod())
										res4=res3.addComplex(val5);
 
									break;
 
								case 1:
									if(tabVal[i].mod()!=val4.mod() && val1.mod()!=val5.mod() && val2.mod()!=val5.mod() && val3.mod()!=val5.mod() && val4.mod()!=val5.mod()){
										if(val5.mod()>res3.mod())
											res4=val5.subComplex(res3);
 
											else res4=res3.subComplex(val5);
									}
 
									break;
 
								case 2:
									if(tabVal[i].mod()!=val4.mod() && val1.mod()!=val5.mod() && val2.mod()!=val5.mod() && val3.mod()!=val5.mod() && val4.mod()!=val5.mod())
										res4=res3.multComplex(val5);
 
									break;
 
								case 3:
									if(tabVal[i].mod()!=val4.mod() && val1.mod()!=val5.mod() && val2.mod()!=val5.mod() && val3.mod()!=val5.mod() && val4.mod()!=val5.mod()){
										if((res3.mod()!=0)&& val5.mod()!=0){
											if(val5.mod()<res3.mod())
												res4=res3.divComplex(val5);
 
												else res4=val5.divComplex(res3);
 
										}
									}
 
									break;
 
								}//fin du switch
							}//fin du si
 
							if(res4.equals(Resultat))
								trouve=true;
 
							double inter1=(res4.subComplex(Resultat)).mod();
							out.println("inter1:"+inter1);
							double inter2=(bestResult.subComplex(Resultat)).mod();
							out.println("inter2:"+inter2);
 
							if(inter1 <inter2){
								if(verifC(tabVal[i],val1,val2,val3,val4,val5)){ 
								//on verifie si toutes les valeurs sont differentes les unes des autres
 
									bestResult=res4;
									Result[0]=tabVal[i];
									Result[1]=new complex(noeud5.getValue(),0);
									Result[2]=val1;
									Result[3]=res;
									Result[4]=new complex(noeud4.getValue(),0);
									Result[5]=val2;
									Result[6]=res1;
									Result[7]=new complex(noeud3.getValue(),0);
									Result[8]=val3;
									Result[9]=res2;
									Result[10]=new complex(noeud2.getValue(),0);
									Result[11]=val4;
									Result[12]=res3;
									Result[13]=new complex(noeud1.getValue(),0);
									Result[14]=val5;
									Result[15]=res4;
								}//fin du si
							}//fin du si
 
						}//fin du for z
					}//fin du for x
				}//fin du for q
			}//fin du for k
		}//fin du for j
 
		i++;
 
	}//fin du while
 
	return Result;
}

attention, je ne suis pas sure de ne pas avoir fait d'erreur dans cette classe!

Je pense que le problème vient du fait que le bloc code qui remplie le tableau t n'est pas toujours exécuté..

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
if(inter1 <inter2){
	if(verifC(tabVal[i],val1,val2,val3,val4,val5)){ 
	//on verifie si toutes les valeurs sont differentes les unes des autres
 
		bestResult=res4;
		Result[0]=tabVal[i];
		Result[1]=new complex(noeud5.getValue(),0);
		Result[2]=val1;
		Result[3]=res;
		Result[4]=new complex(noeud4.getValue(),0);
		Result[5]=val2;
		Result[6]=res1;
		Result[7]=new complex(noeud3.getValue(),0);
		Result[8]=val3;
		Result[9]=res2;
		Result[10]=new complex(noeud2.getValue(),0);
		Result[11]=val4;
		Result[12]=res3;
		Result[13]=new complex(noeud1.getValue(),0);
		Result[14]=val5;
		Result[15]=res4;
	}//fin du si
}//fin du si

Parfois donc la méthode operation_noeudC() renvoie un tableau non remplie..

05/01/2008, 17h58
canary

lol! j'avais fait cette constatation aussi mais je n'avais pas tiré les conséquences!!

c'est effectivement ce qui se passe car en mettant des mouchards println, il me renvoit des "null".

il faut donc que je renvoie la methode operation_noeudC :cry:

merci pour ton aide!
Aud-
05/01/2008, 18h02
Chatbour

Bonne chance :)

ps : tu peux te mettre un avatar maintenant 8-)
05/01/2008, 19h14
Romain B.

ptite critique au passage....

ne jamais utilisé de break ! (sauf dans les switch/case)...
05/01/2008, 19h24
Chatbour

Salut Romain B.,
peut-tu nous expliquer pourquoi ?
05/01/2008, 19h27
alceste

je me pose aussi la question.. parce que à quoi ça sert de continuer à chercher quelque chose alors qu'on l'a déjà trouvé ?
05/01/2008, 20h15
Romain B.

Citation:

Envoyé par Chatbour

Salut Romain B.,
peut-tu nous expliquer pourquoi ?

selon http://fr.wikibooks.org/wiki/Program...t%C3%A9rations :

Citation:

L'instruction break permet de sortir immédiatement de la boucle, c'est-à-dire de sauter immédiatement à la première instruction qui suit la boucle.

L'utiliser pour cela n'est pas de la bonne programmation. Il vaut donc mieux l'éviter. De plus, son utilisation rend le programme plus difficile à déboguer.

Je n'ai pas de tellement meilleures explications, si ce n'est que, c'est sure, ça ne facilite pas la compréhension et donc la maintenance du code. De plus, on peut toujours s'en passé.
C'est un peu comme accéder à une variable public plutôt que créer des assesseurs, c'est possible, mais pas bien !
05/01/2008, 20h38
alceste

ça ne respecte pas non plus la logique de l'algorithme.. comme je l'ai dis à quoi bon continuer à chercher lorsque l'on a déjà trouvé ce que l'on cherchait ? le seul but étant de trouver ce que l'on cherche, une fois ce but atteint, c'est pas logique de continuer.. c'est chercher inutilement.

enfin personnellement je ne suis vraiment pas convaincu par ton lien, surtout qu'en bas de page pour les boucles nommées l'instruction break est utilisée, et si on suit ce qui est dit il ne faudrait pas non plus utiliser l'instruction continue..
05/01/2008, 20h45
CyberChouan
Les instructions "break" et "continue" sont effectivement à éviter car elles donnent un code de type "goto" qui est à proscrire en programmation orientée objet.

Si tu désires arrêter une boucle lorsqu'un objet est trouvé, le mieux est d'utiliser un booléen et un code de style:
Code:

1 2 3 4 while(!trouve && index < taille) { trouve = chercher(index); index++; }
05/01/2008, 20h57
Romain B.

non mais c'est sure que si tu ne fais ton programme que pour toi, que tu penses pas t'en resservir plus tard, alors ne change rien, surtout si ça marche !

en effet selon "mes usages", il ne faudrait pas non plus utiliser continue.

quelqu'un pour appuyer ou contredire mes propos ?
05/01/2008, 20h57
canary

heum je crois que tout cela est de ma faute! ne vous battez pas, je n'ai pas mit tout mon code et le petit bout extrait fait parti d'une boucle while!
voila je pense qu'il n'y a plus de soucis maintenant :lol:

bonne soirée!
et encore merci à tous
Aud-
05/01/2008, 20h58
Romain B.

merci CyberChouan !
05/01/2008, 21h15
alceste

je rejoins l'avis de cyberchouan :)