Abendvortrag bei oose.de am 24.03.2016 Code Reviews sind heute in vielen Teams Standard. Sei es als allgemeine Vier-Augen-Regel, bei pull requests oder tatsächlich durch Teammeetings. All diese Methoden haben Vor- und Nachteile, und nur in den seltensten Fällen kann man damit tatsächlich effektiv Bugs finden. Aber meist werden Code Reviews genau mit diesem Ziel eingeführt: Fehler vermeiden! Der Vortrag zeigt verschiedene Methoden für effektive Code Reviews auf und erklärt, warum toolbasierte Code Reviews eigentlich erst ein zweiter Schritt sein sollten und stattdessen das mindset der Entwickler im Vordergrund stehen sollte. Praktische Tipps um die eigenen Code Reviews zu verbessern runden den Vortrag ab.

1. code-quality.de
 @FrankS
Effektive Code Reviews

2. Moin moin!
Frank SonsFrank Sons  FrankS FrankS
http://code-quality.dehttp://code-quality.de

3. code-quality.de
 @FrankS
Agenda – Code Reviews
Teil 1: Methoden, Ziele und ProblemeTeil 1: Methoden, Ziele und Probleme
Teil 2: Rollen, Guidelines und ProzesseTeil 2: Rollen, Guidelines und Prozesse

4. code-quality.de
 @FrankS
Wichtig!
Stellt Fragen!Stellt Fragen!
Teilt eure Erfahrungen!Teilt eure Erfahrungen!

5. code-quality.de
 @FrankS
Teil 1: Methoden, Ziele und Probleme
Welche Methoden gibt es?Welche Methoden gibt es?
Was sind die Ziele?Was sind die Ziele?
Wo gibt es Probleme?Wo gibt es Probleme?

6. code-quality.de
 @FrankS
Wie macht ihr eure Code Reviews?

7. code-quality.de
 @FrankS
Ad hoc

8. code-quality.de
 @FrankS
Pair Programming

9. code-quality.de
 @FrankS
Review Tools - asynchron

10. code-quality.de
 @FrankS
Review Meeting

11. code-quality.de
 @FrankS
Team Review

12. code-quality.de
 @FrankS
Teil 1: Methoden, Ziele und Probleme
Welche Methoden gibt es?Welche Methoden gibt es?
Was sind die Ziele?Was sind die Ziele?
Wo gibt es Probleme?Wo gibt es Probleme?

13. code-quality.de
 @FrankS
Warum macht ihr eigentlich Code Reviews?

14. code-quality.de
 @FrankS
Motivation von Code Reviews

15. code-quality.de
 @FrankS
Was bringt es wirklich?

16. code-quality.de
 @FrankS
Teil 1: Methoden, Ziele und Probleme
Welche Methoden gibt es?Welche Methoden gibt es?
Was sind die Ziele?Was sind die Ziele?
Wo gibt es Probleme?Wo gibt es Probleme?

17. code-quality.de
 @FrankS
Kommt euch das bekannt vor?
Keine Struktur, keine ZieleKeine Struktur, keine Ziele
Klappt schon irgendwieKlappt schon irgendwie
Jeder für sich  Glückssache!Jeder für sich  Glückssache!

18. code-quality.de
 @FrankS
Was kann man machen?
Tatsächliche Ziele klar machenTatsächliche Ziele klar machen
Reviews im Prozess etablierenReviews im Prozess etablieren
Gemeinsames Qualitätsverständnis aufbauenGemeinsames Qualitätsverständnis aufbauen

19. code-quality.de
 @FrankS
Pause?
Oder Fragen?Oder Fragen?

20. code-quality.de
 @FrankS
Agenda – Code Reviews
Teil 1: Methoden, Ziele und ProblemeTeil 1: Methoden, Ziele und Probleme
Teil 2: Rollen, Guidelines und ProzesseTeil 2: Rollen, Guidelines und Prozesse

21. code-quality.de
 @FrankS
Fokus
Meetings und ToolsMeetings und Tools

22. code-quality.de
 @FrankS
Wie laufen eure Code Reviews ab?

23. code-quality.de
 @FrankS
Teil 2: Rollen, Guidelines und Prozesse
Welche Rollen gibt es?Welche Rollen gibt es?
Wonach schaut man eigentlich?Wonach schaut man eigentlich?
Wie bindet man es in die Prozesse ein?Wie bindet man es in die Prozesse ein?

24. code-quality.de
 @FrankS
Rollen im Review Meeting
AutorAutor
ReaderReader
ModeratorModerator
(Recorder)(Recorder)

25. code-quality.de
 @FrankS
Teil 2: Rollen, Guidelines und Prozesse
Welche Rollen gibt es?Welche Rollen gibt es?
Wonach schaut man eigentlich?Wonach schaut man eigentlich?
Wie bindet man es in die Prozesse ein?Wie bindet man es in die Prozesse ein?

26. code-quality.de
 @FrankS
Moderation ist wichtig!
Auf Annahmen achtenAuf Annahmen achten
Statt Schweigen: laut denken!Statt Schweigen: laut denken!
Diskussionen eingrenzenDiskussionen eingrenzen

27. code-quality.de
 @FrankS
Teil 2: Rollen, Guidelines und Prozesse
Welche Rollen gibt es?Welche Rollen gibt es?
Wonach schaut man eigentlich?Wonach schaut man eigentlich?
Und das läuft dann von alleine?Und das läuft dann von alleine?

28. code-quality.de
 @FrankS
Mit dem Meeting loslegen
Code gemeinsam aussuchenCode gemeinsam aussuchen
Regeln klar machenRegeln klar machen
Rollen verteilenRollen verteilen

29. code-quality.de
 @FrankS
Auf den Fokus achten
Code MetrikenCode Metriken
Code VerständnisCode Verständnis

30. code-quality.de
 @FrankS
Muss man den Code verstehen?

31. code-quality.de
 @FrankS
Lesbarkeit und Verständlichkeit
Nur wer den Code versteht,
kann auch die Probleme finden!
Nur wer den Code versteht,
kann auch die Probleme finden!

32. code-quality.de
 @FrankS
Ein kleiner, aber feiner Unterschied
Coding Style GuideCoding Style Guide
Coding GuidelineCoding Guideline

33. code-quality.de
 @FrankS
Was würde in euren Guidelines stehen?

34. code-quality.de
 @FrankS
Guidelines können als Checkliste dienen!

35. code-quality.de
 @FrankS
Prozesse: oder das drumherum...
Welche Rollen gibt es?Welche Rollen gibt es?
Wonach schaut man eigentlich?Wonach schaut man eigentlich?
Und das läuft dann von alleine?Und das läuft dann von alleine?

36. code-quality.de
 @FrankS
Achtung!
Prozesse bedeutet immer: YMMV!Prozesse bedeutet immer: YMMV!

37. code-quality.de
 @FrankS
Code Review Meetings im Prozess
Im letzten Drittel vom SprintIm letzten Drittel vom Sprint
90 - 120min90 - 120min
Immer wieder anpassenImmer wieder anpassen

38. code-quality.de
 @FrankS
Code Review Tools im Prozess
Extra Status für TasksExtra Status für Tasks
Automatische Systeme helfenAutomatische Systeme helfen
Guideline dient als ChecklisteGuideline dient als Checkliste

39. code-quality.de
 @FrankS
Macht mehr aus euren Reviews
„Peer code reviews are the single biggest
thing you can do to improve your code.“
„Peer code reviews are the single biggest
thing you can do to improve your code.“
Jeff Atwood – coding horror

40. code-quality.de
 @FrankS
Eure Fragen?