一个死循环导致的栈溢出实例：StackOverFlowError

　　有一个功能，要用复选框组做成单选框效果，如果有三个复选框 CheckBox ，并且保证每次只能选中一个。刚开始添加了以下的值改变后的监听方法 addValueChangeListener ，却导致了栈溢出：

 public static void checkBoxAddChangeListener(CheckBox checkBox1, CheckBox checkBox2, CheckBox checkBox3){

     checkBox1.addValueChangeListener(e -> {

         checkBox2.setValue(!checkBox1.getValue());

         checkBox3.setValue(!checkBox1.getValue());

     });

     checkBox2.addValueChangeListener(e -> {

         checkBox1.setValue(!checkBox2.getValue());

         checkBox3.setValue(!checkBox2.getValue());

     });

     checkBox3.addValueChangeListener(e -> {

         checkBox1.setValue(!checkBox3.getValue());

         checkBox2.setValue(!checkBox3.getValue());

     });

 }

　　思考上面代码的逻辑，假如现在第二个复选框是选中状态，此时再选中了第一个复选框，会触发第一个复选框的监听事件，第二、三个复选按钮就变成了未选中状态。因为第二个复选框的状态改变了，会触发它的监听事件，此时第一、三个复选框会变成选中状态。因为第一个已经是选中状态了，所以不会触发值改变的监听器。但是第三个复选框的状态却变了，触发监听器事件，把第一、二个监听器变成未选中状态。此时会再次触发第一个复选框的监听器事件。如此往复循环，StackOverFlowError

　　正确做法之一就是，不要在值改变的时候就触发事件，要在复选框的值变成选中状态时，触发一次事件就好了。添加以下判断。

 public static void checkBoxAddChangeListener(CheckBox checkBox1, CheckBox checkBox2, CheckBox checkBox3){

     checkBox1.addValueChangeListener(e -> {

         if(checkBox1.getValue()){

             checkBox2.setValue(false);

             checkBox3.setValue(false);

         }

     });

     checkBox2.addValueChangeListener(e -> {

         if(checkBox2.getValue()){

             checkBox1.setValue(false);

             checkBox3.setValue(false);

         }

     });

     checkBox3.addValueChangeListener(e -> {

         if(checkBox3.getValue()){

             checkBox1.setValue(false);

             checkBox2.setValue(false);

         }

     });

 }

　　这段代码还可以优化成一组复选框里只能选中一个值的方法，如下：

public static void checkBoxAddChangeListener(List<CheckBox> list){

    for(int i=0; i<list.size(); i++){

        CheckBox checkBox = list.get(i);

        checkBox.addValueChangeListener(e -> {

            if(checkBox.getValue()){

                for (CheckBox tmp : list) {

                    if (tmp == checkBox) {

                        continue;

                    }

                    tmp.setValue(false);

                }

            }

        });

    }

}

附录：

　　但是最开始的错误方法，如果只有两个CheckBox，就不会报错了

public static void checkBoxAddChangeListener(CheckBox checkBox1, CheckBox checkBox2){

    checkBox1.addValueChangeListener(e -> {

        checkBox2.setValue(!checkBox1.getValue());

    });

    checkBox2.addValueChangeListener(e -> {

        checkBox1.setValue(!checkBox2.getValue());

    });

}

　　原因是：这两个监听器，每一个只能触发一个。
　　假如开始第二个复选框是选中状态，此时选中第一个复选框，触发第一个复选框的监听事件，把第二个复选框变成未选中状态。再触发第二个复选框的监听事件，此时把第一个复选框变成了选中状态。但是因为第一个复选框本身已经是选中状态了，所以它的值并没有改变，所以不会触发监听器事件。

　　原创文章，欢迎转载，转载请注明出处！

巴特西

一个死循环导致的栈溢出实例：StackOverFlowError

最新文章

热门文章