如何修改Java代码消除SonarLint的‘条件恒为true’警告？

这个警告出现的原因其实挺直观的：Boolean是个final类，不存在子类，所以任何Boolean类型的value只能是Boolean.TRUE或者Boolean.FALSE这两个实例。你的前两个条件已经把所有Boolean类型的情况都覆盖了——也就是说，当代码走到null == value这一行时，value要么是null，要么是完全和Boolean无关的其他类型（比如String、Integer之类的）。

SonarLint之所以报这个警告，是因为它觉得这个条件的位置和逻辑写法不够清晰，容易让人误解，甚至可能隐含逻辑漏洞。咱们可以通过两种方式修改来消除警告：

方式一：调整判断顺序，优先检查空值 #

把null的判断移到最前面，这样逻辑顺序更符合常规的“先空值、再业务逻辑”的习惯，也能让SonarLint的分析器清晰识别每个分支的范围：

@Override
protected void setValue(Object value) {
    if (null == value) {
        // 处理null的情况
    } else if (Boolean.TRUE.equals(value)) {
        // 处理true的情况
    } else if (Boolean.FALSE.equals(value)) {
        // 处理false的情况
    } else {
        // 针对非空且非Boolean的对象调用父类方法
        super.setValue(value);
    }
}

方式二：先判断类型，再处理真假 #

这种写法逻辑更明确，先确认value是不是Boolean类型，再区分真假，最后处理null和其他类型：

@Override
protected void setValue(Object value) {
    if (value instanceof Boolean) {
        Boolean boolValue = (Boolean) value;
        if (boolValue) {
            // 处理true的情况
        } else {
            // 处理false的情况
        }
    } else if (null == value) {
        // 处理null的情况
    } else {
        // 针对非空且非Boolean的对象调用父类方法
        super.setValue(value);
    }
}

两种写法都能完美消除SonarLint的警告，而且逻辑和原来的完全一致，你可以根据自己的代码风格选一种就行。

内容的提问来源于stack exchange，提问作者Sergiy Medvynskyy