Question

这是我的代码的一小部分。这里break语句不起作用。执行if条件但break语句将控件带到while循环的开头。 “assign”和“clsAssign”是两个数组列表。 “clustersRefGlobal（）”是一个函数，我不想在它为空时传递“assign”。但是由于中断不起作用，即使“assign”为空，也会调用它。我不确定为什么break语句不会停止while循环

   Wh:while (i < n) {
        System.out.println("Start");
        get = clustersRefGlobal(assign);
        clsAssign.add(get.get(0));
        assign = get.get(1);
        if(assign.isEmpty()){
            System.out.println("Inside");
            break Wh;
        } 
        System.out.println("End");
        i++;
    }

这是输出

Start

End

Start

Inside

Start

Exception in thread "main" java.lang.NullPointerException
    at softwareClustering.DominantSetClustering.clustersRefGlobal(DominantSetClustering.java:54)
    at softwareClustering.DominantSetClustering.buildDominatSetClustering(DominantSetClustering.java:76)
    at trees.PrototypeSelectionTree.clustersRefLocal(PrototypeSelectionTree.java:214)
    at trees.PrototypeSelectionTree.clustersRefGlobal(PrototypeSelectionTree.java:180)
    at trees.PrototypeSelectionTree.buildTree(PrototypeSelectionTree.java:59)
    at trees.PrototypeSelectionTree.buildClassifier(PrototypeSelectionTree.java:235)
    at weka.classifiers.Evaluation.crossValidateModel(Evaluation.java:617)
    at trees.TestClassifier.main(TestClassifier.java:45)
Java Result: 1

异常是因为使用空“assign”参数调用“clustersRefLocal（）”函数。如果有人知道这个问题或我错过了什么？

public double[] buildDominatSetClustering(int n) throws Exception {
    int i = 1;
    ArrayList<ArrayList<Integer>> clsAssign = new ArrayList<>();
    ArrayList<Integer> assign = new ArrayList<>();
    ArrayList<ArrayList<Integer>> get;
    for (int j = 0; j < data.numInstances(); j++) {
        assign.add(j);
    }
    Wh:
    while (i < n) {
        System.out.println("hello");
        get = clustersRefGlobal(assign);
        clsAssign.add(get.get(0));
        assign = get.get(1);
        if(assign.isEmpty()){
            System.out.println("inside  "+assign.size());
            break Wh;
        } 
        System.out.println(assign.size());
        i++;
    }
    if(!assign.isEmpty())
    clsAssign.add(assign);
    double[] indexAssToClus = new double[data.numInstances()];
    int count = 0;
    for (ArrayList<Integer> a : clsAssign) {
        for (int k = 0; k < a.size(); k++) {
            indexAssToClus[a.get(k)] = count;
        }
        count++;
    }
return indexAssToClus;
}

这是代码存在的功能

Answer 1

您所看到的简单解释是，实际上break正在停止循环......但是您向我们展示的代码段周围的代码再次启动它。

如果在标记的while语句之前添加跟踪记录，这将很明显。

异常是因为使用空“assign”参数调用“clustersRefLocal（）”函数。

我怀疑你对null感到困惑。空字符串是长度为零的非空String。如果您尝试通过调用null来测试String.isEmpty()字符串是否为空，那么您将获得NPE。对于非null非空字符串的正确测试是：

if (assign == null || assign.isEmpty()) {
    // null or empty ... bail out
    break;
}

Answer 2

我建议你简单地颠倒逻辑：

 if(! assign.isEmpty()){
     i++;
 }

但如果不这样做，请检查变量i：

的内容

您的支票适用于：while (i < n)

但唯一的地方i一直在变化，它会增加。

Answer 3

标记您的循环并且不鼓励使用break label。（它像goto并导致意大利面条代码。）

此外，这里没有意义，因为你没有嵌套循环。您只需将break Wh;更改为break;

即可

Break语句不起作用

3 个答案: