如何修复这个“可能的空指针取消引用”在声纳中的关键问题？

Question

下面是正在讨论的方法。

public String getCarById(Integer carId) {
  ResponseEntity<CarString> response = rTempl.getForEntity(url, CarString.class,
        of("car-id", carId.toString()));
  log.debug("code: {}", response.getStatusCode());
  return Objects.requireNonNull(response.getBody()).getCarString();
}

完整的错误如下。

Possible null pointer dereference in <package>.CarRegistryClient.getCarById(Integer) due to return value of called method

使用 Objects.requireNonNull（） 对于 Sonar 来说还不够吗？

更新:向方法中添加了缺失但相关的代码。现在没有代码丢失。

Answer 1

很可能< code>rTempl.getForEntity的方法被显式声明为可能返回< code>null，然后您的代码只是取消引用< code>getForEntity提供给您的内容，而不进行任何检查。您的< code>requireNonNull代码检查< code>response.getBody()是否返回非null它不检查< code>response本身是否为非空。

代码仍然丢失。rTempl到底是什么？

Answer 2

奇怪的是，这就是解决办法，声纳很高兴。

public String getCarById(Integer carId) {
  ResponseEntity<CarString> response = rTempl.getForEntity(url, CarString.class,
        of("car-id", carId.toString()));

  CarString responseBody = response.getBody();
  assert responseBody != null

  log.debug("code: {}", response.getStatusCode());
  return response.getBody().getCarString();
}

通过＜code＞Objects.requirennull（）＜/code＞我仍然不知道这与我以前的解决方案有什么不同。