Discussion :

[clavier12AZQSWX]

bonjour,

en regardant le changelog d'une API, j'ai vu ça :

Increased overall code quality by reaching PHPStan Level 7

est-ce que ça parle à quelqu'un ?
j'ai jamais entendu de parler de norme qualitative de php.
quelqu'un a une expérience avec cela ?

est-ce par exemple d'écrire du code propre ou du code optimisé ?

bonne journée

[Séb.]

PHPStan est un outil d'analyse du code.

Tu peux le paramétrer sur différents niveaux d'analyse, plus le niveau est haut, plus l'analyse est stricte.

https://phpstan.org/user-guide/rule-levels

[Obsidian]

j'ai jamais entendu de parler de norme qualitative de php.
quelqu'un a une expérience avec cela ?

Je ne connaissais pas PHPStan en particulier mais une brève recherche nous mène à ceci : Rule levels

est-ce par exemple d'écrire du code propre ou du code optimisé ?

Oui, ça se fait beaucoup dès lors qu'un projet commence à avoir de l'envergure. Pas que le code soit forcément meilleur après ça mais au moins, il a la même allure partout. Le même genre de règle est en vigueur dans le noyau Linux par exemple. Le tout est de choisir des règles censées — et justifiées — pour que la communauté les accepte.

Le revers de ce genre de vérificateur est que, de leur côté, ils tendent à intégrer tout ce qui sort un poil de l'ordinaire pour être sûrs d'être exhaustifs et que, du côté du mainteneur, on a tendance à aller activer toutes les options pour être sûrs d'être efficace sans se demander si c'est réellement utile.

Moralité, le programmeur n'a pratiquement plus aucune liberté et se retrouve parfois même contraint d'écrire des choses qu'il évite par principe par ailleurs et ça devient assez frustrant.

L'un des exemples qui m'horripile le plus à titre personnel est le fait d'imposer un bloc d'accolade { } même s'il n'y a qu'une seule instruction dedans. Moi je considère cela comme une mauvaise pratique. Le fait que le Rust, par exemple, ait imposé cela dans sa norme me déprime.

[bretus]

Bonjour,

Oui, ça se fait beaucoup dès lors qu'un projet commence à avoir de l'envergure. Pas que le code soit forcément meilleur après ça mais au moins, il a la même allure partout.

PHPstan traite plutôt l'analyse statique du code que la mise en forme (c'est plutôt du côté de PHP-CS-Fixer qu'il faut regarder pour ça).

L'approche par niveau de PHPstan est vraiment intéressante pour monter en gamme progressivement :

- Les premiers niveaux sont généralement indolores sur code PHP "moderne" (même si les types des paramètres et types de retour ne sont pas toujours définis)
- De mémoire, les niveaux 7 et 8 permettent de s'astreindre à typer les retours et vérifier par exemple que le cas null est bien traité par l'appelant d'une méthode renvoyant string|null.

Avec cette approche, PHPStan permet de s'imposer la rigueur d'un langage fortement typé en PHP (donc de pouvoir vérifier des éléments avant l'exécution, donc de pouvoir éviter plus facilement certains bugs...)

Par rapport à d'autres outils qui balancent une batterie de "code smells" plus ou moins pertinents, ce côté progressif motive au passage (il y a de la fierté dans ce "Increased overall code quality by reaching PHPStan Level 7" )