Ich habe das Refactoring im Notification-Service durchgesehen. Der Code ist deutlich lesbarer als vorher, und die neue Struktur hilft beim Testen. Zwei Dinge sollten wir aber vor dem Merge noch anpassen. Erstens ist die Methode zum Versand von E-Mails noch sehr lang und mischt mehrere Aufgaben in einem Block. Wenn wir die Prüfung der Empfänger und den eigentlichen Versand trennen, wird der Ablauf klarer.
Glossar
- dokumentieren
- schriftlich erklären und speichern„Außerdem wäre es gut, im Kommentar kurz zu dokumentieren, warum du die alte Retry-Logik entfernt hast.“
- das Merge
- Zusammenführen von Code-Änderungen.„Zwei Dinge sollten wir aber vor dem Merge noch anpassen.“
- anpassen
- klein ändern oder verbessern„Zwei Dinge sollten wir aber vor dem Merge noch anpassen.“
- das Refactoring
- Verbesserung des Codes ohne Funktionsänderung.„Ich habe das Refactoring im Notification-Service durchgesehen.“
- der Kommentar
- kurze schriftliche Rückmeldung„wäre es gut, im Kommentar kurz zu dokumentieren, warum du die alte Retry-Logik entfernt hast.“