Bis zum aktuellen Zeitpunkt haben 49 Personen an der Umfrage zur Verwendung von Code Reviews teilgenommen. Dabei sind interessante Ergebnisse zu Tage gekommen. Fast die Hälfte setzt das Mittel der Code Reviews überhaupt nicht ein, der Rest sehr unterschiedlich. Aber sehen wir uns das näher an.
Auswertung
Nachfolgend finden sich einige Charts, die sich aus den abgegebenen Antworten ergeben haben.
Wer noch keine Stimme abgegeben hat, wird herzlich dazu eingeladen, dies hier zu tun. Jede weitere Stimme bringt mir (und in weiterer Folge meinen Lesern) ein klareres Bild hinsichtlich des Einsatzes in der Realität.
Allgemeine Verwendung von Code Reviews
Aufschlüsselung nach Fragen:
Kategorisierung nach Verwendungszweck:
Verwendung von Tests (berücksichtigt wurden nur konkrete Angaben dazu, 21 Stimmen):
Findings
Mit 55% verwendet mehr als die Hälfte Code Reviews als Mittel, den Code zu verbessern. Negativ fällt auf, dass lediglich 29% auch Tests in ihrem Entwicklungsprozess berücksichtigen. Dies lässt den Schluss zu, dass vielfach auf Codequalität Hauptaugenmerk gelegt wird, nicht jedoch auf Fehlerfreiheit. Dies wird zusätzlich durch nicht eingesetzte Zuständigkeiten untermauert. Heißt auf gut Deutsch: Der Code wird häufig nicht durch eine mit dem Bereich vertraute Person reviewed.
Zuständigkeit vs. keine Zuständigkeit
Ein Thema, das mich persönlich aktuell gerade massiv beschäftigt. Im Beitrag Code Reviews im Daily Business – Ein Entwicklungsprozess habe ich noch dafür plädiert, die Reviewer zu würfeln. Davon bin ich mittlerweile wieder abgegangen, wäre doch eine Prüfung durch eine Kontext-nahe Person sehr sinnvoll. Doch mehr dazu in einem weiteren Beitrag.
<h2>Conclusio</h2> <p>Das Ergebnis ist bei weitem nicht so eindeutig ausgefallen, wie ich es mir erhoffte. Code Reviews werden sehr unterschiedlich behandelt, einen einheitlichen Weg gibt es nicht. Darin dürfte auch das Problem liegen, warum viele dieses Hilfsmittel erst gar nicht einsetzen, oder im Laufe der Zeit wieder verworfen haben.</p>