Bei vielen Gelegenheiten im täglichen Programmieren läuft man Gefahr, die Lesbarkeit und Wartbarkeit des Codes deutlich zu verschlechtern, indem man unachtsam mit Verschachtelungen durch If-Anweisungen umgeht.
Die folgende Methode soll eine Pizza-Bestellung anhand eines Parameter-Objekts verarbeiten. Natürlich wird die Bestellung nur verarbeitet, wenn die Parameter plausibel sind.
Soweit so gut. Aber wir wollen natürlich nur solche Eingaben verarbeiten, bei denen auch tatsächlich etwas bestellt wird.
Nun fällt uns jedoch auf, dass wir auch die Fahrer nicht unnötig alarmieren sollten, wenn es keine Lieferung gibt. Ebenso sollten wir kein Geld abbuchen, wenn keine Bestellung erfolgt.
Besser. Aber wenn keine Adresse vorliegt, können wir nicht liefern, und die Bestellung sollte nicht weiterverarbeitet werden.
Wie Sie sehen, hat die Vielzahl an Verschachtelungen unseren Code erheblich schwerer lesbar gemacht. Wir könnten dies beheben, indem wir alle Bedingungen in eine einzige Abfrage zusammenfassen, aber das würde die individuelle Behandlung von Fehlern verhindern.
Zum Glück gibt es eine einfache Möglichkeit, die Komplexität dieser Methode drastisch zu reduzieren. Indem wir die If-Bedingungen invertieren und die Methode bei Nichterfüllen der Bedingung sofort verlassen, können wir den produktiven Code ausführen, ohne ihn zu verschachteln.
Schon haben wir eine Ebene der Verschachtelung vermieden.
Viel besser! Es wird deutlich, dass dieses Verhalten die Funktion auf natürliche Weise in zwei Hälften geteilt hat.
Die Validierung könnte nun in eine eigene Funktion ausgelagert werden, aber dies ist das Thema für einen weiteren Artikel.
Unser Geschäftsführer Tibor Csizmadia und unser Kundenbetreuer Jens Walter stehen Ihnen persönlich zur Verfügung. Profitieren Sie von unserer langjährigen Erfahrung und erhalten Sie eine kompetente Erstberatung in einem unverbindlichen Austausch.
