J'ajouterais sur ma réponse que puisque un "beau" code qui ne sert à rien est un mauvais code. Le code review n'est donc qu'une petite partie du processus qui vise à écrire du code de qualité. Le but est de faire intervenir toutes les personnes qui peuvent fournir un retour dès qu'un changement est fait (et de les faires participer aux décisions quand nécéssaire).
Ceci peut inclure:
- Toute l'équipe pour discuter des choix
- Un autre dev pour faire de la revue de code. Le but étant de trouver les typos, les erreurs, les améillorations possibles etc.
- Un UI/UX qui va regarder l'impact du changement sur l'application etc.
- Un QA qui va avoir une autre vision des tests, sont avis sur l'integration générale etc.
- Un consommateur de la fonctionalité qui va dire si ca lui convient etc.
Bref on écrit du code pour une raison, un beau code c'est un code qui répond à cette raison. Il doit être simple, maintenable et ne pas en faire plus que nécéssaire. Ce qu'on ne fait pas est aussi important que ce que l'on fait.
[^] # Re: Code review
Posté par ckyl . En réponse au journal Du code propre, c'est quoi ?. Évalué à 4.
J'ajouterais sur ma réponse que puisque un "beau" code qui ne sert à rien est un mauvais code. Le code review n'est donc qu'une petite partie du processus qui vise à écrire du code de qualité. Le but est de faire intervenir toutes les personnes qui peuvent fournir un retour dès qu'un changement est fait (et de les faires participer aux décisions quand nécéssaire).
Ceci peut inclure:
- Toute l'équipe pour discuter des choix
- Un autre dev pour faire de la revue de code. Le but étant de trouver les typos, les erreurs, les améillorations possibles etc.
- Un UI/UX qui va regarder l'impact du changement sur l'application etc.
- Un QA qui va avoir une autre vision des tests, sont avis sur l'integration générale etc.
- Un consommateur de la fonctionalité qui va dire si ca lui convient etc.
Bref on écrit du code pour une raison, un beau code c'est un code qui répond à cette raison. Il doit être simple, maintenable et ne pas en faire plus que nécéssaire. Ce qu'on ne fait pas est aussi important que ce que l'on fait.