0️⃣ Quelle est la procédure standard pour améliorer notre standard ?

Relire le code ensemble, afin de trouver des défauts 👀
La personne qui a écrit le code explique son fonctionnement💡
L’équipe identifie des défauts 👆
Une personne prend des notes 📝
Une autre surveille le temps ⏳
À la fin de la session l’équipe décide du sort du code :

  • OK : le code va en production ✅
  • OK si les améliorations … sont apportées 🚦
  • KO : à revoir avant d’aller en production ❌

1️⃣ On peut trouver 1000 défauts à un code : lesquels sont les plus importants et par où commencer ?

Les défauts les plus importants vont émerger en priorité durant la relecture. 🧩
Commencer par le code qui présente le plus de problèmes en production ou le code particulièrement difficile à faire évoluer 🔥

2️⃣ Tout est relatif en qualité logicielle : comment se mettre d’accord sur ce qui est un défaut ou non ?

Comme avec tout sujet de désaccord : en en parlant et en prenant des décisions. C’est l’objet de la relecture de code. 🗺

3️⃣ Quel genre de problème peut on trouver ?

  • standard pas respecté : le code doit être mis au standard ❎
  • absence de standard : le standard doit être amélioré pour résoudre ce problème ➕
  • cas de conception : ce problème spécifique doit être résolu dans une autre session de conception 📐

4️⃣ Comment peut-on dire “ce n’est pas standard” si on n’a pas de standard ?

Si nous avons un produit qui tient à peu près debout en production, alors nous avons un standard, même minimal. Le tout est de ne pas le laisser se dégrader. 🕸

5️⃣ Trouver des défauts c’est bien gentil, mais si on pas le temps de les corriger ?

Si nous prenons le temps de corriger les défauts, nous allons améliorer la qualité de notre application, et donc perdre moins de temps sur des problèmes en production. ♻️

Identifier des défauts que nous ne corrigerons pas revient à organiser la perte de temps 💸

6️⃣ Comment éviter la violence des critiques pendant la revue ?

En respectant les personnes. On peut être critique avec un produit sans être critique avec la personne qui crée ce produit 🙄
On peut également suggérer des améliorations au lieu d’énoncer des critiques 🪜

7️⃣ Comment tu vas éviter les discussions à l’infini ?

Via le process cf 0️⃣

8️⃣ Pourquoi est-ce que ça marche ?

La relecture de code

  • trouve des défauts 🔦
  • améliore le standard 🗺
  • améliore la communication à propos du standard 🤝
  • améliore la cohérence de l’équipe 💫

9️⃣ Et tu crois que c’est ça qui va sauver le projet ?

Je préfèrerais travailler sur un projet qui n’a pas besoin d’être “sauvé” 🚒
Mieux vaut prévenir que guérir, mais si vous vous en sortez mieux pour moins cher en corrigeant la prod’, ne faites pas de prévention 🤷‍♂️

🔟Quel risque comporte cette stratégie ?

La stratégie de prévention élimine les problèmes. En l’absence de problème, les décideurs décident d’éliminer la stratégie de prévention. 🤯

publié sur Linked In le 24/05/2023