Code Smells vor dem Refactoring erkennen

Worum es geht

Ein Code Smell ist ein erkennbares Muster im Code, das kein Fehler ist, aber oft auf eine Designschwäche hindeutet. Gängige Kataloge gruppieren sie: Bloaters (lange Methode, grosse Klasse, lange Parameterliste, Primitive Obsession), Object-Orientation Abusers (Switch-Anweisungen auf den Typ, Refused Bequest), Change Preventers (Divergent Change, Shotgun Surgery), Dispensables (Duplicate Code, toter Code, spekulative Allgemeinheit) und Couplers (Feature Envy, Inappropriate Intimacy, Message Chains).

Warum es wichtig ist

Smells geben Reviewenden und Autoren ein gemeinsames, neutrales Vokabular. «Diese Methode hat Feature Envy nach Order» ist handlungsleitender als «das fühlt sich falsch an», und jeder Smell ist einer kleinen Menge bekannter Refactorings zugeordnet.

So wird es angewendet

• Den Smell in Review-Kommentaren benennen und das dafür vorgesehene Refactoring verlinken.
• Smells in häufig geändertem Code priorisieren; stabiler Code mit Smells kostet wenig.
• Pro Refactoring-Commit einen Smell beheben, mit grünen Tests vorher und nachher.
• Auf Smells achten, die durch eine Änderung neu entstehen, statt die gesamte Codebasis auf einmal zu prüfen.

Stolpersteine

Nicht jeder Smell ist ein Problem: eine lange Methode, die wie eine Checkliste liest, kann klarer sein als fünf winzige. Refactoring ohne Tests macht aus Smells Fehler. Kataloge unterscheiden sich in der Benennung; sich auf einen einigen.