Bonsoir à tous,

Voilà j'ai décidé de réécrire une fonction assez complexe (~ 500 lignes) afin d'améliorer les performances et la lisibilité du code

Je me posais une question au niveau du retour des erreurs: au lieu de faire une condition comme celle-là:

if (count($tableau) > 0) {
    Suite du code
} else return "Erreur";

Y'a-t-il un inconvénient de faire directement:

if (count($tableau) == 0) return "Erreur";
Suite du code

Sachant que le return « break » la fonction, je trouve que c'est plus lisible et que ça correspond plus à l'organisation de mon code

Savez-vous si la seconde option peut influencer la performance et la sécurité de mon script ?

Merci d'avance !

Ta solution est meilleure car elle améliore la lisibilité du code en enlevant un niveau d'imbrication pour ce qui suit.

Y a évidemment aucun impact de performances ou de sécurité, c'est un peu bizarre comme question ça pour le coup

Merci pour ta réponse
Je me posais cette question car je ne savais pas si l'interruption nette de la fonction pouvait avoir un impact négatif (comme une fuite de mémoire)

Hors du fait que RogerLeCaillou t'a apporté une réponse solide comme le roc (haha), une fonction de 500+ lignes, c'est un nid à bug. Tu peux plus que certainement en extraire plusieurs petites fonctions, qui seront plus faciles à manipuler par la suite.

Bien qu'il n'ait pas de réel consensus, on essaie de réduire les fonctions au maximum, et dès qu'une fonction dépasse une trentaine de lignes, elle mérite (probablement) un refactor.

Dans une fonction il y a autant de return que de résultats possibles à la fonction en fonction des paramètres.
Pour prendre un exemple connu, fait et refait par les étudiants apprenant à développer, la fonction Fibonacci(n) donne :

int Fibonacci(int n){
     if (n <= 1) return 1;
     else return Fibonacci(n - 1) + Fibonacci(n - 2); 
}

Comme tu peux le voir y a plusieurs return la dedans, un pour chaque possibilité.

(En l'occurrence c'est du C mais le principe reste identique.)

Bonjour,
La solution donnée est bonne, si je puis me permettre d'ajouter une remarque : pour les remontées d'erreurs je te recommande d'utiliser les exceptions.
Bonne soirée

Resharper dit souvent "Invert "if" statement to reduce nesting". Si je ne fais pas erreur, en qualité logicielle, la méthode du if () return; permet d'améliorer la profondeur du code

tu peux ajouter des assertions statiques afin de check des conditions lors de la compilation du programme.
Ca rendra le code plus lisible et ça sera plus propre.

C'est très bien ce que tu fais, sauf le return "erreur".

Comme ils ont dit les exceptions ca peut être utile, mais si tu les try catch pour au final mettre ta variable à null, renvoie null directement.

Le 11 avril 2016 à 00:01:36 rangerprice a écrit :
tu peux ajouter des assertions statiques afin de check des conditions lors de la compilation du programme.
Ca rendra le code plus lisible et ça sera plus propre.

Attention, les assertions n'ont pas le même objectif que les tests if/return ou if/throw, faut bien se dire que certains compilateurs de certains langages retirent carrément les assertions du binaire généré s'il n'y a pas un flag particulier de déclaré