可以抛出“NullPointerException”；此处“context”可为空

Question

所以我对SonarQube有一个无法解决的问题。

if (result instanceof AsyncResult) {
    // Make sure handle async has been called
    context.handleAsync();
    Result newResult = context.controllerReturned();
    if (newResult != null) {
        result = newResult;
    }
 }

Answer 1

您的问题可能在这里：

context.handleAsync();

Sonar所说的是（给定上下文[...]）在您的代码中的某个地方，您对上下文进行null检查，但不是在本部分：

if (null == context) {
  // bla bla bla
}
...
context.handleAsync(); // yes, but context was tested for null, so it can be null.

使用时重新检查上下文，删除null检查或在方法开始时失败：

if (null == context) {
  throw new IllegalStateException("context is null");
}

或者更好：

  void yourMethod(Context context) {
    Objects.requireNonNull(context, "context");
    ... 
  }

方法被称为先决条件；它的唯一目的是检查空值并提供（如果您不省略）默认消息。

Answer 2

尝试以下操作：

if (result instanceof AsyncResult) {
        if (context == null) {
            //appropriate error handling
        } else {
            // Make sure handle async has been called
            context.handleAsync();
            Result newResult = context.controllerReturned();
            if (newResult != null) {
                result = newResult;
            }
        }
}

Answer 3

SonarQube声称上下文在这里可能是null。如果您知道这是不正确的，您可以将此警告作为误报来抑制。否则，您应该显式检查上下文是否不是null：

if (result instanceof AsyncResult && context != null) {
    // Here -------------------------^

    // Make sure handle async has been called
    context.handleAsync();
    Result newResult = context.controllerReturned();
    if (newResult != null) {
        result = newResult;
    }
}