Code Review Vortrag auf der Continuous Lifecycle Conference 2014 in Mannheim.

1. Code Reviews
Leave your ego at the door
@FrankS

2. InnoGames GmbH
ca. 15 Jahre Entwickler (PHP, Java, etc.)
PHP Usergroup Hamburg
Und ich lerne immer noch…
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
2
Kurz zu mir…

3. Eine kurze Einführung zu Reviews
Collective Code Ownership
Das richtige Werkzeug wählen
Typische Probleme vermeiden
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
3
Worum geht es hier?

4. Eine kurze Einführung zu Reviews
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
4
Lassen wir den Code mal kurz weg…

5. “Peer review methods are employed to
maintain standards, improve performance
and provide credibility.”
Wikipedia: Peer Review
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
5

6. Reviews gibt es nicht nur für Code…
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door 6

7. IBM: Inspektionen seit 1970ern
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
7
Zurück zu den Anfängen der Code Reviews
Strikte Checklisten
Alles andere als agile

8. “Das ist nicht mein Code!”
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
8
Collective Code Ownership ist eine Einstellung

9. “Everyone has the right and
freedom to totally suck in private.”
Jeff Atwood (Stack Overflow ) - @codinghorror
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
9

10. “Attempt to be awesome in
public and embrace the suck!”
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
10
Jeff Atwood (Stack Overflow ) - @codinghorror

11. Reviews sind knowledge sharing!
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door 11

12. Produktwissen
Technisches Wissen
Coding Standards
Transparenz schaffen!
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
12
Wissen und Erfahrungen im Team teilen…

13. Die richtige Review Methode finden
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
13
Das richtige Werkzeug für den richtigen Zweck!

14. “Hast du mal kurz Zeit?
Ich hab da nen Problem.”
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
14

15. Braucht nicht viel Zeit
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
15
Ad hoc reviews
Geht überall
Löst nur das konkrete Problem

16. Talk to the duck
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door 16

17. “Kannst du dir nachher mal
den Code anschauen?”
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
17

18. Geht meist asynchron
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
18
Der “Schreibtischtest”
Gut für große Mengen an Code
Review tools arbeiten so

19. “Lass uns das zusammen machen!”
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
19

20. “Driver” und “Navigator”
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
20
Pair Programming
Schnelle Iterationen
“Review on the fly”

21. “Ich zeige euch mal meinen Code.”
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
21

22. Autor erklärt den Code
Entwickler stellen Fragen
Ideal zum vorstellen und präsentieren
Risiko etwas zu übersehen
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
22
Code walkthrough

23. “Kannst du meinen Code
den anderen erklären?”
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
23

24. Erklären ohne Code zu kennen
Auch auf das große Ganze achten
Autor beantwortet Fragen
Code gezielt auswählen
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
24
Code Review - “in a big round”

25. Typische Probleme vermeiden
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
25
Es gibt Stolpersteine…

26. “My ego is not tied to my perfect
or imperfect work product.”
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
26
Jeff Atwood (Stack Overflow ) - @codinghorror

27. Das Ego an der Tür lassen
Nicht persönlich werden
Ergebnisse nachbereiten
Nicht missbrauchen!
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
27
Stolpersteine vermeiden

28. Nicht zu viele Teilnehmer
Maximal zwei Stunden
Probleme finden, keine Lösungen
Umfang Review und Projekt
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
28
Ein paar praktische Tipps…

29. Code wird verständlicher
Wissen wird weiter gegeben
Kommunikation im Team
Leuchtturm Effekt
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
29
Es gibt viele Vorteile…

30. Einfach mal machen…
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
30
Just do it!

31. Für mehr Hintergrundinfos:
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door 31

32. Reviews helfen dem Team
continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door
32
Viel Spaß bei euren Reviews!
Die Einstellung ist das wichtigste
Immer wieder anpassen

33. Vielen Dank!
frank.sons@starside.de
@FrankS
slideshare.net/frank_sons
Sprecht mich an :-)