Mich würde eine Meinung zu nachfolgendem Streit interessieren.

Programmierstil bei einer nicht erfüllten If - Bedingung.




oder





Ich meine das die 2. Anweisung übersichtlicher ist, unser Objectguru meint, das exit ein verstecktes goto und deshalb böse ist.

Gruß
Carla

Moderiert von Christian S.: Delphi-Tags hinzugefügt

Erstmal ist der Programmierstil bei beiden Beispielen schlecht, weil du den Quelltext nicht in die Delphi-Tags [delphi]...[/delphi] eingeschlossen hast (Christian S. war so freundlich und hat deinen Post mittlerweile verbessert).

Also ich würde weder die erste noch die zweite Variante nehmen:

Allerdings benutze ich auch in manchen Fällen deine zweite Variante, vorallem wenn in den entstehenden If-Blöcken sehr viel Code steht.

Ich würd den dritten Weg gehen,



Ob exit böse ist hängt meiner Meinung nach davon ab, wie gross und übersichtlich die restliche Funktion ist. Wenn das in einer unübersichtlichen Funktion irgendwo in der 3. Schachteltiefe steckt, würd ich Teer und Federn auspacken, wenn die Funktion ansonsten nen 10-Zeiler ist find ichs ok.

Allgemein sollte man Exit nicht so oft benutzen. Wenn ganz am Anfang der Prozedure eine Bedingung abgefragt wird, die für die Ausführung unbedingt erfüllt sein muss, dann hab ich in der ersten Zeile schonmal ein Exit drin. Dann spart man sich für die ganze Prozedur eine begin-end-Ebene. Ansonsten versuche ich, sowas zu vermeiden - das wird nämlich schnell unübersichtlich.

exit hat zwar die Wirkung eines gotos, aber mit der wichtigen Einschränkung, dass immer nur ans Ende der Routine gesprungen wird und nicht wild im Quellcode hin und her. Und ich finde es schon übersichtlicher, denn dann weiß ich gemau, was passiert: Der restluiche Code wird nicht mehr abgearbeitet. Ohne exit müsste man den ganzen Code durchgucken, um zu sehen, was bei nicht erfüllen der Bedingung passiert.

Erst mal danke für die Delphi Tags. Das nächste mal weis ich es.

Der Grund für die Frage.
Ich passe ein Delphiprojekt an und da kommt in einer procedure nach den zwei if Abfragen, welche eine Eingangsbedingung
darstellen, 3 Seiten Quellcode mit mit vielen Beginn - end und try - finally Blöcken.

Alternativ könnte man ja die Eingangsbedingung in den Aufruf unterbringen?

Also




Carla

das halte ich für eine übersichtiliche Methode...
Zumindest, wenn die Abfrage ziwschen vielen anderen Prozeduren steht...
dann kann man den Kernblock am Anfang oder an das Ende des Programms schreiben... oder extern, in einer DLL, wenn es wirklich sehr viel Code ist, ist es sicher nützlich, die Prozedur in eine DLL zu schreiben, dann ist der ganze Text erstmal weg...

Drei Seiten Quellcode sind zu viel, da hat mankeine Übersicht mehr. Teile den Quellcode in einzelne Routinen auf.

Ja, ist dann aber trotzdem noch viel Text... würde dann für die Routinen DLLs oder neue Units machen

Wolle92 hat folgendes geschrieben:

Ja, ist dann aber trotzdem noch viel Text... würde dann für die Routinen DLLs oder neue Units machen

Der Text ist nicht von mir, ich muss "nur" Anpassungen vornehmen.
Das Programm ist übrigens schon eine dll.

Gruß
Carla