Discussion :

[Nooby300]

Bonjour,

Il y a quelques jours, alors que je programmais en cours d'informatique, ma prof m'a dit qu'il ne fallait pas utiliser plusieurs return dans une seule fonction... J'ai appris à programmer sur France-IOI et leurs corrections utilisent souvent plusieurs return... Je lui demande alors pourquoi je ne peux pas utiliser plusieurs return. Je n'ai pas eu de réponses sérieuses ni d'arguments valide, mais un simple : "parce que c'est caca" (mot pour mot)

Je viens donc ici chercher une réponse concrète faute de trouver mieux dans mon école (IUT).

Merci

[sevyc64]

C'est la seule réponse que tu auras.

Çà ne fait pas partie des bonnes pratiques de certains, ça ne dérange pas d'autres. Techniquement rien ne l’empêche. Philosophiquement, tu pourras avoir des arguments du style "Çà fait pas propre", "Çà donne du code qui peut être difficile à maintenir", etc ..

[Médinoc]

Ça reste plus facile à lire dans certains cas, notamment pour le contrôle des pré-conditions d'une fonction (genre vérifier qu'un pointeur passé en entrée n'est pas nul, etc.).

[el_slapper]

C'est purement subjectif, mais je m'y astreins. Un seul return, ça limite le risque de mauvaises surprises à la maintenance. Quand tu commence à avoir plusieurs sorties à ton paragraphe, et que tu dois rajouter des coupures au milieu, tu a vite fait de tout casser.

[Médinoc]

Disons qu'il convient de séparer l'entrée et la "business logic" de la fonction.
Dans la fonction elle-même et un langage comme le C, l'absence de return mène inévitablement à du "code boomerang" qui devient aussi illisible que du code avec return multiples quand il dépasse un certain niveau.

En autorisant les returns sur les préconditions, on économise des niveaux et le code peut rester lisible plus longtemps:

Code C :

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
22
23
24
25
26
27
28
29
char* UneFonction(int a, char*b, int*c)
{
	char* ret = NULL;
	if(a < 0) { return NULL; }
	if(b == NULL) { return NULL; }
	if(c == NULL) { return NULL; }
 
	/*Et maintenant, la "business logic", plus de return jusqu'à la fin de la fonction*/
 
	/*Ici commence le code boomerang*/
	{
		FILE *f = fopen("blabla", "r");	
		if(f != NULL)
		{
			char* buffer = malloc(a);
			if(buffer != NULL)
			{
				ret = malloc(strlen(b)+1);
				if(ret != NULL)
				{
					/* bla bla bla */
				}
				free(buffer), buffer=NULL;
			}
			fclose(f), f=NULL;
		}
		return ret;
	}
}

Après, dans des langages supportant les exceptions, on a forcément un flot de contrôle pouvant être "interrompu", donc un return de plus ou de moins...

[DonQuiche]

SI on a des fonctions de trois pages je comprends qu'on préfère s'abstenir d'introduire des return : cela évite d'avoir sans cesse à devoir vérifier le reste de la fonction.

Mais si on a des fonctions courtes, de dix lignes environ, comme le recommandent les bonnes pratiques actuelles, il n'y a aucun problème à utiliser plusieurs return, c'est même tout à fait recommandé si cela peut limiter les imbrications.