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
– « Uncle » Bob Martin (The Future Of Programming)
“Nous sommes dangereux, nous avons tué des
personnes.”
NORAD
Therac-25
...
@Jo_Kern @nverinaud
Joffrey KERN Nicolas VERINAUD
Qui fait des revues de
code en ce moment ?
Why the fuck revoir le
code ?
L’erreur est humaine
Les tests unitaires ne suffisent pas
Déployer avec plus de sérénité
Collective Code
Ownership
Détecter les erreurs quant
elles coûtent le moins
cher.
Détecter les erreurs quand
elles coûtent le moins
cher.
Discipline
La revue de code dans
ma boîte
Quoi ? Deux personnes qui
travaillent sur le même code ?
Mais, c’est une perte de temps !
Qu’attends le business ?
Logiciel sans bugs
Money qui rentre (ROI) 🤑
Satisfaction des utilisateurs
Bonne image du produit
Éviter les catastrophes
Limiter les risques
Signes que le business
s’est engagé pour les
revues.
Accorde du temps
Maintient la pratique à l’approche des deadlines
Enc...
Comment je fais,
concrètement ?
Culture de la qualité
Former l’équipe ET le
management
Les Outils
• Gestionnaire de code source
• Portail de feedbacks (gitlab, bitbucket, etc.)
• Guidelines
• Serveur d’intégra...
Démo Fizz Buzz Story
En tant qu'utilisateur, je demande la liste des FizzBuzz
entre deux nombres afin de proposer des poste...
Fizz Buzz ?
Si un nombre est divisible par 3, "Fizz" doit être
retourné.
Si un nombre est divisible par 5, "Buzz" doit êtr...
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...
Guildelines
• Indenter correctement et respecter les guidelines
Microsoft pour le code style C#
• Ecrire des tests unitair...
Let’s go !
Les pièges à éviter
Personne n’a à me faire des
remarques car je suis le meilleur
de la team. #bogoss 😎
— Le dev senior qui n'a plus rien à ap...
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....
Pas besoin de réfléchir, c’est le
job du relecteur de trouver mes
bugs. #jailaflemme
— Le flemmard
Ça fait 3 mois que je travaille
sur ce code, il est bientôt parfait
pour la relecture.
#wowsuchperfectcode
— Le perfection...
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 d...
Que les meilleurs relisent, il faut
une promotion pour devenir
relecteur. #topOfThePyramid
— Une hierarchie verticale sauv...
J’ai envoyé mon code en
review, viiiiiiiiiiiiiiite je vais
embêter le relecteur. #deadline
— Le (p|st)ressé
Trop bien les revues de code.
Je vais les utiliser pour juger les
devs. 😈 #KPImania
— Le manager
Avec les revues de code, tous
nos problèmes disparaîtront !
#silverbullet
— Peut-être vous ?
Thanks ! 👊
@Jo_Kern @nverinaud
Bibliographie
• Code Reviews: Just Do It ! — Jeff Atwood
• Crafting better code reviews — Vaidehi Joshi
• Peer Reviews in ...
Les Code Reviews : le guide de survie
Prochain SlideShare
Chargement dans…5
×

sur

Les Code Reviews : le guide de survie Slide 1 Les Code Reviews : le guide de survie Slide 2 Les Code Reviews : le guide de survie Slide 3 Les Code Reviews : le guide de survie Slide 4 Les Code Reviews : le guide de survie Slide 5 Les Code Reviews : le guide de survie Slide 6 Les Code Reviews : le guide de survie Slide 7 Les Code Reviews : le guide de survie Slide 8 Les Code Reviews : le guide de survie Slide 9 Les Code Reviews : le guide de survie Slide 10 Les Code Reviews : le guide de survie Slide 11 Les Code Reviews : le guide de survie Slide 12 Les Code Reviews : le guide de survie Slide 13 Les Code Reviews : le guide de survie Slide 14 Les Code Reviews : le guide de survie Slide 15 Les Code Reviews : le guide de survie Slide 16 Les Code Reviews : le guide de survie Slide 17 Les Code Reviews : le guide de survie Slide 18 Les Code Reviews : le guide de survie Slide 19 Les Code Reviews : le guide de survie Slide 20 Les Code Reviews : le guide de survie Slide 21 Les Code Reviews : le guide de survie Slide 22 Les Code Reviews : le guide de survie Slide 23 Les Code Reviews : le guide de survie Slide 24 Les Code Reviews : le guide de survie Slide 25 Les Code Reviews : le guide de survie Slide 26 Les Code Reviews : le guide de survie Slide 27 Les Code Reviews : le guide de survie Slide 28 Les Code Reviews : le guide de survie Slide 29 Les Code Reviews : le guide de survie Slide 30 Les Code Reviews : le guide de survie Slide 31 Les Code Reviews : le guide de survie Slide 32 Les Code Reviews : le guide de survie Slide 33 Les Code Reviews : le guide de survie Slide 34 Les Code Reviews : le guide de survie Slide 35 Les Code Reviews : le guide de survie Slide 36
Prochain SlideShare
What to Upload to SlideShare
Suivant
Télécharger pour lire hors ligne et voir en mode plein écran

0 j’aime

Partager

Télécharger pour lire hors ligne

Les Code Reviews : le guide de survie

Télécharger pour lire hors ligne

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

  • 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

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

Vues

Nombre de vues

265

Sur Slideshare

0

À partir des intégrations

0

Nombre d'intégrations

30

Actions

Téléchargements

1

Partages

0

Commentaires

0

Mentions J'aime

0

×