SonarQube无法理解用于null检查的Java内联表达式

Question

我们正在使用SonarQube进行静态代码分析。我们面临着SonarQube发现的以下错误

NullPointerException might be thrown as 'evt' is nullable here

在以下代码中

try {
//business logic

}(Exception e){
    throw new MyException("Found issue for event " + evt.getDeatil());
}

为了消除这个bug，我们引入了用于null检查的小型内联代码

try {
//business logic

 }(Exception e){
     throw new MyException("Found issue for event " + evt != null ?
       evt.getDeatil() : null);
}

代码更改后，SonarQube仍然显示相同的违规错误。

有谁能帮我们找到我们这里缺少的东西吗？

Answer 1

行中有错误：

throw new MyException("Found issue for event " + evt != null ? evt.getDeatil() : null);

问题是运算符优先<代码>在之前计算： 因此您的代码等于：

String message = "Found issue for event " + evt;
throw new MyException(message != null ? evt.getDeatil() : null);

NullPointerException仍然可能发生。另外，您引入了一个新问题-表达式：

message != null

始终计算为true。

要修复它，您必须添加花括号：

throw new MyException("Found issue for event " + (evt != null ? evt.getDeatil() : null));