Code-Smell-Detektor

Das Problem

Code Smells haeufen sich schleichend an. Eine Funktion waechst auf 200 Zeilen, eine Klasse uebernimmt sieben Verantwortlichkeiten, Magic Numbers tauchen ueberall auf. Wenn sich die Codebasis “schwer” anfuehlt, sind die Smells so tief verankert, dass Entwickler aus Angst vor Regressionen das Refactoring meiden. Fruehe Erkennung verhindert, dass technische Schulden sich auftuermen.

Der Prompt

Du bist ein erfahrener Code-Qualitaetsingenieur. Analysiere folgenden Code auf Code Smells:

CODE:
[fuege deinen Code hier ein — eine Funktion, Klasse oder ein Modul]

SPRACHE: [z.B. TypeScript, Python, Java]

Fuer jeden gefundenen Code Smell liefere:
1. **Smell-Name**: Die offizielle Bezeichnung (z.B. Long Method, Feature Envy, Primitive Obsession)
2. **Position**: Welche Zeilen oder welcher Abschnitt
3. **Schweregrad**: Hoch / Mittel / Niedrig
4. **Warum es wichtig ist**: Das konkrete Risiko, wenn es nicht behoben wird
5. **Refactoring**: Die spezifische Refactoring-Technik (z.B. Extract Method, Replace Conditional with Polymorphism)
6. **Refactored Code**: Zeige die verbesserte Version dieses Abschnitts

Priorisiere Smells nach Schweregrad. Melde nur echte Probleme — keine Stilpraeferenzen bemaengeln.

Beispielausgabe

1. Long Method — processOrder() umfasst 147 Zeilen (Zeilen 23-170)
   Schweregrad: Hoch
   Warum: Einzelne Schritte sind nicht testbar; jede Aenderung riskiert Seiteneffekte.
   Refactoring: Extract Method — aufteilen in validateItems(), calculateTotal(),
   applyDiscounts(), processPayment(), sendConfirmation().

2. Magic Numbers — Zeilen 45, 67, 89
   Schweregrad: Mittel
   Warum: 0.08 (Steuersatz?), 500 (Versandkostenfreigrenze?), 30 (Rueckgabetage?)
   sind unklar und dupliziert.
   Refactoring: Replace Magic Number with Named Constant.
   const TAX_RATE = 0.08;
   const FREE_SHIPPING_THRESHOLD = 500;
   const RETURN_WINDOW_DAYS = 30;

Wann verwenden

Verwenden Sie diesen Skill bei Code-Reviews, vor groesseren Refactoring-Vorhaben oder beim Einarbeiten in eine Legacy-Codebasis. Am wirkungsvollsten, wenn Sie ganze Dateien oder Klassen einfuegen, da viele Code Smells die Beziehungen zwischen Codeteilen betreffen.

Profi-Tipps

• Pruefen Sie zuerst Ihren eigenen Code — Feedback von der KI ist leichter anzunehmen als von Kollegen, und Sie fangen Smells vor dem Code-Review ab.
• Fordern Sie einen priorisierten Refactoring-Plan an — ergaenzen Sie “Erstelle einen Refactoring-Plan, den ich in 3 einstuendigen Sitzungen umsetzen kann” fuer eine umsetzbare Roadmap.
• Kombinieren Sie mit Tests — fragen Sie “Welche Tests soll ich vor jedem Refactoring schreiben?”, damit Sie ein Sicherheitsnetz haben, bevor Sie etwas aendern.