在try/catch上不应取消SonarQube空指针的引用

Question

我的主要问题是，我正在使用try-catch执行resttemplate.exchange，并在子句try之前声明一个空值变量，然后在try中使用它。最后，我的方法返回一个值为RESTTemplate的响应。

public MyDto exchangeUrlRequest(String url){
 ResponseEntity<MyDto> responseDto = null;
 try{
  responseDto = restTemplate.exchange(url, HttpMethod.PUT...
 }catch(HttpClientErrorException e){
   //some code here
 }
  return responseDto.getBody();
}

本文的预期结果是用Sonarqube解决这个问题。如何将“responsedto”的初始化没有“null”，因为是在声纳上抛出问题。

我已经尝试将“responseEntity responsedto”放在我的try子句中，分配和返回相应的值，但它必须返回try/catch中的一些内容。放入“new responseentity”是错误的，因为我不知道HTTP状态的答案是什么。

Answer 1

当捕获到某个异常时，您的代码需要对可能的NullPointerException执行一些操作，因为在这种情况下，ResponseDto将为空。

有很多方法可以解决这个问题。我推荐的解决方案是不要在Java上使用null返回或变量，尽量避免它。您可以使用可选。

所以，这个代码应该解决声纳问题：

public Optional<MyDto> exchangeUrlRequest(String url){

     ResponseEntity<MyDto> responseDto;
     try{
          responseDto = restTemplate.exchange(url, HttpMethod.PUT...);
     } catch(HttpClientErrorException e) {
         //some code here
     }

     if (responseDto == null) {
         return Optional.empty();
     } 
     return Optional.of(responseDto.getBody());
}

public Optional<MyDto> exchangeUrlRequest(String url){

     Optional<ResponseEntity<MyDto>> optResponseDto = Optional.empty();
     try{
          optResponseDto = Optional.of(restTemplate.exchange(url, HttpMethod.PUT...));
     } catch(HttpClientErrorException e) {
         //some code here
     }

     if (!optResponseDto.isPresent()) {
         return Optional.empty();
     } 
     return optResponseDto.get().getBody();
}

public Optional<MyDto> exchangeUrlRequest(String url){

     ResponseEntity<MyDto> responseDto = null;
     try{
          responseDto = restTemplate.exchange(url, HttpMethod.PUT...);
     } catch(HttpClientErrorException e) {
         //some code here
     }

     if (responseDto == null) {
         return null;
     } 
     return responseDto.getBody();
}