Wie man mehrere foreach Schleife bricht?

Question

Ich habe vier foreach-Schleifen, die durch die Sammlungen iterieren und basierend auf einer Bedingung etwas tun. Hier ist der Code, den ich jetzt schreibe:

boolean breakFlag = false; 
String valueFromObj2 = null; 
String valueFromObj4 = null; 
for(Object1 object1: objects){ 
    for(Object2 object2: object1){ 
    //I get some value from object2 
    valueFromObj2 = object2.getSomeValue(); 
    for(Object3 object3 : object2){ 
     for(Object4 object4: object3){ 
     //Finally I get some value from Object4. 
     valueFromObj4 = object4.getSomeValue(); 
     //Compare with valueFromObj2 to decide either to break all the foreach loop 
     breakFlag = compareTwoVariable(valueFromObj2, valueFromObj4); 
     if(breakFlag){break;} 
     }//fourth loop ends here 
     if(breakFlag){break;} 
    }//third loop ends here 
    if(breakFlag){break;} 
    }//second loop ends here 
    if(breakFlag){break;} 
}//first loop ends here 

Das Hauptobjekt (Objekte im Code) stammt von einem Drittanbieter SDK, so kann ich nichts auf diesem Abschnitt ändern. Ich möchte die Stackoverflow-Community fragen, ob es einen besseren Ansatz gibt, alle vier foreach-Schleifen zu durchbrechen. Oder wenn es eine andere Möglichkeit gibt, diesen Code umzuformulieren, um ihn lesbarer und wartbarer zu machen. Vielen Dank.

Answer 1

Verwenden Sie ein Etikett auf der äußersten Schleife und fügen Sie dieses Etikett in die Anweisung break ein, wenn Sie aus allen Schleifen springen möchten. Im Beispiel unten habe ich Ihren Code modifiziert, um das Etikett zu verwenden OUTERMOST:

String valueFromObj2 = null; 
String valueFromObj4 = null; 
OUTERMOST: for(Object1 object1: objects){ 
    for(Object2 object2: object1){ 
    //I get some value from object2 
    valueFromObj2 = object2.getSomeValue(); 
    for(Object3 object3 : object2){ 
     for(Object4 object4: object3){ 
     //Finally I get some value from Object4. 
     valueFromObj4 = object4.getSomeValue(); 
     //Compare with valueFromObj2 to decide either to break all the foreach loop 
     if(compareTwoVariable(valueFromObj2, valueFromObj4)) { 
      break OUTERMOST; 
     } 
     }//fourth loop ends here 
    }//third loop ends here 
    }//second loop ends here 
}//first loop ends here 

Answer 2

Eine Ausnahme auslösen und außerhalb der Schleifen abfangen? Verwenden Sie etwas, das als "schädlich" angesehen wird?

Es ist ein bisschen komisch, wenn die Informatik selbst in eine Ecke malt ;-)

Answer 3

Eine Möglichkeit, mehrere Anweisungen (eigentlich Stack Frames) zu brechen, oder zu kollabieren, eine Ausnahme zu werfen, aber das, weil es ist nicht wirklich zu empfehlen teuer für die Laufzeit, um den Stapel abzuwickeln und es könnte zu wirklich unangenehm schwierig, undefined Verhalten zu debuggen, (dies zu beachten).

Ansonsten, was ich empfehle, schreiben Sie diesen Code um in der Lage, aus der Schleife in einer anmutigen Weise zu brechen. Wenn Sie diesen Code nicht anders ändern können, dann müssen Sie zu Ausnahmen führen ...

Answer 4

Extrahieren Sie alle Schleifen in die Funktion und verwenden Sie return.

Answer 5

Sie eine markierte break-Anweisung verwenden könnte. Diese Art von Bruch endet eine äußere Aussage

Siehe The break Statement

Answer 6

Die einfachste Lösung ist, den gesamten Suchprozess in einem Verfahren zu setzen und return, sobald Sie eine Antwort haben.

Die abstrakte Form Ihres Beispielcodes lässt jedoch einige andere Möglichkeiten in Frage. Gibt es beispielsweise eine Möglichkeit, einen Teil des Inhalts zu "indexieren" (z. B. mithilfe von Map Instanzen), sodass Sie keine Brute-Force-Schleifen verwenden müssen?

Answer 7

Abgesehen von der Tatsache, dass Java markiertes Break unterstützt, siehe auch Is it possible to exit a for before time in C++, if an ending condition is reached?, was eine ähnliche Frage mit einigen anderen relevanten Lösungen ist.

Answer 8

Sehen Sie die Branching Statements Java Tutorial für den einfachsten Weg, mit einem Etikett. Sie können beliebige oder alle for-Schleifen beschriften und dann break oder continue in Verbindung mit diesen Beschriftungen verwenden.

Eine Alternative zur Verwendung von Etiketten ist stattdessen return zu verwenden. Refactorieren Sie Ihren Code einfach in einen Methodenaufruf, um die Notwendigkeit der Verwendung von Labels zu umgehen.

Answer 9

Ihr Beispiel ist ziemlich generisch, so dass es schwer ist zu sagen, was los ist, aber ich bekomme so einen starken Code-Geruch aus dem Code, den Sie haben, dass ich denken muss, dass es eine andere Möglichkeit gibt, das Ding komplett zu machen , wahrscheinlich durch Refactoring der eigentlichen Datenstruktur zu etwas Sinnvollerem.

Welche Art von Liste objects ist? Was andere (sehr wahrscheinlich wichtig) Daten enthält es? Wenn es nicht zu viel Aufwand ist, würde ich mich freuen, wenn Sie relevanteren Code zur Verfügung stellen würden, da der Refaktor in mir völlig schwindlig wird, nur weil er diesen Haufen von Schleifen sieht.