Ist mit einem 'goto' - Anweisung schlecht?

EDIT:

Wie schlimm ist die goto-Anweisung wirklich, und warum?

Hängt es von der genauen situation. Ich kann mich nicht erinnern, jedes mal wo ich es gemacht, den code lesbarer als refactoring. Es hängt auch von Ihrer persönlichen Sicht auf die Lesbarkeit - einige Leute mögen es mehr als andere, wie aus den anderen Antworten. (Als ein Punkt des Interesses, es ist weit verbreitet in generiert code - all das async/await-code in C# 5 basiert auf effektiv eine Menge von gotos).

Das problem ist, dass Situationen, in denen goto eher neigen zu sein, die Art von Situationen, wo refactoring aids-Dinge, die ohnehin - in der Erwägung, dass goto - sticks mit einer Lösung, die schwieriger zu Folgen, als der code wird komplizierter.

Ist es ein effektiver Weg, um zu brechen die main-loop als mit der 'goto' - Anweisung?

Absolut. Extrahieren Sie Ihre Methode in eine separate Funktion:

while (ProcessValues(...))
{
    //Body left deliberately empty
}

...

private bool ProcessValues()
{
   for (int i = 0; i < 15; i++)
   {
       //Do something
       return false;
   }
   return true;
}

Ich in der Regel bevorzugen, tun dies über die Einführung eines zusätzlichen lokalen Variablen zu verfolgen "habe ich fertig" - obwohl, das wird funktionieren, natürlich.

Wie schlimm ist die goto-Anweisung wirklich, und warum?

Ist es wirklich schlecht für die normalen Gründe. Es prefectly in Ordnung, wenn die Emulation beschriftet Schleifen in Sprachen, die diese nicht unterstützen.

Ersetzen Sie es mit Funktionen wird in vielen Fällen scatter-Logik, die wirklich gelesen werden sollte, da die gleiche Einheit. Dies macht es schwieriger zu Lesen.
Niemand mag es, Folgen Sie den Spuren von Funktionen, die nicht wirklich etwas, bis am Ende der Reise, wenn Sie etwas vergessen
wo aus Sie gestartet sind.

Ersetzen Sie es mit booleans und eine Reihe weiterer ifs und Pausen ist einfach wirklich klobig und macht es schwieriger zu Folgen, wahren Absichten, wie jedes Geräusch.

In java (und javascript), dies ist durchaus akzeptabel (mit Schlaufen):

outer: while( true ) {
    for( int i = 0; i < 15; ++i ) {
        break outer;
    }
}

In C#, es sieht aus wie eine sehr enge Entsprechung nicht:

while( true ) {
   for (int I = 0; I < 15; I++) { 
       goto outer;
   }
}
outer:;

Weil das Wort goto, das hat einen psychologischen Effekt, dass Menschen ablegen, die alle Ihren gesunden Menschenverstand und machen Sie link xkcd unabhängig vom Kontext.

Ist es ein effektiver Weg, um zu brechen die main-loop als mit der 'goto' - Anweisung?

In einigen Fällen gibt es nicht, weshalb die anderen Sprachen beschriftet loops und C# bietet goto. Beachten Sie, dass Ihr Beispiel ist zu einfach und es macht
die Workarounds nicht zu schlecht aussieht, weil Sie sind zugeschnitten auf die Beispiel. In der Tat, ich könnte genauso gut vorschlagen, diese:

   for (int I = 0; I < 15; I++) {
       break;
   }

Wie wäre es damit:

int len = 256;
int val = 65536;

for (int i = 0; i < len; i++)
{
    for (int j = 0; j < len; j++)
    {
        if (i + j >= 2 * val)
        {
            goto outer;
        }
        val = val / 2;
    }
}
outer:;

Tut dies immer noch gut Aussehen auf Sie:

int len = 256;
int val = 65536;

for (int i = 0; i < len; i++)
{
    if (!Inner(i, ref val, len))
    {
        break;
    }
}

private bool Inner(int i, ref int val, int len)
{
    for (int j = 0; j < len; j++)
    {
        if (i + j >= 2 * val)
        {
            return false;
        }

        val = val / 2;
    }

    return true;
}

Stimme ich mit der Mehrheit der Antworten darüber, wie schlecht ist springen.

Meine sugestion zu vermeiden, springen ist etwas wie dieses:

while (condition) { //Main Loop
   for (int i = 0; i < 15; i++) { //Secondary loop
       //Do Something
       if(someOtherCondition){
              condition = false //stops the main loop
              break; //breaks inner loop
       }
   }
}

Ich persönlich halte goto als "goto" führt in die Hölle"...und in vieler Hinsicht ist dies wahr, wie es kann dazu führen, sehr wartbaren code und wirklich schlechte Praktiken.

That being said, es ist immer noch umgesetzt werden für einen Grund, und wenn verwendet, sollte es sparsam verwendet werden, wenn KEINE andere Lösung verfügbar ist. OO-Sprachen bieten sich auch nicht wirklich brauchen (viel).

Das einzige mal, das ich JEMALS sehen, es verwendet in diesen Tagen ist in der Verschleierung...ein goot Beispiel für die Verwendung von goto zu erstellen, die code aus der Hölle, damit die Leute abgeschreckt werden versuchen, es zu verstehen!

Einige Sprachen angewiesen auf entsprechende keywords. Zum Beispiel in x86-assember haben Sie keywords wie JMP (Sprung), JE (Jump if Equal) und JZ (Jump if Zero). Diese sind erforderlich, Häufig in Assembler, da gibt es kein OO auf dieser Ebene, und es gibt paar andere Methoden zum bewegen in einer Anwendung.

AFAIK...bleiben Sie Weg von es, es sei denn ABSOLUT NOTWENDIG.