code-quality.de
 @FrankS
Effektive Code Reviews
Moin moin!
Frank SonsFrank Sons  FrankS FrankS
http://code-quality.dehttp://code-quality.de
code-quality.de
 @FrankS
Agenda – Code Reviews
Teil 1: Methoden, Ziele und ProblemeTeil 1: Methoden, Ziele und Probleme
T...
code-quality.de
 @FrankS
Wichtig!
Stellt Fragen!Stellt Fragen!
Teilt eure Erfahrungen!Teilt eure Erfahrungen!
code-quality.de
 @FrankS
Teil 1: Methoden, Ziele und Probleme
Welche Methoden gibt es?Welche Methoden gibt es?
Was sind d...
code-quality.de
 @FrankS
Wie macht ihr eure Code Reviews?
code-quality.de
 @FrankS
Ad hoc
code-quality.de
 @FrankS
Pair Programming
code-quality.de
 @FrankS
Review Tools - asynchron
code-quality.de
 @FrankS
Review Meeting
code-quality.de
 @FrankS
Team Review
code-quality.de
 @FrankS
Teil 1: Methoden, Ziele und Probleme
Welche Methoden gibt es?Welche Methoden gibt es?
Was sind d...
code-quality.de
 @FrankS
Warum macht ihr eigentlich Code Reviews?
code-quality.de
 @FrankS
Motivation von Code Reviews
code-quality.de
 @FrankS
Was bringt es wirklich?
code-quality.de
 @FrankS
Teil 1: Methoden, Ziele und Probleme
Welche Methoden gibt es?Welche Methoden gibt es?
Was sind d...
code-quality.de
 @FrankS
Kommt euch das bekannt vor?
Keine Struktur, keine ZieleKeine Struktur, keine Ziele
Klappt schon ...
code-quality.de
 @FrankS
Was kann man machen?
Tatsächliche Ziele klar machenTatsächliche Ziele klar machen
Reviews im Pro...
code-quality.de
 @FrankS
Pause?
Oder Fragen?Oder Fragen?
code-quality.de
 @FrankS
Agenda – Code Reviews
Teil 1: Methoden, Ziele und ProblemeTeil 1: Methoden, Ziele und Probleme
T...
code-quality.de
 @FrankS
Fokus
Meetings und ToolsMeetings und Tools
code-quality.de
 @FrankS
Wie laufen eure Code Reviews ab?
code-quality.de
 @FrankS
Teil 2: Rollen, Guidelines und Prozesse
Welche Rollen gibt es?Welche Rollen gibt es?
Wonach scha...
code-quality.de
 @FrankS
Rollen im Review Meeting
AutorAutor
ReaderReader
ModeratorModerator
(Recorder)(Recorder)
code-quality.de
 @FrankS
Teil 2: Rollen, Guidelines und Prozesse
Welche Rollen gibt es?Welche Rollen gibt es?
Wonach scha...
code-quality.de
 @FrankS
Moderation ist wichtig!
Auf Annahmen achtenAuf Annahmen achten
Statt Schweigen: laut denken!Stat...
code-quality.de
 @FrankS
Teil 2: Rollen, Guidelines und Prozesse
Welche Rollen gibt es?Welche Rollen gibt es?
Wonach scha...
code-quality.de
 @FrankS
Mit dem Meeting loslegen
Code gemeinsam aussuchenCode gemeinsam aussuchen
Regeln klar machenRege...
code-quality.de
 @FrankS
Auf den Fokus achten
Code MetrikenCode Metriken
Code VerständnisCode Verständnis
code-quality.de
 @FrankS
Muss man den Code verstehen?
code-quality.de
 @FrankS
Lesbarkeit und Verständlichkeit
Nur wer den Code versteht,
kann auch die Probleme finden!
Nur we...
code-quality.de
 @FrankS
Ein kleiner, aber feiner Unterschied
Coding Style GuideCoding Style Guide
Coding GuidelineCoding...
code-quality.de
 @FrankS
Was würde in euren Guidelines stehen?
code-quality.de
 @FrankS
Guidelines können als Checkliste dienen!
code-quality.de
 @FrankS
Prozesse: oder das drumherum...
Welche Rollen gibt es?Welche Rollen gibt es?
Wonach schaut man e...
code-quality.de
 @FrankS
Achtung!
Prozesse bedeutet immer: YMMV!Prozesse bedeutet immer: YMMV!
code-quality.de
 @FrankS
Code Review Meetings im Prozess
Im letzten Drittel vom SprintIm letzten Drittel vom Sprint
90 - ...
code-quality.de
 @FrankS
Code Review Tools im Prozess
Extra Status für TasksExtra Status für Tasks
Automatische Systeme h...
code-quality.de
 @FrankS
Macht mehr aus euren Reviews
„Peer code reviews are the single biggest
thing you can do to impro...
code-quality.de
 @FrankS
Eure Fragen?
Nächste SlideShare
Wird geladen in …5
×

"Effektive Code Reviews" - Abendvortrag bei oose.de

202 Aufrufe

Veröffentlicht am

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.

Veröffentlicht in: Technologie
0 Kommentare
0 Gefällt mir
Statistik
Notizen
  • Als Erste(r) kommentieren

  • Gehören Sie zu den Ersten, denen das gefällt!

Keine Downloads
Aufrufe
Aufrufe insgesamt
202
Auf SlideShare
0
Aus Einbettungen
0
Anzahl an Einbettungen
3
Aktionen
Geteilt
0
Downloads
0
Kommentare
0
Gefällt mir
0
Einbettungen 0
Keine Einbettungen

Keine Notizen für die Folie

"Effektive Code Reviews" - Abendvortrag bei oose.de

  1. 1. code-quality.de  @FrankS Effektive Code Reviews
  2. 2. Moin moin! Frank SonsFrank Sons  FrankS FrankS http://code-quality.dehttp://code-quality.de
  3. 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. 4. code-quality.de  @FrankS Wichtig! Stellt Fragen!Stellt Fragen! Teilt eure Erfahrungen!Teilt eure Erfahrungen!
  5. 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. 6. code-quality.de  @FrankS Wie macht ihr eure Code Reviews?
  7. 7. code-quality.de  @FrankS Ad hoc
  8. 8. code-quality.de  @FrankS Pair Programming
  9. 9. code-quality.de  @FrankS Review Tools - asynchron
  10. 10. code-quality.de  @FrankS Review Meeting
  11. 11. code-quality.de  @FrankS Team Review
  12. 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. 13. code-quality.de  @FrankS Warum macht ihr eigentlich Code Reviews?
  14. 14. code-quality.de  @FrankS Motivation von Code Reviews
  15. 15. code-quality.de  @FrankS Was bringt es wirklich?
  16. 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. 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. 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. 19. code-quality.de  @FrankS Pause? Oder Fragen?Oder Fragen?
  20. 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. 21. code-quality.de  @FrankS Fokus Meetings und ToolsMeetings und Tools
  22. 22. code-quality.de  @FrankS Wie laufen eure Code Reviews ab?
  23. 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. 24. code-quality.de  @FrankS Rollen im Review Meeting AutorAutor ReaderReader ModeratorModerator (Recorder)(Recorder)
  25. 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. 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. 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. 28. code-quality.de  @FrankS Mit dem Meeting loslegen Code gemeinsam aussuchenCode gemeinsam aussuchen Regeln klar machenRegeln klar machen Rollen verteilenRollen verteilen
  29. 29. code-quality.de  @FrankS Auf den Fokus achten Code MetrikenCode Metriken Code VerständnisCode Verständnis
  30. 30. code-quality.de  @FrankS Muss man den Code verstehen?
  31. 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. 32. code-quality.de  @FrankS Ein kleiner, aber feiner Unterschied Coding Style GuideCoding Style Guide Coding GuidelineCoding Guideline
  33. 33. code-quality.de  @FrankS Was würde in euren Guidelines stehen?
  34. 34. code-quality.de  @FrankS Guidelines können als Checkliste dienen!
  35. 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. 36. code-quality.de  @FrankS Achtung! Prozesse bedeutet immer: YMMV!Prozesse bedeutet immer: YMMV!
  37. 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. 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. 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. 40. code-quality.de  @FrankS Eure Fragen?

×