Macht das Konstrukt .. während (falsch) dazu beitragen, den Fluss besser zu kontrollieren?

Question

Ich habe über diesen Code vor kurzem kommen:

do { 
    if (! checkSomething()) 
     break; 

    // some code 

    if (! checkSomeOtherThing()) 
     break; 

    // some other code 
} while(false); 

// some final code 

Der Programmierer, der es geschrieben hat, um einen Kommentar entlang der Linien von "cleaner control flow" schrieb.

Meiner Meinung nach könnte der ursprüngliche Code besser aussehen, wenn er in etwas anderes umgestaltet wird. Aber gibt es eine Wahrheit in dieser Aussage? Ist dieses Konstrukt gut?

Answer 1

Wenn Sie nichts tun Schleifen mit mehreren break Aussagen, dann das einzige Problem hier ist, dass C (aus offensichtlichen Gründen) Sie nicht aus einem bloßen Block ausbrechen lassen, daher die "Non-Loop", die einige ahnungsloser zukünftiger Betreuer könnte Fehler für eine echte Schleife machen.

Die Überlegungen, glaube ich, sind:

• wenn es nur zwei break Punkte sind, was ist so schlecht über zwei if Aussagen?
• Wenn es mehr als zwei Haltepunkte gibt, dann kann der Eindruck mit if Anweisungen unangenehm werden, und das spart das, aber dann wieder ist die Funktion zu viel? Und selbst wenn nicht, wäre es besser, einfach goto zu verwenden und die Verrücktheit einer Schleife zu vermeiden, die keine Schleife macht?

Da Sie diese sprachunabhängig Tag, habe ich eine macroised Assemblersprache zu bedienen, mit einem block ... endblock, die Sie aus brechen könnte. Dies führte zu ziemlich nett Code für die notwendigen Bedingungen überprüft, wie zB:

block 
    breakif str1 == null 
    breakif str2 == null 
    get some combined property of str1 and str2 
    breakif some other condition that stops us getting on with it 
    get on with it 
endblock 

Eigentlich war es nicht breakif str1 == null, war es breakifeq.p str1, null, oder so etwas, aber ich vergesse genau das, was.

Answer 2

Ich finde das viel einfacher zu lesen, und es erzeugt ein identisches Ergebnis:

if (checkSomething()) 
{ 
    // some code 
    if (checkSomeOtherThing()) 
    { 
     // some other code 
    } 
} 
// some final code 

Ich denke do ... while normalerweise schwer zu folgen, wobei jedoch ist es für etwas anderes als eine Schleife bestenfalls irreführend.

Answer 3

Dies entspricht einer goto.

In solchen Situationen ist es besser, einen goto als einen hässlichen Hack zu verwenden.

es ein goto macht es viel besser lesbar zu verwenden, ändern:

if (!checkSomething()) 
    goto Done; 

// some code 

if (!checkSomeOtherThing()) 
    goto Done; 

// some other code 
Done: //some final code 

Answer 4

Ich habe die do-while gesehen Form als Standard übernommen, an die Programmierer angepasst. Der Vorteil ist, dass es kommuniziert und implementiert, dass die Schleife immer mindestens einmal ausgeführt wird. Dies hilft, konsistent die Situationen zu isolieren, in denen etwas anderes auftritt, d. H. Wo der Code in der Schleife nicht ausgeführt wird.

Dieser Standard wurde angenommen, weil die Warnier-Orr-Technik angewendet wurde.