Discussion :

[Sallie]

Bonsoir tout le monde,

Il y'a une question qui me trotte dans la tête depuis un moment. Voilà je fais par habitude mes recherches ou autres itérations de la façon suivante :

1:

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
 
public PosteBudgétaire getPoste(String s){
		boolean trouvé=false;
		PosteBudgétaire posteBudgétaireARetourner=null;
		PosteBudgétaire posteBudgétaire;
		Iterator ite = postes.iterator();
		while(ite.hasNext() && !trouvé){
			posteBudgétaire = (PosteBudgétaire) ite.next();
			if( posteBudgétaire.getLabel().compareToIgnoreCase(s)==0 ){
				trouvé=true;
				posteBudgétaireARetourner=posteBudgétaire;
			}
		}
		return posteBudgétaireARetourner;
	}

Il me semblait qu'un professeur m'avait recommandé de ne jamais mettre plusieurs return. Pourtant en réfléchissant le mettre de mettre un return avt permet de sortir de la boucle plus tôt si l'objet est trouvé. Et en plus sur les exemples de codes rencontrés sur le net c'est souvent l'autre façon

ce qui donnerait

2:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
	public PosteBudgétaire getPoste(String s){
		PosteBudgétaire posteBudgétaire;
		Iterator ite = postes.iterator();
		while(ite.hasNext()){
			posteBudgétaire = (PosteBudgétaire) ite.next();
			if( posteBudgétaire.getLabel().compareToIgnoreCase(s)==0 ){
				return posteBudgétaire;
			}
		}
		return posteBudgétaire;
	}

Quelle méthode est donc meilleure syntaxiquement? la 1 ou la 2?

merci d'avance =)

[Romain B.]

Pour ma part, je préfère la première car je n'aime pas non plus mettre plusieurs return dans une fonction.

[mavina]

Coucou,

la deuxième sans hésitation.

Le return te permet d'éviter de continuer l'itération pour rien. Tu as trouvé, tu retournes. C'est comme ci tu cherchais apres quelque chose, que tu le trouvais dans une étagere mais que tu continuais à chercher dans les autres...

F.

[Sanguko]

Par contre, ca s'ecrit plutot comme ca :

Code :

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

1
2
3
4
5
6
7
8
9
10
public PosteBudgétaire getPoste(String s){
  Iterator ite = postes.iterator();
  while(ite.hasNext()){
    PosteBudgétaire posteBudgétaire = (PosteBudgétaire) ite.next();
    if( posteBudgétaire.getLabel().compareToIgnoreCase(s)==0 ){
	return posteBudgétaire;
    }
  }
  return null;
}

Mais c'est surtout une histoire de gout.

[T`lash]

Moi aussi je préfère la solution des return, mais il en existe une autre ; plutôt que ta variable trouvé tu peux utiliser un break.

[GanYoshi]

Je crois que les breaks sont aussi très déconseillés...

Je pense que la meilleur façon est de faire comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public PosteBudgétaire getPoste(String s){
		PosteBudgétaire posteBudgétaire;
                PosteBudgétaire posteBudgétaireARetourner = null;
		Iterator ite = postes.iterator();
                boolean trouve = false;
		while(ite.hasNext() && trouve == false){
			posteBudgétaire = (PosteBudgétaire) ite.next();
			if( posteBudgétaire.getLabel().compareToIgnoreCase(s)==0 ){
                                posteBudgétaireARetourner = posteBudgétaire;
                                trouve = true;
			}
		}
		return posteBudgétaireARetourner ;
	}

De cette manière, on ne continue pas à chercher alors qu'on a pas trouvé, et on n'utilise pas plusieurs return....
Mais dans la pratique, je mettrais plusieurs return, puisque la fonction est petite et donc facilement compréhensible.

[Napalm51]

Par contre, ca s'ecrit plutot comme ca :

Code :

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

1
2
3
4
5
6
7
8
9
10
public PosteBudgétaire getPoste(String s){
  Iterator ite = postes.iterator();
  while(ite.hasNext()){
    PosteBudgétaire posteBudgétaire = (PosteBudgétaire) ite.next();
    if( posteBudgétaire.getLabel().compareToIgnoreCase(s)==0 ){
	return posteBudgétaire;
    }
  }
  return null;
}

Mais c'est surtout une histoire de gout.

Je prendrais cette solution aussi :p même si j'ai toujours cette grosse voix d'un prof d'IUT dans la tête qui me dit : "Une fonction, un return !" méthode monsieur, une méthode

[themadmax]

Ah, je suis content de tomber sur ce poste!
Voila moi je suis un adepte du return, et j'ai aussi entendu du mal de lui!
Parcontre on ne ma jamais dit aucun argument!

Donc j'exprime les miens pour le return :

• Quand on trouve l'item, on sort directement on sait que rien d'autre ne sera fait pas la suite (parfois loin dans la fonction). On retourne directement l'objet trouvé, qui parfois n'est plus valable en dehors de la boucle.
• Simplicité de code : rajouter un boolean dans un for ou un while je trouve cela pas très visible, et biensur qui dit rajout de code dit rajout potentiel de bug!
• Lors de 15 conditions dans une fonction on se retrouve avec un bon nombre d'indentation si on ne fait pas des return à chaque condition.
• En C le code generé est plus long (rajout d'une condition) ce qui pour des grande recherche peut s'averer un peu plus couteux.