Code-Reviews sind ein wesentlicher Bestandteil der Arbeit eines Softwareentwicklers. Ein sehr interessanter Aspekt dabei ist, dass ein Entwickler beide Seiten des Review-Prozesses erlebt - als Autor und als Reviewer.
Ein Autor ist oft derjenige, der am besten mit dem Konzept, den Nebeneffekten und den Einschränkungen der Lösung vertraut ist. Wenn er den Code jedoch zu oft gesehen hat, leidet er möglicherweise unter der sogenannten „Code Blindheit“ – er sieht nicht mehr das große Ganze und verliert sich in den Details. Dies ist der Moment, in dem ein frisches Paar Augen von besonderer Bedeutung ist.
Ein Reviewer ist derjenige, der einen frischen Blick auf die Dinge hat. Er sieht die Lösung in der Regel aus einem anderen Blickwinkel und bemerkt Probleme, die der Autor nicht wahrgenommen hat. Ähnlich wie ein Redakteur kann ein Prüfer den Code kommentieren oder sogar Änderungen fordern, um die Gesamtqualität zu verbessern. Er ist auch derjenige, der das beste Feedback darüber gibt, wie verständlich und klar die Änderungen für die Entwickler sind, die nicht an der Funktion beteiligt sind.
Wir bei djangsters glauben, dass Code-Reviews entscheidend für eine gesunde Projektentwicklung sind. Unser Motto lautet „Du bist nicht mit deinem Code verheiratet“, was bedeutet, dass wir eine emotionale Bindung an eine bestimmte Lösung vermeiden und offen für Änderungen und Verbesserungen bleiben. In unserem Team ermutigen wir zu Kommentaren, Fragen und Vorschlägen. Wir schätzen das Feedback und die Bemühungen eines jeden Kritikers sehr und sehen es nicht nur als Sicherheitsnetz, sondern auch als Zusammenarbeit zwischen den Entwicklern. Manchmal kann es zu einer neuen und qualitativ besseren Lösung führen, als es ein Alleingang vermocht hätte.
Unsere Erfahrung zeigt, dass die Code-Review-Praxis zu erheblichen Verbesserungen der Architektur und der Lesbarkeit führt, was wiederum eine schnellere Entwicklung von Funktionen ermöglicht. Darüber hinaus sind wir gerne bereit, während der Entwicklung eine zusätzliche Review-Iteration durchzuführen, anstatt Feuer am laufenden System zu bekämpfen – eine Win-win-Strategie für Entwickler und Stakeholder.
