Human talks - Ratez vos revues de code en 5 leçons

www.octo.com - www.usievents.com 1@mdomenjoud #AgileTourLille #CodeReview
RATEZ RÉUSSISSEZ VOS REVUES
DE CODE EN 5 LEÇONS
Michel DOMENJOUD
Agile Tour Lille – 15 octobre 2015

Développeur, in love with code since 2003.
Tech Lead – Coach Craft chez Octo Technology
@mdomenjoud #BetterPlacesWithBetterCode
MICHEL DOMENJOUD

…la revue de code !
S’il n’y avait qu’une seule chose à garder…

Efficace depuis la nuit des temps
Un objectif : Détecter les défauts le plus tôt
possible
Jusqu’à 65% des défauts détectés
ROI de 4 pour 1
1h de revue → 4h de debug économisées
Des bénéfices conséquents :
Partager des standards
Renforcer la propriété collective du code
Faciliter l’apprentissage
Améliorer la communication autour du code
http://sail.cs.queensu.ca/publications/pubs/msr2014-mcintosh.pdf
http://www.reviewtechnik.de/NationalSoftwareQualityExperiment.pdf

Revue par un pair
Pair programming
Revue collective

Fail #1 : On croule toujours sous les bugs !

Allez le plus vite
possible !
Allez-y les mains
dans les poches !
Ne demandez à
personne comment
s’y prendre !
- Prenez le temps, à un rythme soutenable
- Demandez une revue dès que possible
- Préparez la revue via une lecture à tête
reposée
- Utilisez des standards et checklists
partagés
- Formez vous ensemble
Pour passer à côté
des défauts Et pour les détecter ?

Fail #2 Mon équipe ne progresse pas

Ne vous parlez
surtout pas !
Corrigez tout à la
place de l’auteur !
N’écrivez pas vos
standards
- Échangez de vive voix pendant les revues
- L’auteur corrige, en binôme si nécessaire
- Ecrivez vos standards
- Faites les évoluer au fil des revues
Pour stagner au fil des
revues Et pour progresser ?

Fail #3 : On se tape dessus

Critiquez en priorité
les personnes !
Lancez toujours
une language /
framework war !
- Critiquez le code, pas le développeur
- Appuyez vous sur les principes Egoless
Programming
- C’est pas le moment
- Trouvez des compromis
- Choisissez un standard et écrivez-le
Pour être sûr de
s’engueuler Et pour une revue sereine ?

Trois formats complémentaires
Revue par un pair
Revue
collective
Pair
programming
Démarrer une User Story,
Code complexe,
Code Legacy, Debug,
Apprendre
Etablir des standards
Code critique
Nouvelles technos
Tout le tempsRégulièrement A la demande

5 leçons à
retenir
Leçon n°1 - Prenez le temps
Préparation à tête reposée
Rythme soutenable (max 1h30 & 300 LoC à la fois)
Leçon n°2 - Progressez ensemble
Échangez de vive voix pendant les revues
L’auteur corrige, en binôme si nécessaire
Leçon n°3 - Utilisez des standards
Utilisez des standards et checklists partagés
Faites les évoluer au fil des revues
Leçon n°4 - Critiquez le code, pas le dév.
Principes Egoless Programming
Leçon n°5 - Trouvez le(s) formats qui vous conviennent
et adaptez les à vos besoins

Pour aller plus loin
http://blog.octo.com/revue-de-code-quel-format-choisir/
Série d’articles sur la revue de code
Better Places With Better Code

recrutement@octo.com
Vous croyez que les technologies changent le monde ?
Nous aussi ! Rejoignez-nous, à Lille aussi !

Human talks - Ratez vos revues de code en 5 leçons

Recommended

Recommended

More Related Content

Viewers also liked

Viewers also liked (20)

Similar to Human talks - Ratez vos revues de code en 5 leçons

Similar to Human talks - Ratez vos revues de code en 5 leçons (20)

Human talks - Ratez vos revues de code en 5 leçons