Java：这样写为什么会被CheckStyle报警告呢？

Question

public class Demo {

    public boolean function() {

        if (条件1) {

            return true;

        } else if (条件2) {  // 警告：额外的判断应被移除

            return true;

        } else {

            return false;

        }

        return false;

    }

}

改成如下写法就不报错了

public class Demo {

    public boolean function() {

        if (条件1) {

            return true;

        }

        if (条件2) {

            return true;

        }

        return false;

    }

}

第一种写法的else if犯了什么规范的错误呢？

Answer 1

很正常，你的前两个判断语句如果判断为false，那么第三条语句直接返回false，但是后面你又有一个return false;那么其实你的最后一个else语句是完全没有必要写的。所以ide提示你咯。。

Answer 2

应该是你判断条件的问题，你第二句应该是一个废话，所以IDE提示你。仔细观察一下的判断条件，看看是不是写了无用的内容。比如这样的条件public class Demo {    public boolean function(int i) {        if (i!=0) {            return true;        } else if (i==3) {   // 第二个永远不能触发            return true;        } else {            return false;        }        return false;    }}