Ce diaporama a bien été signalé.
Nous utilisons votre profil LinkedIn et vos données d’activité pour vous proposer des publicités personnalisées et pertinentes. Vous pouvez changer vos préférences de publicités à tout moment.

Les Code Reviews : le guide de survie

200 vues

Publié le

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 ! 

Publié dans : Technologie
  • Soyez le premier à commenter

  • Soyez le premier à aimer ceci

Les Code Reviews : le guide de survie

  1. 1. Les Code Reviews Le guide de survie
  2. 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. 3. @Jo_Kern @nverinaud Joffrey KERN Nicolas VERINAUD
  4. 4. Qui fait des revues de code en ce moment ?
  5. 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é
  6. 6. Collective Code Ownership
  7. 7. Détecter les erreurs quant elles coûtent le moins cher.
  8. 8. Détecter les erreurs quand elles coûtent le moins cher.
  9. 9. Discipline
  10. 10. La revue de code dans ma boîte
  11. 11. Quoi ? Deux personnes qui travaillent sur le même code ? Mais, c’est une perte de temps !
  12. 12. Qu’attends le business ? Logiciel sans bugs Money qui rentre (ROI) 🤑 Satisfaction des utilisateurs Bonne image du produit
  13. 13. Éviter les catastrophes Limiter les risques
  14. 14. 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
  15. 15. Comment je fais, concrètement ?
  16. 16. Culture de la qualité
  17. 17. Former l’équipe ET le management
  18. 18. Les Outils • Gestionnaire de code source • Portail de feedbacks (gitlab, bitbucket, etc.) • Guidelines • Serveur d’intégration continue (linter…)
  19. 19. 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.
  20. 20. 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é.
  21. 21. 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 ].
  22. 22. 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 🦄
  23. 23. Let’s go !
  24. 24. Les pièges à éviter
  25. 25. 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)
  26. 26. 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
  27. 27. Pas besoin de réfléchir, c’est le job du relecteur de trouver mes bugs. #jailaflemme — Le flemmard
  28. 28. Ça fait 3 mois que je travaille sur ce code, il est bientôt parfait pour la relecture. #wowsuchperfectcode — Le perfectionniste
  29. 29. 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
  30. 30. Que les meilleurs relisent, il faut une promotion pour devenir relecteur. #topOfThePyramid — Une hierarchie verticale sauvage apparaît
  31. 31. J’ai envoyé mon code en review, viiiiiiiiiiiiiiite je vais embêter le relecteur. #deadline — Le (p|st)ressé
  32. 32. Trop bien les revues de code. Je vais les utiliser pour juger les devs. 😈 #KPImania — Le manager
  33. 33. Avec les revues de code, tous nos problèmes disparaîtront ! #silverbullet — Peut-être vous ?
  34. 34. Thanks ! 👊 @Jo_Kern @nverinaud
  35. 35. 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

×