Ratez vos revue de code en 5 lecons devoxx fr 2016

#DevoxxFR #CultureCode @mdomenjoud

Ratez vos revues de code

en 5 leçons

Michel Domenjoud @mdomenjoud

1


Développeur, in love with code since 2003.

Tech Lead – Coach Craft @OctoTechnology

@mdomenjoud #CultureCode #BetterPlaces #BetterCode

MICHEL DOMENJOUD


La première pratique d’équipe à mettre en place

3

… la revue de code !


Software Craftsmanship : pratiques indispensables

4

Clean CodeTests

automatisésTest Driven

Development

Feedback

Revue de code

+ Intégration continue, Standards, Refactoring, Design, etc.


Efficace depuis la nuit des temps


• 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

The Impact of Code Review Coverage and Code Review Participation on Software Quality

NATIONAL SOFTWARE QUALITY EXPERIMENT - A LESSON IN MEASUREMENT

http://sail.cs.queensu.ca/Downloads/MSR2014_TheImpactOfCodeReviewCoverageAndCodeReviewParticipationOnSoftwareQuality_ACaseStudyOfTheQt,VTK,AndITKProjects.pdf

http://www.reviewtechnik.de/NationalSoftwareQualityExperiment.pdf


Pair programming

Revue par un pair

Revue collective

7


Fail #1 : On croule toujours sous les bugs !

http://thelinkedinman.com/wp-content/uploads/sites/2/2013/10/surprised-man.jpg


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

http://taxiblog.fr/wp-content/uploads/2013/01/crossing_the_road.jpg


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

http://i.huffpost.com/gen/1789693/images/o-DISPUTE-FAMILIALE-facebook.jpg


Critiquez en priorité les personnes !

Lancez toujours une language / framework war !

- Critiquez le code, pas le développeur- Appuyez vous sur les principes EgolessProgramming

- Ce n’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 ?


Revue par un pair

Revue collective

Pair programming

Démarrer une User Story,Code complexe,

Code Legacy, Debug,Apprendre

Etablir des standardsCode critique

Nouvelles technos

Tout le tempsRégulièrement A la demande

Trois formats complémentaires


• 1 - Prenez le temps

•Préparation à tête reposée

•Rythme soutenable (max 1h30 & 300 LoC à la fois)

• 2 - Progressez ensemble

•Échangez de vive voix pendant les revues

•L’auteur corrige, en binôme si nécessaire

• 3 - Utilisez des standards

•Utilisez des standards et checklists partagés

•Faites les évoluer au fil des revues

• 4 - Critiquez le code, pas le dév.

•Principes Egoless Programming

• 5 - Trouvez le(s) formats qui vous conviennent et adaptez les à vos besoins

5 leçons à retenir pour réussir vos revues


• Un livre collectif sur la culture et les pratiques Software Craftsmanship

• Disponible gratuitement au téléchargement http://www.octo.com/fr/publications/20-culture-code

Culture CodeSoftware Craftsmanship : Better Places with Better Code

http://blog.octo.com/tag/software-craftsmanship

http://www.octo.com/fr/publications/20-culture-code

http://blog.octo.com/tag/software-craftsmanship

Ratez vos revue de code en 5 lecons devoxx fr 2016

Software

Transcript of Ratez vos revue de code en 5 lecons devoxx fr 2016