Discussion :

[TotoDev59]

Bonjour,

j'aimerais savoir si le fait de mettre plusieurs return dans une méthode était une bonne pratique en programmation ou pas si ca permet d'économiser des lignes.

Voici un exemple de code :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public bool supprimerBranchement (int numero) {
   Branchement lesBranchements = new Branchement();
   try {
      this.lesBranchements.RemoveAt(numero);
      return true;
   }
   catch (Exception) {
      return false;
   }

[BenoitM]

Bonjour,
j'aimerais savoir si le fait de mettre plusieurs return dans une méthode était une bonne pratique en programmation ou pas si ca permet d'économiser des lignes.

Non le but n'est pas de faire le programme le plus court, part contre si ca facilite la lisibilité oui


(dans ce cas ca ne me choque pas)

[morgand]

Personnellement avoir plusieurs return dans une fonction ce n'est pas grave. Par contre, si tu as la main sur la class "Branchement", j'aurai pas fait ça

Code :

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

1
2
3
4
5
 
public bool supprimerBranchement (int numero) {
   //Branchement lesBranchements = new Branchement(); // Je suppose que cette ligne est à titre informatif pour notre compréhension et non dans ton code à ce niveau. Car sinon tu ne pourra pas faire le "this."
     return this.lesBranchements.RemoveAt(numero);
}

Et tu délègue à ta fonction RemoveAt le soins de retourner True si c'est remove sinon false.
Car le test via un Try Catch par contre n'est pas une bonne pratique. En faite il faut tester si la valeur existe, avec un bon block IF

[Babyneedle]

La bonne pratique est de ne pas utiliser les exceptions comme une mesure de succès.

Premièrement, même si aucune exception n'est générée, peut-être qu'aucun fichier n'a été effacé.

Un exemple de code compact qui prend avantage de la valeur de retour du nombre de fichiers affectés.

Code :

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

return new MyEntity(myID).Delete() == numberOfExpectedDeletedRecords;

Si une exception survient, laisse la responsabilité au module client de la gérer de de modifier son comportement en conséquence.

[Graffito]

le fait de mettre plusieurs return dans une méthode est-il une bonne pratique en programmation ou pas

Il y a 20 ans nos méthodes de développement avaient été audités et l'une des critiques de cet audit était qu'il y avait parfois plusieurs return dans les fonctions.

Depuis, nous avons toujours mis un return unique en fin de procedure et l'expérience nous a montré qu'il s'agissait effectivement d'une bonne pratique.
En complément, nous nommons systématiquement "Result" la variable utilisée dans l'instruction return en fin de procedure.

Code :

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

1
2
3
4
5
6
7
8
public bool supprimerBranchement (int numero)
{    
     bool Result = false ;
     Branchement lesBranchements = new Branchement();    
     try { this.lesBranchements.RemoveAt(numero); Result=true ; } 
     catch (Exception) { Result = false; }
     return Result ;
}

[BenoitM]

bon je ne parlerais pas de ce cas-ci mais plutot dans la généralité.

Avoir un return -1 au debut de la fonction ne me choque pas.
Avoir un return element quand on le trouve ne me choque pas non plus.

Perso je préfere avoir plusieurs return dans une fonction que de devoir réfléchir où va le programme en suite
exemple

Code :

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

1
2
3
if(paramNonOk) return -1;
code
return retVal;

me semble plus lisible que

Code :

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

1
2
3
4
5
6
7
8
9
if(paramNonOk) 
{
    retVal =-1;
}
else 
{
    code... 
}
return retVal;

ca fait une indentation pour rien , je dois verifier si il n'y a rien après le else même si l'ide aide à voir les les block {}