How to properly do a code review or do pair programming. Also what NOT to do.

1. Code Reviews &
Pair Programming_
Erasmus Universiteit
03.12.2015

2. •Introductie Pair Programming
•Oefeningen
•Introductie Code Reviews
•Oefening
Agenda_
2
Wat gaan we doen:

3. Pair Programming
3

4. •Efficient
•Kwaliteit design
•Leren
•Communicatie
Pair Programming_
4
Voordelen

5. /* sarcasm-mode:on */
5

6. •Geef zoveel mogelijk aanwijzingen
•Stel alles ter discussie
•Neem het toetsenbord over als de vaart eruit raakt
•Gebruik je telefoon om dingen op te zoeken
Pair Programming_
6
Pairen

7. /* sarcasm-mode: off */
7

8. •Driver en Navigator
•Doelen bepalen en noteren
•Communiceren
•5 sec regel
Pair Programming_
8
Rolverdeling en aanpak

9. Pairing opdracht
9

10. Oefening 1_
Gezamenlijk apart bouwen
-Maak groepen van twee personen
-Kies een puzzel
-Afzonderlijk van elkaar bouwen, zonder communiceren
-Aan het einde delen samenvoegen
Als klaar; puzzel afbreken + uitdrukvel bewaren
10

11. Oefening 2_
Pairen
11
-Kies een andere puzzel
-Eén bouwer
-Eén aangever (mag geen stukjes in elkaar zetten)
Als klaar; puzzel afbreken en stukjes terug in het uitdrukvel

12. Evaluatie
12

13. Code Reviews
13

14. •Minder fouten
•Beter design
•Leren
Code Reviews_
14
Waarom

15. /* sarcasm-mode:on */
15

16. •Commits zo groot mogelijk
•Combineer functionaliteit en refactoring
•Eigen formatting rules automatisch laten uitvoeren
Code Reviews_
16
Voorbereiding code review

17. •Kijk alleen naar de ‘diffs’
•Focus op de grootste / meest complexe wijzigingen
•Als je fouten vindt, terug sturen en niet teveel aandacht aan
besteden
•Dubbele punten voor spelfouten
Code Reviews_
17
Code review uitvoeren

18. /* sarcasm-mode: off */
18

19. •Duur < 1 uur
•Korte cycle time (code -> review -> feedback)
•Tools ter ondersteuning
•Ook de test code
Code Reviews_
19
Code review guidelines

20. Review opdracht
20

21. Oefening 1_
Review de code
-Wat kan beter?
-Heb je fouten gevonden?
21

22. Evaluatie
22
-112; ‘!==‘ => ‘==‘
-137; comment incorrect
-202 / 206; regels omgewisseld

23. 23