Code reviews and pair programming

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