| Auszug |
|---|
|
Nuclos guter Code: Refactoring, Interface, DefaultValueProviderWithIds, Lesbarkeit, Codeverdopplung vermeiden. |
Sprache: Deutsch · English
Wie man es richtig macht:
Refactoring des Negativbeispiels – gemeinsames Interface statt Codeverdopplung.
| Status |
|---|
| colour | Blue |
|---|
| title | Entwickler |
|---|
|
| Status |
|---|
| colour | Green |
|---|
| title | Stand: Jul 2026 |
|---|
|
| Status |
|---|
| colour | Grey |
|---|
| title | gilt fuer Nuclos 4.2026.x |
|---|
|
Der refaktorierte Gegenentwurf zum Negativbeispiel: Ein sinnvolles Interface (hier DefaultValueProviderWithIds), das beide Provider implementieren, beseitigt Codeverdopplung und wirre Verschachtelung – der Code wird lesbar und logisch:
| Codeblock |
|---|
|
Beispiel für schlechte Programmierung |
Verwandte Seiten
Beispiel für schlechte Programmierung:
Vorher.
Öffnen →
Das Beispiel für schlechte Programmierung must nicht sein: Es ist wirklich kein Hexenwerk, man erzeugt zunächst ein nur allzu sinnvolles Interface, das sowohl von CachingCollectableFieldsProvider, als auch von DatasourceBasedCollectableFieldProvider implementiert wird:
public interface DefaultValueProviderWithIds extends DefaultValueProvider {
List<CollectableField> getCollectableFields(List<Integer> lstFieldIds) throws CommonBusinessException;
}
Und schon sieht der ganze Code viel freundlicher und logischer aus, ohne Codeverdoppelung und wirre Verschachtelung:
private CollectableField getValueFromValueListProvider(CollectableField clctfValue) {
...
