Discussion :

[Djakisback]

Bonjour,
je me pose une question sur ce warning obtenu (après analyse de code via Eclipse suivant les J2SE Best Practices) pouvant apparaître sur des codes du type :

Code :

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

1
2
3
4
5
6
7
8
		boolean done = false;
		int[] arr = new int[50];
		arr[25] = 12;
		for (int i = 0; i < arr.length && done == false; i++) {
			if(arr[i] == 12)	{
				done = true;
			}
		}

ou :

Code :

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

1
2
3
4
5
6
7
		UnObjet obj = null;
		String unNomDObjet = "nom";
		for (int i = 0; i < UneListeDObjets.size() && obj == null; i++) {
			if(UneListeDObjets.get(i).getName().equals(unNomDObjet))	{
				obj = UneListeDObjets.get(i);
			}
		}

Savez-vous quels pourraient être les problèmes engendrés par ce type d'affectation dans le corps d'une boucle ? car là je ne vois pas. (Les warning sont sur les affectations de 'done' et de 'obj' dans les boucles)
Merci d'avance.

[thelvin]

C'est juste que ce n'est pas très clair.

"C'est bon j'ai fini mon travail, alors pour le montrer j'affecte une valeur à cette variable"
=> O_o quoi ?
Et puis ça fait deux conditions au lieu d'une pour continuer la boucle, ce qui n'est pas super clair non plus.

Pas clair, et une nécessité de maintenir une logique sur une variable à deux endroits distincts. Tout cela encourage des bugs à apparaître quand le code évolue et se fait modifier.

Quand on veut terminer une boucle, on fait break ou return. C'est en tout cas ce que veut dire ton outil.

[JoeChip]

Code :

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

1
2
3
for (int i = 0; i < arr.length && done == false; i++) {
			if(arr[i] == 12)	
				done = true;

est équivalent à

Code :

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

for (int i = 0; i < arr.length && arr[i] != 12; i++) ;

de toutes façons, la boucle ne sert à rien puisqu'aucune info n'en sort, donc en fait tout cela est équivalent à

Code :

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

;

[Djakisback]

@thelvin :
Merci, je comprends ce que tu veux dire. En plus ça ajoute effectivement un test à chaque itération

@BenWillard :
J'avoue, l'exemple 1 était un peu bidon mais ce que tu dis semble faux car ça pourrait être un algo de présence ou non de la valeur 12 dans un tableau.

[JoeChip]

Ah oui c'est vrai qu'il y a "done" qui peut avoir un état différent selon qu'on a trouvé ou non. Mais alors il doit s'appeller "found".