Reinhard Kern hat folgendes geschrieben :

markieren Delphi-Quelltext     1: 2: 3: 4: {schreibe ich lieber} if not FileExists (..) then exit; if not FileIsOpen (..) then exit; if not Assigned (..) then exit;

In dem einen Fall ist es ja ok, aber schreib mal das:



Auch wenn man das Problem hier vielleicht nicht sofort erkennt und das Beispiel nicht besonders ist (auf die Schnelle fällt mir nicht besseres ein), aber wie leicht hat man bei verschachtelten ifs andere Ergebnisse als man dachte.

Dafür gibt es ja CnWizards oder andere (meist kommerzielle) Tools, die diese Fehler sofort sichtbar machen.
Zumindest, wenn man es ordentlich (und in dem Fall falsch) einrückt.

Und zudem automatische Codeformatter, die das auch sofort aufdecken.

Beides unter der Voraussetzung, dass man mit den Ergebnissen auch etwas anfangen kann.

Popov hat folgendes geschrieben :

In dem einen Fall ist es ja ok, aber schreib mal das: markieren Delphi-Quelltext     1: 2: 3: 4: if not FileExists (..) then   if not FileIsOpen (..) then exit else  (..); ...

Hallo,

wahrscheinlich ist es schon zu spät in der Nacht, aber vielleicht hilfst du mir auf die Sprünge (schreckliches Wort um die Uhrzeit) - ich komme nicht drauf, wann ein else nach einem exit sinnvoll ist. Nach einem "if Abbruchbedingung then exit" ist doch alles weitere sowieso der Else-Zweig?

Übrigens gerade wegen der Übersicht verschachtele ich die Ifs eben nicht, im Gegensatz zur Lehrmeinung. Meine bevorzugte Struktur ist


Ist praktiziertes KISS-Prinzip.

Gruss Reinhard

exit;