Je préfère, pour des raison de clarté, écrire mon code ainsi :
```php
if(!$success)
{
echo 'message à l'utilisateur';
return;
}
if(!$otherSuccess)
{
echo 'message à l'utilisateur';
return;
}
doSomething;
```
On va sombrer dans le troll, mais placer des return à tout va dans une méthode est une très mauvaise pratique quant à la maintenance de ladite méthode.
Il est bien plus pratique (et à peine moins clair) de ne créer qu'une "porte de sortie". J'en ai passé des heures à chercher la raison d'un bug avant de trouver un "return" caché au détour d'un if qui avait été oublié…
[^] # Re: Et la qualité ?
Posté par Etienne RABY . En réponse à la dépêche Caranille 3.5 - Éditeur de MMORPG en PHP. Évalué à 4.
On va sombrer dans le troll, mais placer des return à tout va dans une méthode est une très mauvaise pratique quant à la maintenance de ladite méthode.
Il est bien plus pratique (et à peine moins clair) de ne créer qu'une "porte de sortie". J'en ai passé des heures à chercher la raison d'un bug avant de trouver un "return" caché au détour d'un if qui avait été oublié…