Code-Review in Java: Was zu prüfen ist und wie man Feedback gibt

Code-Review in Java: Was zu prüfen ist und wie man Feedback gibt

Code-Review (Code-Überprüfung) ist nicht nur eine Formalität, sondern eine der effektivsten Methoden, um die Codequalität zu verbessern und das professionelle Niveau des Teams zu steigern. Für Java-Entwickler-Anfänger kann die Teilnahme an Reviews einschüchternd wirken: "Was, wenn ich einen Bug übersehe?", "Wie sage ich einem Kollegen, dass sein Code nicht perfekt ist?". In diesem Artikel analysieren wir, worauf bei der Überprüfung von Java-Code zu achten ist und wie man Feedback gibt, das nützlich und nicht verletzend ist.

1. Was in Java-Code prüfen: Von Syntax bis Architektur

Code-Review beschränkt sich nicht auf die Suche nach Tippfehlern. Ein guter Reviewer betrachtet den Code systematisch. Hier sind die wichtigsten Prüfkategorien für Java-Projekte:

1.1. Korrektheit und Logik

Die erste und wichtigste Frage: "Löst der Code die gestellte Aufgabe?". Prüfen Sie:

\n
• Grenzfälle (null, leere Collections, negative Werte).
\n
• Die Logik von Bedingungsanweisungen und Schleifen.
\n
• Die korrekte Behandlung von Ausnahmen (verwenden Sie Exception nicht als universelle Lösung).
\n

Beispiel für schlechten Code:

public int divide(int a, int b) {\n    return a / b; // Potenzielle Division durch Null\n}

Was vorschlagen:

public int divide(int a, int b) {\n    if (b == 0) {\n        throw new IllegalArgumentException("Der Divisor darf nicht Null sein");\n    }\n    return a / b;\n}

1.2. Lesbarkeit und Stil

Java ist eine Sprache mit etablierten Konventionen (Java Code Conventions, Google Java Style). Achten Sie auf:

\n
• Namen von Variablen, Methoden, Klassen (sollten aussagekräftig sein).
\n
• Formatierung (Einrückungen, Leerzeichen, Zeilenlänge).
\n
• Vorhandensein von magischen Zahlen (ziehen Sie sie in Konstanten heraus).
\n

Beispiel:

// Schlecht\ndouble x = 3.14159 * r * r;\n\n// Gut\nprivate static final double PI = 3.14159;\ndouble area = PI * radius * radius;

1.3. Leistung und Optimierung

Für Java besonders wichtig:

\n
• Vermeiden Sie unnötige Objekterstellung in Schleifen.
\n
• Verwenden Sie StringBuilder für die String-Verkettung in Schleifen.
\n
• Überprüfen Sie die Komplexität von Algorithmen (O(n), O(n log n)).
\n

Beispiel:

// Schlecht (es werden viele String-Objekte erstellt)\nString result = "";\nfor (int i = 0; i < 1000; i++) {\n    result += i;\n}\n\n// Gut\nStringBuilder sb = new StringBuilder();\nfor (int i = 0; i < 1000; i++) {\n    sb.append(i);\n}\nString result = sb.toString();

1.4. Testbarkeit und Tests

Überprüfen Sie, ob der neue Code mit Unit-Tests abgedeckt ist. Achten Sie auf:

\n
• Vorhandensein von Tests für Grenzfälle.
\n
• Fehlen von Abhängigkeiten von externen Systemen (DB, API) in Unit-Tests.
\n
• Lesbarkeit der Tests selbst (assert-Methode mit verständlichen Nachrichten).
\n

2. Wie man Feedback gibt: konstruktiv und ohne Toxizität

Selbst das korrekteste Code-Review kann scheitern, wenn das Feedback aggressiv oder unkonkret gegeben wird. Hier sind einige Regeln für Anfänger:

2.1. Die "Sandwich"-Regel

Beginnen Sie mit Positivem, weisen Sie dann auf Mängel hin, und enden Sie mit einer Ermutigung. Zum Beispiel:

\n
• Positiv: "Tolle Idee, die Validierung in eine separate Methode auszulagern!"
\n
• Anmerkung: "Aber hier wäre es besser, Optional anstelle von null zurückzugeben, um NullPointerException zu vermeiden."
\n
• Abschluss: "Ansonsten ist der Code sauber, danke für die Arbeit!"
\n

2.2. Verwenden Sie "Ich"-Aussagen und Fragen

Sagen Sie statt "Du hast das falsch geschrieben" lieber: "Ich glaube, hier könnte ein Bug bei einer leeren Liste auftreten. Was denkst du?". Dies reduziert die Abwehrreaktion und lenkt das Gespräch in konstruktive Bahnen.