java - SonarQube 无法理解 Java 内联表达式进行 null 检查

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

因此，为了消除此错误，我们引入了用于空检查的小型内联代码

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

问题是operators precedence 。 + 在 ?: 之前评估，因此您的代码等于:

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