Code Reviews Devoxx France 2012
•
1 j'aime•1,996 vues
Code Reviews from 97 Things Every Programmar Should Know for Devoxx France 2012
Recommandé
Contenu connexe
Similaire à Code Reviews Devoxx France 2012
Similaire à Code Reviews Devoxx France 2012 (20)
Dernier
Dernier (15)
Code Reviews Devoxx France 2012
- 1. Code Reviews One Thing Every Programmer Should Know by Mattias Karlsson Java Champion, JUG Leader & Agile coach Avega Group Sweden ©Copyright 2012
- 2. Disclaimer My own experience Ma propre expérience I am probably wrong Je me trompe probablement ©Copyright 2012
- 3. Bio Mattias Karlsson Java Champion, JUG Leader & Agile coach Avega Group Stockholm Sweden @matkar LinkedIn.com/in/mattiask ©Copyright 2012
- 5. Jfokus Jfokus & Parleys ©Copyright 2012
- 6. Code Reviews Revue de code One Thing Every Programmer Should Know Une Chose que Chaque Programmeur Devrait Savoir ©Copyright 2012
- 8. Why? Increase Quality (Améliorer la qualité) Reduce Defects (Réduire les défauts) Increase Team Spirit? (Améliorer l'esprit d'équipe?) ©Copyright 2012
- 9. Anti-pattern Architects Review Everything Les Architectes Verifient Tout ©Copyright 2012 Photo by dpstyles
- 10. Pair Programming? Photo Kniberg H by dpstyles ©Copyright 2012
- 11. Goal Sharing knowledge Le partage des connaissances ©Copyright 2012
- 12. Code Reviews Code Review Meeting (Réunion de revue de Code) 1-2 hour/week (1-2 heures / semaine) Code Review Day (Journee de revue de Code) Informal brown bag style (Informelle, “Brown Bag”) ©Copyright 2012
- 14. Preparation Code Conventions (conventions de code) checked by tools (vérifiées par les outils) Tested (Testé) Definition of DONE (Définition de FAIT) ©Copyright 2012
- 15. Code Review meeting Code Walkthrough (Parcours du code pas à pas) Introduce Review Roles (Revue des rôles) Have a Moderator (Avoir un Modérateur) Take notes (Prendre des notes) Involve Experts (Impliquer les Experts) Involve Newbie’s (Impliquer les Juniors) ©Copyright 2012
- 16. Mood Learning Mood (Etre en situation d’apprentissage) Open and Respectful (Ouvert et respectueux) Collaborative (Collaboratif) Constructive (Constructive) Don´t be sarcastic! (Ne pas être sarcastique!) Ask Questions (Posez des questions) ©Copyright 2012
- 17. Things to watch out for Ego Effect / Hurt Feelings (Effet Ego) “I have to contribute..” (Je dois contribuer) Don´t Point Out Spelling Errors (Ne faites pas remarquer les fautes d'orthographe) It´s all about sharing the code (Tout est une question de partage du code) ©Copyright 2012
- 18. Have Fun Amusez-vous ©Copyright 2012
- 20. ©Copyright 2012
- 21. Mattias Karlsson linkedin.com/in/mattiask twitter: @matkar Thanks! mattias.karlsson@avegagroup.se Feedback: #CodeReviews Réaction: #DevoxxFR ©Copyright 2012
Notes de l'éditeur
- And about measurement... I can only agre with this.. Numbers of WTF/min is a a good indicator and it will hopefully be less of the if you start with code review.
- And about measurement... I can only agre with this.. Numbers of WTF/min is a a good indicator and it will hopefully be less of the if you start with code review.
- Thanks (and buy the book It has a lot more tips and trix than this)