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;
    }
}