Sonar提示可能抛出NullPointerException，已做空检查仍报错？

这个问题我之前也碰到过，其实核心原因是Sonar的静态分析对第三方空校验方法的识别有限制。

你用的Strings.isNullOrEmpty(value)应该是Guava或者Apache Commons这类第三方库提供的工具方法吧？Sonar的默认规则里，并没有把这类第三方工具的空判断标记为“可信的非空校验”——它只认Java原生的value != null、Objects.nonNull(value)这种显式的空判断逻辑。所以哪怕你已经用Strings.isNullOrEmpty做了校验，Sonar依然没办法确定这个方法已经完全排除了value为null的可能，于是还是会抛出空指针警告。

不过看你更新后的代码，这个写法就非常棒：

String value = getValue("/data/" + set, property); 
if (value!=null) { 
    return Boolean.parseBoolean(value); 
} 
return false;

一来用了Java原生的value != null判断，Sonar能精准识别这个校验已经把null情况排除了，自然不会再报错；二来Boolean.parseBoolean本身就比手动判断contains("true")更靠谱——它不仅能处理空字符串（返回false），还能忽略大小写识别"TRUE"、"True"这类情况，逻辑更严谨。

如果实在想保留Strings.isNullOrEmpty的写法，也可以去Sonar的规则配置里，把这个方法添加到“认可的非空校验方法”列表中，但这种配置需要调整项目的Sonar设置，不如直接用原生空判断来得省心。

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