Les revues de code ou                 comment faire fructifier                 son capital humainForum PHP 2010
Qui suis-je ?   Jean-Marc Fontaine   Consultant pour Alter Way Consulting   Responsable du centre de compétences PHP po...
Livre blanc « Industrialisation PHP »   Co-écrit avec Damien Seguy   Panorama des outils et    méthodes dindustrialisati...
Lerreur est humaineLes développeurs font des erreurs, cest inévitable.La vraie question est quand souhaite-t-on les découv...
Revues de code : kezako ?   Une revue de code consiste à examiner le code de    quelquun dautre à la recherche de défauts...
Pas limitées au code source   Les revues ne sont pas limitées au code, on peut les    appliquer également aux documents i...
Il ny a pas des outils danalyse pour ça ?   Complémentaires   Adaptés aux problèmes de syntaxe et doptimisation    subti...
Et les tests ?   Les tests automatisés (unitaires, fonctionnels, etc.) et    manuels nindiquent rien de la qualité et de ...
Buts   Amélioration de la qualité du code         Découverte de bogues         Qualité de la conception         Applic...
Bénéfices indirects   Partage de la connaissance   Formation des juniors   Recherche de consensus   Amélioration de la...
Objections habituelles   Coût   Perte de temps   Freins humains         Fainéantise         Ego         Conflits per...
Cest un truc expérimental ?!   Les revues de code sont pratiquées par tous les acteurs    importants         Google     ...
Les types de revues de codee                                                                                     ff       ...
Organiser une revue de code Forum PHP 2010   Les revues de code ou comment faire fructifier son capital humain   14
Quand effectuer une revue de code ? Forum PHP 2010   Les revues de code ou comment faire fructifier son capital humain   15
Planifier une revue de code   Entre 3 et 7 personnes   Rôles         Auteur         Inspecteur         Modérateur    ...
Sélectionner le code à étudier   Systématique   À la demande du développeur   Parties problématique de lapplication   ...
Préparer une revue   Réunion de présentation         Informelle         Présente rapidement le code qui sera       revu...
Déroulement dune réunion de revue Forum PHP 2010   Les revues de code ou comment faire fructifier son capital humain   19
Après la revue de code   Lauteur modifie son code en fonction du journal des    défauts   Le modérateur rédige un compte...
Évaluer le fruit des revues de code Forum PHP 2010   Les revues de code ou comment faire fructifier son capital humain   21
Évaluer le fruit des revues de code   Lefficacité des revues doit être évalué par rapport au    temps gagné et pas au nom...
Mesures de base Forum PHP 2010   Les revues de code ou comment faire fructifier son capital humain   23
Mesures avancées Forum PHP 2010   Les revues de code ou comment faire fructifier son capital humain   24
Outiller ses revues de code Forum PHP 2010   Les revues de code ou comment faire fructifier son capital humain   25
Outiller ses revues de code   Outils Open Source         Review Board         Rietveld   Outils commerciaux         S...
10 bonnes pratiques   Ne pas étudier plus de 300 lignes à la fois   Adopter un rythme de 300 à 500 lignes étudiées par  ...
Merci !   Me contacter :    jean-marc.fontaine@alterway.fr   Donnez votre avis sur cette    conférence :    http://joind...
Prochain SlideShare
Chargement dans…5
×

Revues de code forum php

1 869 vues

Publié le

Chaque industrie possède un élément clé dans son modèle économique. Dans l'industrie du développement, le facteur de succès est sans conteste le capital humain. Savoir recruter les meilleurs développeurs est une chose difficile mais les amener à réaliser leur plein potentiel l'est tout autant.
En ouvrant le code à d'autres développeurs, les revues de code permettent de rompre l'isolement et de partager les connaissances afin de créer des émulations positives au sein des équipes.
Nous verrons les gains qu'on peut attendre de cette pratique, les différentes formes (formelles, itératives, pair programming, etc.) qu'elle peut prendre ainsi que les écueils à éviter pour en tirer pleinement parti.

0 commentaire
0 j’aime
Statistiques
Remarques
  • Soyez le premier à commenter

  • Soyez le premier à aimer ceci

Aucun téléchargement
Vues
Nombre de vues
1 869
Sur SlideShare
0
Issues des intégrations
0
Intégrations
11
Actions
Partages
0
Téléchargements
7
Commentaires
0
J’aime
0
Intégrations 0
Aucune incorporation

Aucune remarque pour cette diapositive

Revues de code forum php

  1. 1. Les revues de code ou comment faire fructifier son capital humainForum PHP 2010
  2. 2. Qui suis-je ? Jean-Marc Fontaine Consultant pour Alter Way Consulting Responsable du centre de compétences PHP pour Alter Way Solutions Formateur / Professeur vacataire en université Membre Actif de lAFUP Auteur du blog Industrialisation-PHP.com Co-auteur du livre blanc « Industrialisation PHP » Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 2
  3. 3. Livre blanc « Industrialisation PHP » Co-écrit avec Damien Seguy Panorama des outils et méthodes dindustrialisation des développements PHP Publié en octobre 2009 2 500 téléchargementsTéléchargement : http://www.alterway.fr/publications/livre-blanc-industrialisation-phpBlog : http://www.industrialisation-php.com/ Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 3
  4. 4. Lerreur est humaineLes développeurs font des erreurs, cest inévitable.La vraie question est quand souhaite-t-on les découvrir ?En développement, en recette ou en production ? Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 4
  5. 5. Revues de code : kezako ? Une revue de code consiste à examiner le code de quelquun dautre à la recherche de défauts ou daméliorations potentielles La vérification (contrôle de la qualité pendant le développement) est plus efficace, et donc moins coûteuse, que la validation (contrôle de la qualité après le développement). Proximité avec les méthodes agiles (Pair programming, responsabilité collective, courage) Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 5
  6. 6. Pas limitées au code source Les revues ne sont pas limitées au code, on peut les appliquer également aux documents importants dun projet comme les spécifications, les documents de conception, etc. Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 6
  7. 7. Il ny a pas des outils danalyse pour ça ? Complémentaires Adaptés aux problèmes de syntaxe et doptimisation subtile Pas adaptés aux problèmes fonctionnels ou de logique Un humain peut identifier : – les messages derreur peu clairs – les commentaires inappropriés – les valeurs en dur (Nombres magiques) – les manques de factorisation Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 7
  8. 8. Et les tests ? Les tests automatisés (unitaires, fonctionnels, etc.) et manuels nindiquent rien de la qualité et de la maintenabilité du code. Les tests trouvent les symptômes tandis que les revues de code trouvent les causes des problèmes. Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 8
  9. 9. Buts Amélioration de la qualité du code  Découverte de bogues  Qualité de la conception  Application des bonnes pratiques  Respect des conventions  Suppression des redondances  Élimination des incohérences Vérification de la conformité Vérification de lexhaustivité Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 9
  10. 10. Bénéfices indirects Partage de la connaissance Formation des juniors Recherche de consensus Amélioration de la maîtrise collective du code Émergence didées neuves Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 10
  11. 11. Objections habituelles Coût Perte de temps Freins humains  Fainéantise  Ego  Conflits personnels Difficultés dorganisation Méthode non exhaustive Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 11
  12. 12. Cest un truc expérimental ?! Les revues de code sont pratiquées par tous les acteurs importants  Google  Cisco  Intel  NASA  BNP Paribas  UPS Chez Google rien nest commité sans être revu au préalable. Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 12
  13. 13. Les types de revues de codee ff of ri mc aa lic si mt eé Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 13
  14. 14. Organiser une revue de code Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 14
  15. 15. Quand effectuer une revue de code ? Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 15
  16. 16. Planifier une revue de code Entre 3 et 7 personnes Rôles  Auteur  Inspecteur  Modérateur  Lecteur  Secrétaire  Vérificateur Les rôles sont cumulables sauf pour lauteur qui ne peut éventuellement être que lecteur Choisir un endroit calme Léquipe doit rester isolée durant toute la revue Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 16
  17. 17. Sélectionner le code à étudier Systématique À la demande du développeur Parties problématique de lapplication Couverture de code Expérience Hasard Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 17
  18. 18. Préparer une revue Réunion de présentation  Informelle  Présente rapidement le code qui sera revu et les objectifs de la revue de code  Peut être supprimée si léquipe est familière du code Définition des règles, standards et spécifications en vigueur Le lecteur doit se familiariser avec le code Les inspecteurs doivent étudier le code à la recherche de problèmes et dopportunités doptimisations Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 18
  19. 19. Déroulement dune réunion de revue Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 19
  20. 20. Après la revue de code Lauteur modifie son code en fonction du journal des défauts Le modérateur rédige un compte-rendu de revue Le vérificateur sassure que le code a été retravaillé comme convenu Livrables  Code retravaillé  Journal des défauts  Compte-rendu de revue Possibilité de générer des statistiques avec le temps Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 20
  21. 21. Évaluer le fruit des revues de code Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 21
  22. 22. Évaluer le fruit des revues de code Lefficacité des revues doit être évalué par rapport au temps gagné et pas au nombre de défauts trouvés. Éviter les mesures individuelles Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 22
  23. 23. Mesures de base Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 23
  24. 24. Mesures avancées Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 24
  25. 25. Outiller ses revues de code Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 25
  26. 26. Outiller ses revues de code Outils Open Source  Review Board  Rietveld Outils commerciaux  SmartBear Code Collaborator  Atlassian Crucible Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 26
  27. 27. 10 bonnes pratiques Ne pas étudier plus de 300 lignes à la fois Adopter un rythme de 300 à 500 lignes étudiées par heure Ne pas dépasser 90 minutes pour une revue Les inspecteurs doivent étudier le code avant la réunion de revue Établir des objectifs quantifiables et recueillir des mesures Utiliser des checklists Vérifier que les problèmes trouvés sont effectivement corrigés Développer la culture de la revue de code Jouer sur la pression sociale Éviter le sentiment de surveillance Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 27
  28. 28. Merci ! Me contacter : jean-marc.fontaine@alterway.fr Donnez votre avis sur cette conférence : http://joind.in/2075 Me suivre sur Twitter : (@jmfontaine) Des questions ?Présentation : http://www.slideshare.net/jmf/Livre blanc : http://www.alterway.fr/publications/livre-blanc-industrialisation-phpBlog : http://www.industrialisation-php.com/ Forum PHP 2010 Les revues de code ou comment faire fructifier son capital humain 28

×