Best Practices

Code Review

Robin Sedlaczek

Fairmas GmbH, Berlin 2013 2

Code Review Best Practices

Sensibilisierung

Maßnahmen/Vorgehen

Ziele

Fairmas GmbH, Berlin 2013 3

Code Review Best Practices

Review-Technik

Was ist ein Code Review?

Analytische Qualitätssicherungsmaßnahme

Methode agiler Prozesse

Forschungsgegenstand

Fairmas GmbH, Berlin 2013 4

Code Review Best Practices

Finden von Bugs

Warum Code Reviews?

Verbesserung der Code Qualität

Reduktion von Kosten

Lerneffekt + Teambildung

Fairmas GmbH, Berlin 2013 5

Code Review Best Practices

InspektionsrateWie schnell können wir Code reviewen (kLOC/h)?

Begriffe

DefektrateWie schnell können wir Fehler/Bugs finden (Anzahl/h)?

DefektdichteWie viele Fehler finden wir in einer bestimmten Menge Code (Anzahl/kLOC)?

Fairmas GmbH, Berlin 2013 6

Code Review Best Practices

Nicht mehr als 200-400 LOC

Nicht länger als 60-90 Minuten

Best Practice #1

Quelle: http://smartbear.com/SmartBear/media/pdfs/WP-CC-11-Best-Practices-of-Peer-Code-Review.pdf

Fairmas GmbH, Berlin 2013 7

Code Review Best Practices

Geht langsam vor!

Qualität vor Quantität.

Best Practice #2

Fairmas GmbH, Berlin 2013 8

Code Review Best Practices

Bereitet Euch vor!

Wählt vor dem Review Code aus, über den

Ihr reden möchtet und schaut Euch diesen

im Vorfeld an!

Best Practice #3

Fairmas GmbH, Berlin 2013 9

Code Review Best Practices

Best Practice #4

Autorenvorbereitung!

Autor macht Notizen zum

ausgewählten Code.

Quelle: http://smartbear.com/SmartBear/media/pdfs/WP-CC-11-Best-Practices-of-Peer-Code-Review.pdf

Fairmas GmbH, Berlin 2013 10

Code Review Best Practices

Protokolliert Ergebnisse!

Notiert die gefundenen Probleme, um

daraus Aufgaben/Tickets abzuleiten.

Best Practice #5

Fairmas GmbH, Berlin 2013 11

Code Review Best Practices

Macht Ergebnisse Messbar!

Z.B. Defektdichte. Protokolliert am Ende des

Reviews, wie viele Probleme Ihr gefunden

habt. Setzt Ziele für das nächste Meeting.

Best Practice #6

Fairmas GmbH, Berlin 2013 12

Code Review Best Practices

Review des Reviews!

Prüft und notiert, wie viele Probleme aus

dem letzten Code Review beseitigt wurden.

Best Practice #7

Fairmas GmbH, Berlin 2013 13

Code Review Best Practices

Erstellt eine Checkliste!

An Hand einer Checkliste können Themenbereiche

strukturiert abgearbeitet werden. Nutzt die

Checkliste als Protokoll (siehe #5).

Best Practice #8

Fairmas GmbH, Berlin 2013 14

Code Review Best Practices

Defekte sind cool!

Gebt Euch gegenseitig das Gefühl, dass niemand

angegriffen wird! Im Gegenteil: gefundene Probleme

bedeuten, dass Ihr als Team großartig

zusammenarbeitet.

Best Practice #9