Post on 15-Feb-2017
Code Reviews &Pair Programming_Erasmus Universiteit03.12.2015
•Introductie Pair Programming•Oefeningen
•Introductie Code Reviews•Oefening
Agenda_
2
Wat gaan we doen:
Pair Programming
3
•Efficient•Kwaliteit design•Leren•Communicatie
Pair Programming_
4
Voordelen
/* sarcasm-mode:on */
5
•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
/* sarcasm-mode: off */
7
•Driver en Navigator•Doelen bepalen en noteren•Communiceren•5 sec regel
Pair Programming_
8
Rolverdeling en aanpak
Pairing opdracht
9
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
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
Evaluatie
12
Code Reviews
13
•Minder fouten•Beter design•Leren
Code Reviews_
14
Waarom
/* sarcasm-mode:on */
15
•Commits zo groot mogelijk•Combineer functionaliteit en refactoring•Eigen formatting rules automatisch laten uitvoeren
Code Reviews_
16
Voorbereiding code review
•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
/* sarcasm-mode: off */
18
•Duur < 1 uur•Korte cycle time (code -> review -> feedback)•Tools ter ondersteuning•Ook de test code
Code Reviews_
19
Code review guidelines
Review opdracht
20
Oefening 1_
Review de code- Wat kan beter?- Heb je fouten gevonden?
21
Evaluatie
22
- 112; ‘!==‘ => ‘==‘- 137; comment incorrect- 202 / 206; regels omgewisseld
23