FindBugs 警告删除

Question

运行 FindBugs 时出现以下错误：

错误类型 NP_NULL_ON_SOME_PATH。

给出错误的代码如下：

List<Request> reviewerList = null;
    
response = restAPI CAll // assume some rest call.
reviewerListDto = response.getBody(); //it never returns null

// this check is added because of find bugs, else it wont allow to access getReviewers() method 
if (reviewerListDto!=null) { 
    reviewerList = reviewerListDto.getReviewers();
}
// HERE is the point I am stuck 
for (Request reviewer : reviewerList) {
    reviewer.setPullrequestId(pullrequest.getId());
    reviewer.setRepositoryId(pullrequest.getRepositoryId());
}

错误即将到来，因为根据 for 循环中的 FindBugs，reviewerList 可以为空。但我知道它永远不会是null。删除它的一种方法是在 for 循环之前检查 reviewerList 是否为 null，但是还有其他方法吗？

Answer 1

你可以use @SuppressFBWarnings if you add com.google.code.findbugs:annotations to the classpath:

@SuppressFBWarnings(value="NP_NULL_ON_SOME_PATH", justification="This can never be null")

但是，这与您将if-statement 放在那里的原因基本相同。添加 if 语句时，您说可能是这种情况，并且仅在发生这种情况时才执行代码。如果总是执行 if 语句，则没有理由使用 if 语句。

因此，您也可以禁止 if 语句并忽略导致您添加 if 语句的警告。

另一种方法是将使用 reviewerList 的代码放入 if 语句中，或者如果它为空则抛出异常（例如 AssertionError）（或立即使用 assert）。

List<Request> reviewerList = null;
// response = restAPI CAll // assume some rest call.
reviewerListDto = response.getBody(); //it never returns null
assert reviewerListDto!=null;
reviewerList = reviewerListDto.getReviewers();
for (Request reviewer : reviewerList) {
    reviewer.setPullrequestId(pullrequest.getId());
    reviewer.setRepositoryId(pullrequest.getRepositoryId());
}