SonarQube空指针警告疑问：已做多层非空校验仍提示getBody()可能返回空

嗨，我之前也碰到过一模一样的困惑——明明把所有可能的null情况都用Objects.nonNull()堵上了，SonarQube怎么还揪着getBody()可能返回空的问题不放？

咱们先扒一扒你这段代码的问题所在：

ResponseEntity<CommonResponseDTO> commonResponseDto = testClient.getCategoryById(sampleEntity.getCategoryId());

if(Objects.nonNull(commonResponseDto) && Objects.nonNull(commonResponseDto.getBody()) && Objects.nonNull(commonResponseDto.getBody().getName())){
    SuperDataDto superDataCategoryDto =  SuperDataDto.builder()
        .id(sampleEntity.getCategoryId())
        .name(commonResponseDto.getBody().getName())
        .build();
}

你在if条件里调用了一次commonResponseDto.getBody()做非空校验，但是在构建SuperDataDto的时候，又调用了一次commonResponseDto.getBody()。SonarQube的静态分析器没办法100%确定这两次调用返回的是同一个非空对象——它会假设getBody()可能不是单纯的getter方法，比如有没有可能每次调用它都返回新的实例，甚至在两次调用之间被其他逻辑改成了null？虽然咱们自己知道实际代码里它就是个普通的成员变量读取，但Sonar的分析是基于代码结构的，它不会做“这个方法没有副作用”的假设。

知道了原因，解决办法就很清晰了，给你两个实用的方案：

方案一：提前提取局部变量（最直接） #

把getBody()的结果存到一个局部变量里，只做一次非空校验，之后全程用这个变量：

ResponseEntity<CommonResponseDTO> commonResponseDto = testClient.getCategoryById(sampleEntity.getCategoryId());
// 提前把body提取出来
CommonResponseDTO responseBody = commonResponseDto != null ? commonResponseDto.getBody() : null;

if(Objects.nonNull(responseBody) && Objects.nonNull(responseBody.getName())){
    SuperDataDto superDataCategoryDto =  SuperDataDto.builder()
        .id(sampleEntity.getCategoryId())
        .name(responseBody.getName()) // 直接用局部变量，Sonar能识别到非空
        .build();
}

这样SonarQube就能明确追踪到responseBody已经通过了非空校验，后续调用getName()的时候就不会再报空指针警告了。

方案二：用Optional链式调用（更优雅） #

如果你的项目用的是Java 8及以上版本，用Optional的链式调用既能避免空指针，又能让代码逻辑更清晰，Sonar也能完美识别：

ResponseEntity<CommonResponseDTO> commonResponseDto = testClient.getCategoryById(sampleEntity.getCategoryId());

Optional.ofNullable(commonResponseDto)
    .map(ResponseEntity::getBody) // 提取body，自动处理null
    .filter(body -> Objects.nonNull(body.getName())) // 校验name非空
    .ifPresent(body -> {
        SuperDataDto superDataCategoryDto = SuperDataDto.builder()
            .id(sampleEntity.getCategoryId())
            .name(body.getName()) // 这里的body肯定非空
            .build();
        // 这里可以添加superDataCategoryDto的后续处理逻辑
    });

这种写法把所有非空校验都串联起来，逻辑一目了然，也完全符合Sonar的代码分析规则。

还有个极端情况：如果你的CommonResponseDTO是不可变类，而且getBody()就是个纯getter方法，那Sonar的警告就是过度敏感了。这时候你可以选择在Sonar里忽略这个特定位置的警告，但非常不推荐——毕竟前面两种方法都能让代码更健壮，还能顺便提升可读性。

备注：内容来源于stack exchange，提问作者Jeet