Wartungstechnisch gesehen, gilt "else while" ohne eingreifende Zahnspangen als sicher?

Gilt die else whileWartung ohne dazwischenliegende Zahnspangen als "sicher"?

Schreiben von if-elseCode ohne geschweifte Klammern wie unten ...

if (blah)
    foo();
else
    bar();

... birgt ein Risiko, da das Fehlen von geschweiften Klammern es sehr einfach macht, die Bedeutung des Codes versehentlich zu ändern.

Ist unten aber auch riskant?

if (blah)
{
    ...
}
else while (!bloop())
{
    bar();
}

Oder gilt else whileohne dazwischenliegende Zahnspange "sicher"?

Das erinnert mich an diesen Code:

if ( ... ) try {
..
} catch (Exception e) {
..
} else {
...
}

Jedes Mal, wenn Sie zwei Arten von Blöcken kombinieren, geschweifte Klammern vergessen und die Einrückung nicht erhöhen, erstellen Sie Code, der nur schwer zu verstehen und zu warten ist.

Vielleicht liegt es daran, dass ich meinen Beruf (vor langer Zeit ) mithilfe der Jackson-Entity-Structure-Diagram- Methode gelernt habe , aber ich bin der Ansicht, dass der einzig richtige nicht umrahmte Begriff nach einem ifoder einem elseein nachfolgender ist if(dh eine else ifLeiter zulassen ).

Alles andere (kein Wortspiel beabsichtigt) kann zu Missverständnissen und / oder Wartungsproblemen führen. Das ist der zentrale Aspekt der Idee des OP, "unsicher" zu sein.

Ich würde auch sehr käfig sein, wenn ich das while()in die gleiche Zeile wie das else- ob verspannt oder nicht - aufnehmen würde. Es liest sich für mich nicht richtig ... das Fehlen zusätzlicher Einrückungsmasken, die es ist, ist die elseKlausel. Und mangelnde Klarheit führt zu Missverständnissen (so).

In diesem Beispiel würde ich (und in meinem Team darauf bestehen) dringend Folgendes empfehlen:

if ( blah )
{
    ...
}
else
{
    while ( !bloop() )
    {
        bar();
    }
}

Natürlich würde ich auch passende Kommentare erwarten.

- Bearbeiten -

Vor kurzem litt Apple an einer SSL-Sicherheitsanfälligkeit, die durch einen schlechten Wartungsfix verursacht wurde und eine zweite Zeile zu einer nicht geschweiften einzelnen Zeile hinzufügte. Lassen Sie uns die Idee zu Bett bringen, dass ungerahmte einzelne Linien in Ordnung sind?

Mir wurde immer beigebracht, alles in Klammern zu halten, eingerückt und kommentiert. Ich glaube, es ist viel einfacher, Fehler zu lesen und zu erkennen. Persönlich denke ich, dass Modularität der Schlüssel ist, also würde ich den Code immer so schreiben:

    if(blah)
    {
     ....
    }
    else
    {
       while(!bloop()
       {
        bar;
       }
    }

Ich würde Methode extrahieren und es machen

if ( blah )
{
    ...
}
else
{
   newMethod();
}

Siehe den auf der Refactoring-Katalog- Website erläuterten Ansatz "Methode extrahieren" :

Sie haben ein Codefragment, das zusammen gruppiert werden kann.

Verwandeln Sie das Fragment in eine Methode, deren Name den Zweck der Methode erklärt.

void printOwing() {
    printBanner();

    //print details
    System.out.println ("name:    " + _name);
    System.out.println ("amount    " + getOutstanding());
}

                                                                                                        

void printOwing() {
    printBanner();
    printDetails(getOutstanding());
}

void printDetails (double outstanding) {
    System.out.println ("name:    " + _name);
    System.out.println ("amount    " + outstanding);
}

Ich würde es als schlechte Angewohnheit betrachten, zu codieren. Manchmal funktioniert es einwandfrei und Sie werden kein Problem damit haben, den Code zu kompilieren und auszuführen. In anderen Fällen kann dies zu schwerwiegenden Fehlern führen, und Sie werden am Ende Stunden damit verbringen, diesen Fehler zu beheben.

Es wird immer empfohlen, Ihren Code zu modularisieren. Wenn Sie eine while-Schleife in einem anderen Teil einfügen müssen, setzen Sie sie in einen Block, damit andere beim Arbeiten an Ihrem Code die Logik leicht verstehen können.

if ( blah )
{
    ...
}
else
{
    while ( !bloop() )
    {
        bar();
    }
}

Es ist eine gute Praxis, Code in Blöcken zu platzieren. Es macht es einfacher und einfacher zu verstehen und zu debuggen.

Es könnte in Ordnung sein, wenn es so einfach bleibt, obwohl es mir persönlich nicht gefällt und ich es vorziehe, selbst für die einfachsten if / else-Codeblöcke geschweifte Klammern zu verwenden. Es ist nur ordentlicher für mich.

Dies ist jedoch problematisch, wenn Sie geschachtelt haben, wenn / else einige mit geschweiften Klammern, andere ohne, durchläuft. Eine While-Schleife mitten in all den Spaghetti! Ich habe mit so schlechtem Code gearbeitet und es ist ein Albtraum, Fehler zu beheben und zu verstehen. Dies folgt ab dem ersten Punkt. Es ist in Ordnung, wenn es einfach bleibt und Sie damit zufrieden sind, aber dann kommen andere Programmierer und fügen Dinge hinzu, wahrscheinlich ein if else innerhalb der while-Schleife. Wenn es an erster Stelle sauber geschrieben ist, ist die Wahrscheinlichkeit geringer, dass dies geschieht.

Es geht darum, Dinge klar zu machen, damit andere Programmierer auf einen Blick erkennen können, was richtig und was falsch ist. Schreiben Sie den Code so, dass das Muster richtig aussieht und nicht ruckelt.

Die Kehrseite davon ist, dass ich vielleicht einige Leute sehen könnte, die behaupten, dass dies in bestimmten Fällen gut liest. Wenn ich die Ressource habe, die ich für die weitere Verarbeitung benötige, während ich auf etwas warte, tue dies. Auch dann gibt es für mich keinen Unterschied darin, die while-Schleife im else-Block zu verschachteln.

Nun, obwohl alle über die Verwendung von Zahnspangen streiten und sie zu lieben scheinen, bevorzuge ich eigentlich das Gegenteil. Ich benutze Klammern nur, wenn ich muss, weil ich sie ohne besser lesbar und prägnant finde.

if (...)
   foo();
else while(...)
   bar();

... ich finde das wirklich " else while(...)" bemerkenswert lesbar! Es liest sich sogar wie normales Englisch! Aber ich denke, die Leute werden es alle seltsam finden, weil es gelinde gesagt ungewöhnlich ist.

Letztendlich neigen wir alle dazu, es mit Zahnspangen idiotensicher zu machen ... weil, na ja, wissen Sie.

Ich setze immer geschweifte Klammern ein, nur weil Sie in Eile möglicherweise eine weitere Zeile einfügen, sie einrücken, die geschweiften Klammern vergessen und sich am Kopf kratzen, was gerade passiert. Ich halte die öffnende Klammer in der gleichen Zeile, aber es spielt keine Rolle. In beiden Fällen ist es besser, sie zu haben.

if(blah) {
    foo();
}
else {
    bar();
}

Was ist so falsch an geschweiften Klammern, dass so viele Menschen versuchen, es zu vermeiden, sie zu schreiben?

Welches Problem löst genau else while?

Zahnspangen sind billig und gut, und sie machen die Code-Intention klar und deutlich im Gegensatz zu clever und witzig.

Zitieren der Unix-Philosophie:

Regel der Klarheit: Klarheit ist besser als Klugheit.

Da Wartung so wichtig und so teuer ist, schreiben Sie Programme so, als ob die wichtigste Kommunikation nicht an den Computer gerichtet wäre, auf dem sie ausgeführt werden, sondern an die Menschen, die den Quellcode in Zukunft lesen und pflegen werden (einschließlich Sie selbst).

Es ist nichts falsch daran

if (blah)
    foo();
else
    bar();

als wäre nichts falsch daran

if (blah) foo(); else bar();

unter den richtigen Umständen (Ihre Umstände können variieren, aber dieser bestimmte Stil wird am besten verwendet, wenn Sie viel sich wiederholenden Code haben - dann ist die Ansicht jeder Zeile wichtiger als der stilistische Einzug)

Aber wenn Sie einen Weg wählen, bleiben Sie dabei - Konsistenz ist König. Ihr zweites Beispiel ist also sehr schlecht, da der if-Ausdruck eckige Klammern für seine Anweisung enthält und die while-Anweisung nicht außerhalb, sondern innerhalb der eckigen Klammern für die else-Klausel stehen sollte.

außerdem habe ich diesen Code schon mal gesehen:

if (x) foo()
{
  bar();
}

und es wurde von der Kodierungsnorm nazi selbst geschrieben (der für alles auf Klammern bestand).

Moderne IDEs können einfach so konfiguriert werden, dass sie neu formatieren (einschließlich Entfernen oder Hinzufügen unnötiger Klammern) und / oder den Code beim Speichern einer Datei erneut einblenden. So würde Ihr Beispiel zum Beispiel automatisch aussehen

if (blah) {
  blub();
} else
  while (!bloop()) {
    bar();
  }

Durch die Einkerbung ist die Verschachtelung auch ohne unnötige Klammern immer gut sichtbar. Wenn Sie es also schaffen, solche IDE-Einstellungen durchzusetzen, sehe ich kein Risiko.

Persönlich finde ich Code, der die geschweiften Klammern und Zeilenumbrüche weglässt, viel lesbarer, da er Unordnung vermeidet:

if (blah) blub();
else while (!bloop()) bar();

Aber natürlich streiten sich viele Entwickler sehr gerne für immer und einen Tag über solche Dinge.