Vereinfachen: Code-Review und Bereinigung
Überprüfen Sie alle geänderten Dateien auf Wiederverwendbarkeit, Qualität und Effizienz. Beheben Sie alle gefundenen Probleme.
Phase 1: Änderungen identifizieren
Führen Sie git diff (oder git diff HEAD bei bereitgestellten Änderungen) aus, um zu sehen, was sich geändert hat.
Phase 2: Überprüfung
Code-Wiederverwendung
- Suchen Sie nach vorhandenen Hilfsprogrammen, die neuen Code ersetzen könnten.
- Kennzeichnen Sie doppelte Funktionalität und schlagen Sie die vorhandene Funktion vor.
- Kennzeichnen Sie Inline-Logik, die ein vorhandenes Hilfsprogramm nutzen könnte.
Code-Qualität
- Redundanter Zustand oder zwischengespeicherte Werte, die abgeleitet werden könnten.
- Kopieren und Einfügen mit geringfügigen Abweichungen – vereinheitlichen Sie mit einer gemeinsamen Abstraktion.
- Unnötige Kommentare, die erklären, WAS der Code tut.
- Rohe Zeichenfolgen, wo bereits Konstanten oder Aufzählungen existieren.
Effizienz
- Redundante Berechnungen, wiederholte Dateilesevorgänge.
- Unabhängige Operationen, die nacheinander ausgeführt werden, obwohl sie parallel laufen könnten.
- Unnötige Existenzprüfungen vor der Bearbeitung (TOCTOU).
Phase 3: Behebung
Beheben Sie jedes Problem direkt. Wenn ein Befund ein False Positive ist, notieren Sie es und fahren Sie fort. Wenn Sie fertig sind, fassen Sie kurz zusammen, was behoben wurde (oder bestätigen Sie, dass der Code bereits sauber war).
Verhalten bewahren. Bevorzugen Sie kleine mechanische Refactorings. Vermeiden Sie nicht zusammenhängende Umschreibungen.