Sonarqube显示NullPointerException错误，即使在Java中存在null检查之后

Question

Sonarqube说这段代码中有两个主要错误，在if条件下的第二条语句中。

第一：

可以抛出“NullPointerException”；“getResponseHeaders（）”可以返回null。

第二个：

取消引用“get响应头（）”的结果

private ResponseEntity<ErrorResponse> handleHttpStatusCodeException(HttpStatusCodeException exception) {
    ResponseEntity.BodyBuilder response = ResponseEntity.status(exception.getStatusCode());
    if (exception.getResponseHeaders() != null && exception.getResponseHeaders().getContentType() != null) {
        response.contentType(exception.getResponseHeaders().getContentType());
    }
    return response.body(createErrorResponse(exception.getResponseBodyAsString()));
}

有没有办法，我可以更改代码来避免这个bug？

Answer 1

理论上，另一个线程可能会更改异常的结果。getResponseHeaders（）在您检查它是否为非null之后，但在您再次调用它之前。这种检查然后行动并不总是安全的。

处理此问题的更安全方法是将

var headers = exception.getResponseHeaders();
if (headers != null) {
    var contentType = headers.getContentType();
    if (contentType != null) {
        response.contentType(contentType);
    } 
}

我认为这比你的解决方案更具可读性。

当然，如果您的对象没有被多个线程变异或不可变，那么此警告是误报，可以忽略。