7

こんにちは私はJavaの初心者で、私のプログラムには4つのforループがあります。私のプログラムは次のように機能します。の場合、要素bはtruepointListから削除されn、n-- すべてのforループから出て、最初からやり直したいと思います。 forループなので、どうすればlこれを行うことができますか？l++ブレークステートメントで？

for (int l = 0; l < n; l++) {
  for (int i = 1; i < (n - 2); i++) {
      for (int j = i + 1; j < (n - 1); j++) {
          for (int k = j + 1; k < n; k++) {
              if (l != i && l != j && l != k) {
                  boolean b = isOK(pointList.get(l), pointList.get(i), pointList.get(j), pointList.get(k));
                  System.out.println(b);
                  if (b == true) {
                      pointList.remove(pointList.get(l);
                      n--;
                      break;
                  }
                  else
                      System.out.println(b);
              }
           }
       }
    }
}
4

9 に答える 9

17

次のように、ラベル付きのブレークを使用できます。

      for (int l = 0; l < n; l++) {
 foo:    for (int i = 1; i < (n - 2); i++) {
            for (int j = i + 1; j < (n - 1); j++) {
                for (int k = j + 1; k < n; k++) {
                    if (l != i && l != j && l != k) {
                        boolean b = isOK(pointList.get(l), pointList.get(i), pointList.get(j), pointList.get(k));
                        System.out.println(b);
                        if (b == true) {
                            pointList.remove(pointList.get(l);
                            n--;
                            break foo;
                        }
                        else
                            System.out.println(b);
                    }

                }

            }
        }
    }
于 2010-11-18T11:01:03.170 に答える
5

ループでは、breakステートメントは内側のループを終了continueし、次の反復にスキップします。これらの2つのステートメントが内側のループとは異なるループで機能するためには、ラベルを使用する必要があります。このようなものが機能するはずです：

outerloop:      
        for (int l = 0; l < n; l++) {
            for (int i = 1; i < (n - 2); i++) {
                for (int j = i + 1; j < (n - 1); j++) {
                    for (int k = j + 1; k < n; k++) {
                        if (l != i && l != j && l != k) {
                            boolean b = isOK(pointList.get(l), pointList.get(i), pointList.get(j), pointList.get(k));
                            System.out.println(b);
                            if (b == true) {
                                pointList.remove(pointList.get(l);
                                n--;
                                continue outerloop;
                            }
                            else
                                System.out.println(b);
                        }

                    }

                }
            }
        }
于 2010-11-18T11:02:29.783 に答える
3

ラベル付きのbreakステートメントを見てください

たとえば、ここで：分岐ステートメント

于 2010-11-18T11:00:33.310 に答える
3 
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
于 2010-11-18T11:01:26.507 に答える
2

ラベル付きループを使用する

for (int l = 0; l < n; l++) {
    loopa:
    for (int i = 1; i < (n - 2); i++) {
        for (int j = i + 1; j < (n - 1); j++) {
            for (int k = j + 1; k < n; k++) {
                if (l != i && l != j && l != k) {
                    boolean b = isOK(pointList.get(l), pointList.get(i), pointList.get(j), pointList.get(k));
                    System.out.println(b);
                    if (b == true) {
                        pointList.remove(pointList.get(l);
                        n--;
                        break loopa;
                    }
                    else
                        System.out.println(b);
                }

            }

        }
    }
}

次に、ラベル付けされたループから抜け出します

于 2010-11-18T11:02:31.710 に答える
1 
again:
for (int l = 0; l < n; l++) {
        for (int i = 1; i < (n - 2); i++) {
            for (int j = i + 1; j < (n - 1); j++) {
                for (int k = j + 1; k < n; k++) {
                    if (l != i && l != j && l != k) {
                        boolean b = isOK(pointList.get(l), pointList.get(i),  pointList.get(j), pointList.get(k));
                        System.out.println(b);
                        if (b == true) {
                            pointList.remove(pointList.get(l);
                            n--;
                            break again;
                        }
                        else
                            System.out.println(b);
                    }

                }

            }
        }
    }
于 2010-11-18T11:06:23.753 に答える
1

私は他のすべての答えに同意します。ただし、代わりにexit、そのコードを独自のルーチンに配置し、returnステートメントを使用して全体から抜け出すこともできます。クワッドネストループはそれ自体が非常に複雑であるため、とにかく独自のルーチンに含める価値があります。

私は、1つのルーチンに対して6以下の循環的複雑度を必要とするDoDジョブに取り組んできました（一部の例外を除く）。この一連のループだけでも4です。もっと簡単な方法が見つからない場合は、このコードを維持しなければならない貧弱なシュマックの健全性を維持するためだけに、ループを独自のルーチンに組み込む必要があります。

于 2010-11-18T11:07:16.603 に答える
0

最初の「迅速で汚い」解決策は、stay_into_loops変数を使用して、次のforようにループを変更することです。

 boolean stay_into_loops = true
 // here goes the first for loop
 for (int i = 1; i < (n - 2) && stay_into_loops ; i++) {
            for (int j = i + 1; j < (n - 1) && stay_into_loops ; j++) {
                for (int k = j + 1; k < n && stay_into_loops ; k++) {
                    if (l != i && l != j && l != k) {
                        boolean b = isOK(pointList.get(l), `pointList.get(i), pointList.get(j), pointList.get(k));`
                        System.out.println(b);
                        if (b == true) {
                            pointList.remove(pointList.get(l);
                            n--;
                            stay_into_loops = false;
                            break;

ただし、このような事態に遭遇した場合、一般的にはコードの臭いです。コードのリファクタリングを検討してください。これは、ある時点で混乱に発展するためです。

于 2010-11-18T11:01:31.493 に答える
0

各内部forループに自分用の出口を作成します。
これが迅速で痛みのない解決策です。

  bool breakout;
  for (int l = 0; l < n; l++) 
  {
        breakout = false;
        for (int i = 1; i < (n - 2) && !breakout; i++)
            for (int j = i + 1; j < (n - 1) && !breakout; j++)
                for (int k = j + 1; k < n && !breakout; k++)
                {
                    if(b == true)
                        breakout = true;                            
                }
  }

つまり、breakoutブール値は各内部ループからのチケットであり、すべてのfor宣言でチェックされます。そして、最初に繰り返すたびにリセットされforます。

于 2010-11-18T11:02:39.390 に答える