Einige Refactoring Themen sind hier von der CS IL09 zusammengetragen:
Name | Beschreibung | Bad-Smell | Behebung | Eclipse Unterstützung |
---|---|---|---|---|
Duplicate Code | Es entstehen leicht Copy-Paste-Fehler und der Code wird schwierig zu warten, da man den Code an mehreren Stellen anpassen muss | Selber Code an mehreren Stellen | Methode extrahieren (Wenn in verschiedenen Klassen Extract-Class ausgenommen Vererbung) | ☑ Extract-Method (Wenn in verschiedenen Klassen Extract-Class ausgenommen Vererbung) |
Large Class | Grosse Klassen sind unübersichtlich und schwer zu warten. Meistens tun sie auch zu viel | ~ Mehr als 7 Public-Methoden (ohne Getter und Setter) oder mehr als 7 Attribute | Klasse extrahieren | ☑ Extract-Class /Subclass |
Long Parameter List | Nicht klar was passiert, wenn nicht für alle Parameter ein gültiger Wert mitgegeben wird | Mehr als 4 Paramter | Objekte und nicht Attribute übergeben | ☐ Keine Unterstützung |
Sinnlose Namen | Unklare Namen werden für Methoden oder Variablen verwendet | Namen bei denen man nicht weiss was sie beinhalten | Rename (sprechende Namen verwenden) | ☑ Rename |
Dead Code | Code welcher nicht aufgerufen wird. Macht die Klasse unübersichtlich | Unbenutzter Code | Code löschen | ☑ "Dead Code" wird angezeigt und kann gelöscht werden |
Comments | Kommentare werden verwendet, um Code zu erklären, wenn der Code aber schon klar ist, benötigt man keinen Kommentar | Kommentar | Rename (sprechende Namen verwenden) | ☑ Rename |