如何解决findbug问题：非Null参数传递Null

Question

我得到以下findbugs错误：

“方法调用为非null参数传递null：为getApiStatus（ApiResponse）的非null参数传递null”

如果CallApi方法中的apiResponse为null（为了简洁起见，这里没有显示），它只会抛出一个异常，该异常在HandLeapIEException中被捕获，如果我们不能对该异常执行任何其他操作，它会再次抛出。

在这段代码片段的基础上，apiACK的空值不可能传递给getApistatus（）方法。我如何告诉findbugs情况就是这样，而不在apiService中完成的空检查之上进行另一次空检查。CallApi方法？我尝试过使用NonNull注释，但这并不能解决问题。这是有问题的代码：

ApiResponse apiResponse = null;
try {
    apiResponse = apiService.CallApi(apiURL, requestObject);
}
catch (ApiException ex) {
    handleApiException(ex);
}

boolean apiStatus = getApiStatus(apiResponse);

有什么想法吗？

Answer 1

如果CallApi抛出异常，那么它将被处理，控制将继续getApiStatus，而apiResponse除了最初的null之外，不会被分配任何内容。

Answer 2

在您的代码中，无论是否发生ApiException，都会调用getApistatus（apiACK）。

你应该用这个代替：

try {
    ApiResponse apiResponse = apiService.CallApi(apiURL, requestObject);
    // line bellow will not execute if CallApi throws ApiException
    boolean apiStatus = getApiStatus(apiResponse);
}
catch (ApiException ex) {
    handleApiException(ex);
}
// lines bellow will execute after try-catch block above
// regardless of the ApiException occurring or not

Answer 3

我的建议是不处理异常，而是将此方法设置为异常。然后把它放在链条的上面。如果您的代码在该try块中获得一个exeption，然后在catch中处理该异常，那么apiResponse很容易为null。然后将继续尝试getApiStatus方法，从而传入null。

public void yourMethod() throws ApiException {
    ApiResponse apiResponse = apiService.CallApi(apiURL, requestObject);
    boolean apiStatus = getApiStatus(apiResponse);
    // Whatever else you need to do here.
}

您唯一的其他选择是将apiStatus调用放在try块内apiResponse调用的下面，如下所示：

ApiResponse apiResponse = null;
try {
    apiResponse = apiService.CallApi(apiURL, requestObject);
    boolean apiStatus = getApiStatus(apiResponse);
} catch (ApiException ex) {
    handleApiException(ex);
}

或者，正如您所说，在调用getApiStatus之前进行空检查，但这并不像上面的选项那样可取。