Je n'ai pas encore jeté un œil aux sources, mais je m'incruste sur le sujet des fonctions.
Pour ma part, je ne suis pas adepte non plus de fonctions à tout va... Cependant, je crois comprendre que la remarque initiale était qu'il y avait des blocs factorisables ; et là je trouve que c'est pertinent pour l'évolution et la maintenabilité future. Cela n'oblige nullement à « sans arrêt faire des sauts d'un bout à l'autre du script » si on respecte certaines règles (une fonction bien nommée et avec une signature bien pensée, on n'a pas à y retourner constamment voir pas du tout...)
Pour les fonctions qui ne sont utilisées qu'une seule fois, oui ça peut ne pas être utile de faire pareil ...sauf si justement ces fonctions permettent de réduire la complexité (au sens de prise en main par d'autres) de la routine principale, ou si un aspect fonctionnel/métier complexe pouvant évoluer et/ou être réutilisé par la suite (on est alors dans le cas d'évolutions/améliorations anticipées.)
Pour le niveau d'indentation, cinq ne me semble pas trop mais c'est déjà limite. J'ai remarqué qu'à partir du troisième ou quatrième niveau on perd beaucoup de gens (ou plutôt il leur faut plus d'effort pour arriver à suivre.)
Quand on partage du code, il faut faire quelques compromis pour favoriser la contribution d'un assez grand nombre (sans bien sûr chercher à contenter tout le monde)
"It is seldom that liberty of any kind is lost all at once." ― David Hume
[^] # Re: Outil intéressant mais...
Posté par Gil Cot ✔ (site web personnel, Mastodon) . En réponse au journal [PHP] Apache Check, première release. Évalué à 7.
Salutations,
Je n'ai pas encore jeté un œil aux sources, mais je m'incruste sur le sujet des fonctions.
Pour ma part, je ne suis pas adepte non plus de fonctions à tout va... Cependant, je crois comprendre que la remarque initiale était qu'il y avait des blocs factorisables ; et là je trouve que c'est pertinent pour l'évolution et la maintenabilité future. Cela n'oblige nullement à « sans arrêt faire des sauts d'un bout à l'autre du script » si on respecte certaines règles (une fonction bien nommée et avec une signature bien pensée, on n'a pas à y retourner constamment voir pas du tout...)
Pour les fonctions qui ne sont utilisées qu'une seule fois, oui ça peut ne pas être utile de faire pareil ...sauf si justement ces fonctions permettent de réduire la complexité (au sens de prise en main par d'autres) de la routine principale, ou si un aspect fonctionnel/métier complexe pouvant évoluer et/ou être réutilisé par la suite (on est alors dans le cas d'évolutions/améliorations anticipées.)
Pour le niveau d'indentation, cinq ne me semble pas trop mais c'est déjà limite. J'ai remarqué qu'à partir du troisième ou quatrième niveau on perd beaucoup de gens (ou plutôt il leur faut plus d'effort pour arriver à suivre.)
Quand on partage du code, il faut faire quelques compromis pour favoriser la contribution d'un assez grand nombre (sans bien sûr chercher à contenter tout le monde)
"It is seldom that liberty of any kind is lost all at once." ― David Hume