Les bienfaits de la
CodeReview
LA SIMPLICITÉ EST LA SOPHISTICATION SUPRÊME
Léonard de Vinci
-
Intro : Mise en situation
-
C'est quoi une code review ?
- Quoi ?
- Qui ?
- Quand ?
- Comment ?
-
Les avantages
-
Chronophage ?
-
Les vrais obstacles
-
Comment ça marche ?
Mise en situation
Mise en situation
Mise en situation
Aaaah mes yeux !! ça pique ...
Mise en situation
Mise en situation
Quelles sont les conséquences d'un tel code ?
- L'application bascule vers du code spaghetti
- La dette technique augmente
- Besoin de refacto
- Difficulté pour les nouveaux membres du projet
- Frustration de l'équipe
C'est quoi une code review ?
La revue de code est un examen systématique du code source d'un logiciel.
Il peut être comparé au processus ayant lieu dans un comité de lecture, l'objectif étant de trouver des bugs ou des vulnérabilités potentielles ou de corriger des erreurs de conception afin d'améliorer la qualité, la maintenabilité et la sécurité du logiciel.
Qui relit le code ?
Au mieux :
L'ensemble des devs d'une équipe
A minima :
Un dev qui a une connaissance du projet
Quand faire une code review ?
Au mieux :
A minima :
Qu'est ce qu'on regarde dans une code review ?
- Respect des normes PSR
-
Respect des bonnes pratiques
- KISS,
- DRY,
- Fail First,
- Modélisation
- Cohérence des tests
Chronophage ?
Qui pense que c'est chronophage ?
Le problème, c'est que la vérité est ailleurs
Les vrais obstables
- Besoin de formation
- Peur d'être juger
- Rester dans sa zone de confort
- Susceptibilité
- Pas de remise en question du dev
- Le relecteur est un Waffen SS
Les avantages
- Maintenir ou améliorer la qualité du code
- L'appropriation du code par l’équipe
- Veiller aux respects des bonnes pratiques
- Détecter les erreurs d'implémentations
- Formation des développeurs
- Challenger les choix techniques
- Un R.O.I sur la qualité conséquent
- Diminuer la complexité cyclomatique (CRAP)
- Cohésion d'équipe
Le relecteur
- Valider du code, c'est approuvé et avoir confiance au code implémenté
- Doit être bienveillant
- S'assurer que c'est efficace et optimisé
- Se concentrer sur le fond plutot que la forme
- Proposer des solutions
- Consulter et arbitrer avec l'équipe
Comment ca marche ?
ATTENTION : Avant de demander une validation par l'équipe, le développeur doit lire son code.
Comment ca marche ?
Les biens faits de la code review
By Hando Masahashi