Vorzeitiges Verlassen einer Funktion/Schleife mit Return - Guter Programmierstil oder nicht?

In dem Beispiel oben ist definitiv Switch ordentlicher.
Normalerweise verlässt man die Funktion nur vor dem eigentlichen Ende mit Return, wenn ein Fehler auftritt (aka Return SetError(...)).

Und... Was denkst du, wie diese Schleife hier in einer tieferen Systemebene aussieht?

While $a < 5
$a += 1
WEnd

In C:

while(a < 5)
{
a++;
}

Und dann noch tiefer, in ASM:

loop_while_head:
cmp a,5
jae loop_while_foot
inc a
jmp loop_while_head
loop_while_foot:

(Keine Gewährleistung auf diesen ASM-Code! :D)

Letztendlich ist jede Konstruktion eigentlich nur JMP/GOTO.^^

lg

In diesem Fall wäre Switch definitiv die elegantere Lösung.
Außerdem solltest du Return benutzen um Werte zu übertragen und keine globalen Variablen ändern, oder zumindest nur in seltenen Fällen. (Das sollte bei dir eigentlich sowieso nicht funktionieren, da $satz außerhalb der Funktionen als lokal deklariert wurde, diese somit eigentlich keinen Zugriff darauf haben sollten.)

ExitLoop/Return etc. ist nicht direkt mit GoTo vergleichbar... In manchen Situationen ist es übersichtlicher und schneller eine Schleife direkt zu verlassen, wenn der nachfolgende Code in diesem Moment sowieso nicht benötigt wird. Wenn du aber eine Schleife hast, die sowieso immer komplett durchlaufen werden muss und nur von einer Bedingung beendet wird, macht es mehr Sinn die Schleifenbedingung entscheiden zu lassen (also z.B. "While $bTest = True).
Solange du nicht wild durch seltsam verschachtelte Schleifen springst oder dein bestes gibst GOTO, mit ExitLoop und ContinueLoop, wieder zum Leben zu erwecken, solltest du diese Befehle verwenden können, ohne dass dir jemand deswegen auf die Füße tritt ;).
Ich finde das hängt von der Situation ab (und auch ein bisschen von deiner eigenen Meinung ^^).
ExitLoop/Return/ContinueLoop springen immer zu bestimmten logischen Punkten im Code die du nur durch umliegende Mechanismen definieren kannst. Bei GOTO setzt du einfach Sprungmarken und kannst zu jeder beliebigen Zeile im Code springen die du direkt im Befehl definierst.

Zu deiner zweiten Frage: In diesem Fall ist die zweite Variante mit "Switch" nicht nur (meiner Meinung nach) sauberer, sondern auch geringfügig schneller.

Zitat von Friesel

Meiner Meinung nach ist in dem konkreten Beispiel oben und auch bei meinem Programm mit FileInstall der Vorteil, dass man nach dem Return nur den Code hinzufügen muß, der neu ist und nicht wie bei Switch/Case jedesmal den gesamten Code wiederholen muß.

Ich würde auch lieber das mit if und return nehmen. Switch ist nur bei entweder-oder Auswahlen wirklich sinnvoll, bei aufeinanderfolgenden Abfragen wird if verwendet.
Mit Switch könnte man das auch so lösen: (unübersichtlich)

Zitat von Friesel

Hast du das gemessen oder weiß man sowas einfach?

Bei den if-abfragen muss jedes Mal der Wert der Variable neu ausgelesen werden und dafür muss in der Liste aller Variablen nachgeschlagen werden, welcher Wert dazu gehört steht.
Bei Switch wird der Wert einmal bestimmt und dann nur noch verglichen.

EDIT:

Zitat von chesstiger

Letztendlich ist jede Konstruktion eigentlich nur JMP/GOTO.^^

Ja klar, aber letztendlich ist goto/jmp unübersichtlich, weil man nicht sofort weiß, wann und warum gesprungen wird.
Hat man hingegen While da stehen, weiß man, dass man eine Schleife hat, die solange gilt, wie die Bedingung zutrifft.
Bei goto kann als Schleife, if-Abfrage, Exception und möglicherweise sogar als Funktionsaufruf bzw. -rücksprung fungieren, man sieht aber nicht sofort, was es sein soll und wann es auftritt.

Das Beispiel im Startpost ist schon ein Sonderfall.
Ich glaube da würde ich auch mit If ... then Return arbeiten.

Allgemein versuche ich das vorzeitige verlassen einer Funktion nurnoch fürs Errormanagment zu verwenden.
Wenn man bei jeder Gelegenheit aus den Funktionen springt ergibt das, gerade bei größeren Skripten/Funktionen, leicht Spaghetti-Code, der mit einigem Abstand nurnoch schwer lesbar ist.

Ein Beispiel was ich für sehr unsauber halte:

Func _Demo($i_zahl)

switch $i_tagl
case 1
Return $i_zahl
case 2
Return $i_zahl*'$i_zahl
endswitch
endfunc

besser:

Func _Demo($i_zahl)
local $ret
switch $i_tagl
case 1
$ret=$i_zahl
case 2
$ret=$i_zahl*'$i_zahl
endswitch
return $ret
endfunc

Marthog

Zitat von Friesel

Ist das nicht auch einfach nur ein "getarnter" GOTO-Befehl?

Ich hab mich darauf bezogen, um zu zeigen, das jede logische Struktur (If, Switch, Select, While, Do, For) darauf basiert.^^

lg

@James/Friesel

Zitat von James1337

Außerdem solltest du Return benutzen um Werte zu übertragen und keine globalen Variablen ändern, oder zumindest nur in seltenen Fällen. (Das sollte bei dir eigentlich sowieso nicht funktionieren, da $satz außerhalb der Funktionen als lokal deklariert wurde, diese somit eigentlich keinen Zugriff darauf haben sollten.)

Local außerhalb einer Funktion -> Global.

Ich höre auch oft GoTo sei schmieren Programmierung. Als ich dann mal nachgefragt habe wieso kommt eig. immer nur das Argument Übersicht des Quellcodes. Du weißt irgendwann nicht mehr wo wie was und wann passiert.

Ich arbeite zu 99% mit Schleifen und Switch.

So Far

Zitat von chesstiger

@James/Friesel

Local außerhalb einer Funktion -> Global.

Aber aber aber...
Ich dachte immer alle Variablen außerhalb von Funktionen sind Global, es sei den man verwendet Local. So langsam schwindet auch mein letztes bisschen Vertrauen in AutoIt...

Zitat von James1337

Ich dachte immer alle Variablen außerhalb von Funktionen sind Global, es sei den man verwendet Local.

Wo ist der Unterschied?
Vor welchem höheren Definitionsbereich willst du denn eine Variable mit Local verstecken wenn es darüber nichts mehr gibt?
Du kannst es gerne als Local definieren - ändert aber nichts daran dass es eine globale Variable ist.

Zitat von AspirinJunkie

Wo ist der Unterschied?

Man könnte zum Beispiel in Funktionen die selben Variablennamen verwenden, da man in Funktionen keinen Zugriff auf die Variablen hätte.

Exakt dafür verwendet man doch Local IN Funktionen...

Du möchtest außerhalb von Funktionen eine Variable als lokal definieren damit du in Funktionen (wahrscheinlich dann ohne Angabe von Local) denselben Namen vergeben kannst?
Das ist irgendwie von hinten durch die Brust ins Auge...

Zitat von AspirinJunkie

Du möchtest außerhalb von Funktionen eine Variable als lokal definieren damit du in Funktionen (wahrscheinlich dann ohne Angabe von Local) denselben Namen vergeben kannst?

Nein, ich verwende Local in und außerhalb von Funktionen. Es ging mir nur um den Fehlenden Unterschied zwischen Local und Global (außerhalb von Funktionen). Aber dann passt ja trotzdem alles, ich war nur kurzzeitig verwirrt.

Zitat von BugFix

Das kannst du doch. Wenn du eine Globale Variable $X hast und deklarierst in einer Funktion ebenfalls $X aber als Local, dann verwendet die Funktion die lokale Variable.

Danke, ich war nur etwas durch das sinnlose Local außerhalb von Funktionen verwirrt.

Zitat von BugFix

Ich persönlich halte es für nicht sehr glücklich identische Variablennamen in unterschiedlichen Scopes zu nutzen.

Naja, Variablen wie $i verwende ich zum Beispiel für fast jede For-Schleife...

BugFix Das ist wie vor C99:

int main(void)
{
int i;
for(i = 0; i < 10; i++)
{
printf("Zahl: %i\n", i);
}
printf("For-End: %i\n", i);
}

Aber ja: Lokale Variablen für sowas wären besser.

lg