Les bienfaits de la 

CodeReview

LA SIMPLICITÉ EST LA SOPHISTICATION SUPRÊME

Léonard de Vinci

  1. Intro : Mise en situation

  2. C'est quoi une code review ?

    • Quoi ?
    • ​Qui ?
    • Quand ?
    • Comment ?

  3. Les avantages

  4. Chronophage ?

  5. Les vrais obstacles

  6. Comment ça marche ?

Mise en situation

Mise en situation

Mise en situation

Aaaah mes yeux !! ça pique ...

Mise en situation

Mise en situation

Quelles sont les conséquences d'un tel code ?

  • L'application bascule vers du code spaghetti 
  • La dette technique augmente
  • Besoin de refacto
  • Difficulté pour les nouveaux membres du projet
  • Frustration de l'équipe

C'est quoi une code review ?

La revue de code est un examen systématique du code source d'un logiciel. 

Il peut être comparé au processus ayant lieu dans un comité de lecture, l'objectif étant de trouver des bugs ou des vulnérabilités potentielles ou de corriger des erreurs de conception afin d'améliorer la qualité, la maintenabilité et la sécurité du logiciel.

 

Qui relit le code ?

Au mieux :

L'ensemble des devs d'une équipe

A minima :

Un dev qui a une connaissance du projet

Quand faire une code review ?

Au mieux :

A minima :

Qu'est ce qu'on regarde dans une code review ?

  • Respect des normes PSR
  • Respect des bonnes pratiques
    • KISS,
    • DRY,
    • Fail First,
  • Modélisation
  • Cohérence des tests

Chronophage ?

Qui pense que c'est chronophage ?

Le problème, c'est que la vérité est ailleurs

Les vrais obstables

  • Besoin de formation
  • Peur d'être juger
  • Rester dans sa zone de confort 
  • Susceptibilité
  • Pas de remise en question du dev
  • Le relecteur est un Waffen SS

Les avantages

  1. Maintenir ou améliorer la qualité du code
  2. L'appropriation du code par l’équipe
  3. Veiller aux respects des bonnes pratiques
  4. Détecter les erreurs d'implémentations
  5. Formation des développeurs
  6. Challenger les choix techniques
  7. Un R.O.I sur la qualité conséquent
  8. Diminuer la complexité cyclomatique (CRAP)
  9. Cohésion d'équipe

Le relecteur

  1. Valider du code, c'est approuvé et avoir confiance au code implémenté
  2. Doit être bienveillant
  3. S'assurer que c'est efficace et optimisé
  4. Se concentrer sur le fond plutot que la forme
  5. Proposer des solutions
  6. Consulter et arbitrer avec l'équipe

Comment ca marche ?

ATTENTION : Avant de demander une validation par l'équipe, le développeur doit lire son code.

Comment ca marche ?

Made with Slides.com