SonarLint提示wsr.getBody()可能空指针，已做校验仍存疑

你的代码里的空校验逻辑是有效的，不会出现wsr.getBody()为空导致的空指针异常，SonarLint的提示属于误报，原因是静态分析工具没能正确识别你的前置判空逻辑。

具体分析： #

• 你通过if (wsr == null || wsr.getBody() == null)做了完整的前置校验，只要任一条件成立就直接抛出运行时异常终止方法，后续的return wsr.getBody().getWorkspace()根本不会执行，完全规避了空指针风险。
• SonarLint这类静态分析工具的代码流分析能力有限，它无法精准关联判空语句和后续调用的依赖关系，所以会误判存在空指针风险。

消除SonarLint提示的可选方案： #

• 方案一：提取局部变量
  将wsr.getBody()赋值给局部变量后再做判空，让静态分析工具更容易识别非空逻辑：

  public WorkspaceSummary getWorkspace(String workspaceName) {
      try {
          ResponseEntity<GetWorkspaceResponse> wsr = this.workspacesApi.getWorkspaceWithHttpInfo(workspaceName, this.quiet);
  
          GetWorkspaceResponse body = wsr.getBody();
          if (wsr == null || body == null) {
              throw new RuntimeException("getWorkspaceWithHttpInfo 返回了空响应");
          }
  
          return body.getWorkspace();
      } catch(HttpClientErrorException e) {
          [...]
      }
  }
  

• 方案二：使用Objects.requireNonNull()
  如果你使用Java 16及以上版本，可以用JDK自带的工具类显式声明非空，同时简化判空代码：

  import java.util.Objects;
  
  public WorkspaceSummary getWorkspace(String workspaceName) {
      try {
          ResponseEntity<GetWorkspaceResponse> wsr = this.workspacesApi.getWorkspaceWithHttpInfo(workspaceName, this.quiet);
          GetWorkspaceResponse body = Objects.requireNonNull(wsr, "响应实体不能为空").getBody();
          Objects.requireNonNull(body, "响应体不能为空");
  
          return body.getWorkspace();
      } catch(HttpClientErrorException e) {
          [...]
      }
  }
  

内容的提问来源于stack exchange，提问作者Marc Le Bihan