首页 > Java:这样写为什么会被CheckStyle报警告呢?

Java:这样写为什么会被CheckStyle报警告呢?

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犯了什么规范的错误呢?


应该是你判断条件的问题,你第二句应该是一个废话,所以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;
    }
}

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

【热门文章】
【热门文章】