Votre boss doute de la pertinence des revues de code ? Vous avez essayé mais ça n'a pas marché ? Joffrey et Nicolas vous donneront les clés pour comprendre comment conduire des revues de codes efficaces et pertinentes. Ils parleront de leurs expériences au sein de leurs équipes ainsi que des pièges à éviter. Si les revues de code attisent votre curiosité, cette conférence est faite pour vous ! 

1. Les Code Reviews
Le guide de survie

2. – « Uncle » Bob Martin (The Future Of Programming)
“Nous sommes dangereux, nous avons tué des
personnes.”
NORAD
Therac-25
Chinook helicopter crashed
Mars Climate Orbiter

3. @Jo_Kern @nverinaud
Joffrey KERN Nicolas VERINAUD

4. Qui fait des revues de
code en ce moment ?

5. Why the fuck revoir le
code ?
L’erreur est humaine
Les tests unitaires ne suffisent pas
Déployer avec plus de sérénité

7. Collective Code
Ownership

8. Détecter les erreurs quant
elles coûtent le moins
cher.

9. Détecter les erreurs quand
elles coûtent le moins
cher.

10. Discipline

11. La revue de code dans
ma boîte

12. Quoi ? Deux personnes qui
travaillent sur le même code ?
Mais, c’est une perte de temps !

13. Qu’attends le business ?
Logiciel sans bugs
Money qui rentre (ROI) 🤑
Satisfaction des utilisateurs
Bonne image du produit

14. Éviter les catastrophes
Limiter les risques

15. Signes que le business
s’est engagé pour les
revues.
Accorde du temps
Maintient la pratique à l’approche des deadlines
Encourage la pratique
Récompense les early adopters
Et 7 autres: voir le livre de Karl E. Wiegers

16. Comment je fais,
concrètement ?

17. Culture de la qualité

18. Former l’équipe ET le
management

19. Les Outils
• Gestionnaire de code source
• Portail de feedbacks (gitlab, bitbucket, etc.)
• Guidelines
• Serveur d’intégration continue (linter…)

20. Démo Fizz Buzz Story
En tant qu'utilisateur, je demande la liste des FizzBuzz
entre deux nombres afin de proposer des posters
personnalisés aux développeurs.

21. Fizz Buzz ?
Si un nombre est divisible par 3, "Fizz" doit être
retourné.
Si un nombre est divisible par 5, "Buzz" doit être
retourné.
Si un nombre est divisible par 3 et par 5, "FizzBuzz"
doit être retourné.
Sinon, le nombre doit être retourné.

22. Critères d'acceptance
Lorsque j'appelle la fonction,
en donnant les nombres 0 et 15,
je reçois la liste [ 0, 1, 2, Fizz, 4, Buzz, 6, 7, 8, Fizz,
Buzz, 11, Fizz, 13, 14, FizzBuzz ].

23. Guildelines
• Indenter correctement et respecter les guidelines
Microsoft pour le code style C#
• Ecrire des tests unitaires pour le code métier
• Le code métier doit être indépendant des
technologies de stockage et d’affichage
• Préfixer les classes avec Licorne 🦄

24. Let’s go !

25. Les pièges à éviter

26. Personne n’a à me faire des
remarques car je suis le meilleur
de la team. #bogoss 😎
— Le dev senior qui n'a plus rien à apprendre (enfin c’est ce qu’il croit)

27. Je me fais allumer à chaque
code review, je ne comprends
pas pourquoi, c'est très frustrant
d'être le plus nul de la team. 😭
#NullDevException
— Le dev junior

28. Pas besoin de réfléchir, c’est le
job du relecteur de trouver mes
bugs. #jailaflemme
— Le flemmard

29. Ça fait 3 mois que je travaille
sur ce code, il est bientôt parfait
pour la relecture.
#wowsuchperfectcode
— Le perfectionniste

30. Revues de quoi ? Je déploie et
puis c’est tout. C’est qui le chef
ici ? #dictature
— Le chef qui s’octroie un traitement de faveur

31. Que les meilleurs relisent, il faut
une promotion pour devenir
relecteur. #topOfThePyramid
— Une hierarchie verticale sauvage apparaît

32. J’ai envoyé mon code en
review, viiiiiiiiiiiiiiite je vais
embêter le relecteur. #deadline
— Le (p|st)ressé

33. Trop bien les revues de code.
Je vais les utiliser pour juger les
devs. 😈 #KPImania
— Le manager

34. Avec les revues de code, tous
nos problèmes disparaîtront !
#silverbullet
— Peut-être vous ?

35. Thanks ! 👊
@Jo_Kern @nverinaud

36. Bibliographie
• Code Reviews: Just Do It ! — Jeff Atwood
• Crafting better code reviews — Vaidehi Joshi
• Peer Reviews in Software: Practical Guide — Karl E.
Wiegers (free chapters)
• The Future Of Programming — Robert C. Martin
• 10 historical software bugs with extreme
consequences — Pingdom Tech Blog