Ist es eine schlechte Praxis, eine if-Anweisung ohne geschweifte Klammern zu verwenden? [geschlossen]

Ich habe Code wie diesen gesehen:

if(statement)
    do this;
else
    do this;

Ich denke jedoch, dass dies besser lesbar ist:

if(statement){
    do this;
}else{
    do this;
}

Da beide Methoden funktionieren, ist dies einfach eine Frage der Präferenz, die verwendet werden soll, oder wird ein Weg dem anderen empfohlen?

Das Problem mit der ersten Version besteht darin, dass Ihr Code auf unerwartete und amüsante Weise unterbrochen wird, wenn Sie zurückgehen und den if- oder else-Klauseln eine zweite Anweisung hinzufügen, ohne daran zu denken, die geschweiften Klammern hinzuzufügen.

In Bezug auf die Wartbarkeit ist es immer klüger, die zweite Form zu verwenden.

EDIT: Ned weist in den Kommentaren darauf hin, aber ich denke, es lohnt sich auch, hier darauf zu verlinken. Dies ist nicht nur ein hypothetischer Bullshit aus dem Elfenbeinturm: https://www.imperialviolet.org/2014/02/22/applebug.html

Ein Problem beim Weglassen von Anweisungsblöcken ist die else-Ambiguität. Das heißt, C-inspirierte Sprachen ignorieren Einrückungen und haben daher keine Möglichkeit, dies zu trennen:

if(one)
    if(two)
        foo();
    else
        bar();

Davon:

if(one)
    if(two)
        foo();
else
    bar();

Mein allgemeines Muster lautet: Wenn es in eine Zeile passt, mache ich Folgendes:

if(true) do_something();

Wenn es eine else-Klausel gibt oder wenn der Code, für den ich ausführen möchte, eine trueerhebliche Länge hat, setzen Sie die Klammern ganz durch:

if(true) {
    do_something_and_pass_arguments_to_it(argument1, argument2, argument3);
}

if(false) {
    do_something();
} else {
    do_something_else();
}

Letztendlich kommt es auf ein subjektives Problem von Stil und Lesbarkeit an. Die allgemeine Programmierwelt teilt sich jedoch ziemlich stark in zwei Parteien (für Sprachen, die geschweifte Klammern verwenden): Verwenden Sie sie entweder ausnahmslos die ganze Zeit oder mit Ausnahmen die ganze Zeit. Ich bin Teil der letzteren Gruppe.

Ich verwende den Code-Formatierer der von mir verwendeten IDE. Das kann abweichen, kann aber in den Einstellungen / Optionen eingerichtet werden.

Ich mag dieses:

if (statement)
{
    // comment to denote in words the case
    do this;
    // keep this block simple, if more than 10-15 lines needed, I add a function for it
}
else
{
    do this;
}

Die "Regel", der ich folge, lautet:

Wenn die "if" -Anweisung getestet wird, um etwas zu tun (IE-Aufruffunktionen, Konfigurieren von Variablen usw.), verwenden Sie geschweifte Klammern.

if($test)
{
    doSomething();
}

Dies liegt daran, dass Sie meiner Meinung nach klarstellen müssen, welche Funktionen aufgerufen werden und wohin der Programmfluss unter welchen Bedingungen geht. Es ist wichtig, dass der Programmierer genau versteht, welche Funktionen aufgerufen werden und welche Variablen in diesem Zustand festgelegt sind, damit er genau versteht, was Ihr Programm tut.

Wenn die "if" -Anweisung getestet wird, um etwas zu stoppen (IE-Flusssteuerung innerhalb einer Schleife oder Funktion), verwenden Sie eine einzelne Zeile.

if($test) continue;
if($test) break;
if($test) return;

In diesem Fall ist es für den Programmierer wichtig, schnell zu erkennen, in welchen Ausnahmefällen der Code nicht ausgeführt werden soll. Dies wird in $ test und nicht im Ausführungsblock behandelt.

Wenn Sie die Zahnspange vom ersten Moment an haben, sollten Sie verhindern, dass Sie dies jemals debuggen müssen:

if (statement)
     do this;
else
     do this;
     do that;

Verwenden Sie geschweifte Klammern für alle if-Anweisungen, auch für einfache. Oder schreiben Sie eine einfache if-Anweisung neu, um den ternären Operator zu verwenden:

if (someFlag) {
 someVar= 'someVal1';
} else {
 someVar= 'someVal2';
}

Sieht viel schöner aus:

someVar= someFlag ? 'someVal1' : 'someVal2';

Verwenden Sie den ternären Operator jedoch nur, wenn Sie absolut sicher sind, dass in den if / else-Blöcken nichts anderes enthalten sein muss!

Ich benutze lieber Zahnspangen. Das Hinzufügen von geschweiften Klammern erleichtert das Lesen und Ändern.

Hier sind einige Links für die Verwendung von Zahnspangen:

• Bevorzugen Sie Multiline, wenn

• Zahnspangen weglassen: Nicht nur eine Frage des Stils

• Paketüberfluss

Nach meiner Erfahrung ist der einzige (sehr) geringfügige Vorteil des ersten Formulars die Lesbarkeit des Codes, das zweite Formular fügt "Rauschen" hinzu.

Bei modernen IDEs und Code-Autogeneration (oder Autocompletion) empfehle ich jedoch dringend, das zweite Formular zu verwenden. Sie werden keine zusätzliche Zeit damit verbringen, geschweifte Klammern einzugeben, und Sie werden einige der häufigsten Fehler vermeiden.

Es gibt genug energieaufwendige Fehler, die Leute sollten einfach keine Türen für große Zeitverschwendung öffnen.

Eine der wichtigsten Regeln beim Schreiben von Code ist die Konsistenz. Jede Codezeile sollte auf die gleiche Weise geschrieben werden, egal wer sie geschrieben hat. Strenge Verhinderung verhindert, dass Fehler "auftreten";)

Dies gilt auch für die eindeutige und explizite Benennung Ihrer Variablen, Methoden, Dateien oder für das korrekte Einrücken ...

Wenn meine Schüler diese Tatsache akzeptieren, hören sie auf, gegen ihren eigenen Quellcode zu kämpfen, und beginnen, das Codieren als eine wirklich interessante, anregende und kreative Aktivität zu betrachten. Sie fordern ihren Verstand heraus, nicht ihre Nerven!

Es ist eine Frage der Präferenz. Ich persönlich verwende beide Stile. Wenn ich mir ziemlich sicher bin, dass ich keine weiteren Anweisungen hinzufügen muss, verwende ich den ersten Stil, aber wenn dies möglich ist, verwende ich den zweiten. Da Sie dem ersten Stil keine weiteren Aussagen hinzufügen können, habe ich gehört, dass einige Leute davon abraten, ihn zu verwenden. Bei der zweiten Methode ist jedoch eine zusätzliche Codezeile erforderlich. Wenn Sie (oder Ihr Projekt) diese Art von Codierungsstil verwenden, wird die erste Methode für einfache if-Anweisungen sehr bevorzugt:

if(statement)
{
    do this;
}
else
{
    do this;
}

Ich denke jedoch, dass die beste Lösung für dieses Problem in Python ist. Mit der Whitespace-basierten Blockstruktur haben Sie nicht zwei verschiedene Methoden zum Erstellen einer if-Anweisung: Sie haben nur eine:

if statement:
    do this
else:
    do this

Während dies das "Problem" hat, dass Sie die geschweiften Klammern überhaupt nicht verwenden können, erhalten Sie den Vorteil, dass der erste Stil keine Zeilen mehr enthält und die Möglichkeit besteht, weitere Anweisungen hinzuzufügen.

Ich habe immer versucht, meinen Code zum Standard zu machen und ihm so nahe wie möglich zu kommen. Dies erleichtert anderen das Lesen, wenn sie für die Aktualisierung verantwortlich sind. Wenn Sie Ihr erstes Beispiel machen und in der Mitte eine Zeile hinzufügen, schlägt dies fehl.

Funktioniert nicht:

if (Anweisung) dies tun; und das; sonst mach das;

Persönlich verwende ich den ersten Stil nur, um eine Ausnahme auszulösen oder vorzeitig von einer Methode zurückzukehren. Wie Argument Prüfen am Anfang einer Funktion, weil ich in diesen Fällen selten mehr als eine Sache zu tun habe und es nie eine andere gibt.

Beispiel:

if (argument == null)
    throw new ArgumentNullException("argument");

if (argument < 0)
    return false;

Ansonsten benutze ich den zweiten Stil.

Meine persönliche Präferenz ist die Verwendung einer Mischung aus Leerzeichen und Klammern wie folgt:

if( statement ) {

    // let's do this

} else {

    // well that sucks

}

Ich denke, das sieht sauber aus und macht meinen Code sehr einfach zu lesen und vor allem - Debuggen.

Ich stimme den meisten Antworten darin zu, dass es besser ist, in Ihrem Code explizit zu sein und geschweifte Klammern zu verwenden. Persönlich würde ich eine Reihe von Codierungsstandards übernehmen und sicherstellen, dass jeder im Team sie kennt und sich anpasst. Wo ich arbeite, verwenden wir von IDesign.net veröffentlichte Codierungsstandards für .NET-Projekte.

Ich ziehe es vor, eine geschweifte Klammer zu setzen. Aber manchmal hilft der ternäre Operator.

Anstatt :

int x = 0;
if (condition) {
    x = 30;
} else {
    x = 10;
}

Man sollte einfach tun: int x = condition ? 30 : 20;

Stellen Sie sich auch einen Fall vor:

if (condition)
    x = 30;
else if (condition1)
    x = 10;
else if (condition2)
    x = 20;

Es wäre viel besser, wenn Sie die geschweifte Klammer einsetzen.