meldestelle/docs/99_Journal/2026-04-20_Code-Cleanup-Smells.md

1.7 KiB

Journal: Code-Cleanup & Smell-Entfernung

Datum: 20. April 2026 Agent: 🧐 [QA Specialist] & 🏗️ [Lead Architect]

🎯 Ziel

Beseitigung von Code-Smells, ungenutzten Parametern und Code-Duplikaten in den kürzlich refactorten Turnier-Komponenten.

🛠️ Durchgeführte Änderungen

1. TurnierDetailScreen & Shell-Integration

  • Problem: Parameter onBack in TurnierDetailScreen wurde nicht verwendet.
  • Lösung: Parameter entfernt und alle Aufrufstellen in DesktopMainLayout.kt sowie ScreenPreviews.kt angepasst.
  • Grund: Leaner Code-Design und Vermeidung von Verwirrung bei der API-Nutzung.

2. DesktopMainLayout (Navigation)

  • Problem: Der Zweig is AppScreen.Vereine war redundant und teilweise nicht erreichbar.
  • Lösung: Redundanten Zweig entfernt. Die Navigation zu Vereinen wird bereits weiter oben im when-Block (Z. 668) abgehandelt.

3. TurnierStammdatenTab (Refactoring)

  • Problem: Ungenutzter Parameter veranstalterName. Mehrfache Code-Duplikate bei der Datumsvalidierung und den DatePicker-Dialogen.
  • Lösung:
    • Parameter veranstalterName entfernt.
    • Neue Hilfsfunktion isDateRangeValid(von, bis, eventVon, eventBis) erstellt, um die Validierungslogik zu zentralisieren.
    • Neue Composable-Funktion TurnierDatePickerDialog erstellt, um die redundante Dialog-Struktur zu eliminieren.
  • Ergebnis: Reduzierung der Dateigröße und deutlich bessere Wartbarkeit.

Verifikation

  • Build: ./gradlew :frontend:shells:meldestelle-desktop:compileKotlinJvm war erfolgreich.
  • Code-Check: Manuelle Prüfung der bereinigten Stellen auf Konsistenz.

Status: Abgeschlossen. Codebase ist nun sauber für die weitere Feature-Entwicklung.