Réussir sa review

Utiliser des noms de variable/méthodes explicites

Avantage: On sait ce qu'on manipule

https://medium.com/swlh/how-to-better-name-your-functions-and-variables-e962a4ef335b

Réussir sa review

Ne pas pré-setter toutes les variables au début de la méthode

Avantage: Pas besoin de se rappeler la valeur de la variable

https://wiki.c2.com/?DeclareVariablesAtFirstUse

Réussir sa review

Ne pas pré-setter une variable pour l'override après

Avantage: Pas besoin de se rappeler la valeur d'initialisation

Réussir sa review

Faire des early-return si la suite du code n'est pas nécessaire

Avantage: Pas besoin de lire la suite si on est dans le "early" cas

https://szymonkrajewski.pl/why-should-you-return-early/

Réussir sa review

Préférer les exceptions aux fallback pour les cas impossibles

Avantage: Les bugs ne sont pas silencieux, donc détectés plus tôt

https://afilina.com/learn/nulls/return-types

Réussir sa review

Toujours gérer les valeurs null/false, même si "impossibles"

Avantage: Le code est plus robuste
+ une erreur en moins PHPStan 8

https://medium.com/swlh/2-defensive-coding-techniques-you-should-use-today-4225cacc1c29

Réussir sa review

Commenter le pourquoi et non le quoi

Avantage: On comprend mieux l'intérêt du code en question et on évite d'introduire un bug en le modifiant

https://javascript.plainenglish.io/clean-code-writing-comments-in-a-good-way-8203c7d80c65

Réussir sa review

Typer le contenu des array dans la phpdoc

Avantage: PHPStan bosse pour vous et la méthode est plus claire

https://phpstan.org/writing-php-code/phpdoc-types

Réussir sa review

Utiliser des booléens dans les conditions

Avantage: On comprend plus facilement le cas écarté

https://dev.to/aleksikauppila/using-isset-and-empty-hurts-your-code-aaa

Réussir sa review

Éviter les méthodes statiques

Avantage: La méthode peut être mockée dans les tests unitaires lorsqu'elle est utilisée par un autre service

https://stovepipe.systems/post/avoiding-static-in-your-code

Réussir sa review

Ne pas utiliser de state interne dans un service ou faire des "appels en deux temps"

Avantage: L'utilisation de la méthode est définie dans sa signature, la méthode est pure (les mêmes params donnent le même résultat)

https://igor.io/2013/03/31/stateless-services.html

Réussir sa review

Ne pas caster les floats en int ou les comparer directement

Avantage: Pas de risque de bug

https://techtalkbook.com/why-0-1-0-2-does-not-equal-0-3​