java - Retrofit2:如何正确检查response.body()的有效性？

Question

自版本 retrofit2.retrofit:2.3.0 我在 response.body() 上收到 NullPointerException 警告，即使在检查正文时也是如此对于 null 之前:

Method invocation 'getCar()' may produce 'java.lang.NullPointerException'

在 2.3.0 的变更日志中有一个与 null 检查相关的条目:

Retrofit now uses @Nullable to annotate all possibly-null values. [...] We use @ParametersAreNonnullByDefault and all parameters and return types are never null unless explicitly annotated @Nullable.

这是预期的行为吗？在我看来 response.body() 应该是不可变的，所以我检查图片 1不应显示警告。

这不是关于 NullPointerExceptions 的问题 - 这是关于如何正确处理 Retrofit2 响应的方式。为了现在没有警告，我必须做这样的事情:

if(response != null) {
    CarResponseBody body = response.body();
    if (body != null && body.getCar() != null){
        CarResponse car = body.getCar();
    }
}

很多代码只是为了检查是否有有效的响应...

Answer 1

Is that the intended behavior?

如果你查看 Response<T> 的 JavaDoc你可以阅读

@Nullable public T body() The deserialized response body of a successful response.
Javadoc: Response

如其所示，body()不会是null如果响应成功。要检查是否成功，您有 isSuccessful()哪个

Returns true if code() is in the range [200..300).

所以 @Nullable是一个有效的选择，因为响应可以是 null在任何不成功的情况下，例如无网络、请求无效或其他错误。

您的 IDE 中的提示是一个 lint 警告，用于检查您的源代码是否存在常见错误或错误的可能来源。

这就是原因 body()可能是 null以及为什么 lint 首先将此报告为警告。

In my point of view response.body() should be immutable, so my check in Picture 1 should not show a warning.

理论上你是对的。你我都知道 body()不是可变的，但这里的问题是 this lint check 无法知道。

T res1 = response.body(); // could be null
T res2 = response.body(); // ...maybe still null?

Lint 是一种静态源代码和字节代码分析器，有助于防止常见错误和错误，其中一种 lint 检查会尝试防止 NPE。如果你注释一个方法 @Nullable检查所知道的只是返回值可能是null如果您尝试直接对调用结果进行操作，它会发出警告。

// does response return the same value twice? who knows?
response.body() != null && response.body().getCar() != null

您处理响应的方式实际上是摆脱 lint 警告的唯一方法，而不是抑制或禁用它。

通过将其分配给局部变量，您可以确保值不是 null在某些时候它不会变成 null将来，lint 也能看到这一点。

CarResponseBody body = response.body(); // assign body() to a local variable
if (body != null && body.getCar() != null) { // variable is not null
    CarResponse car = body.getCar(); // safe to access body, car is also not null
    // everything is fine!
}

---

It that the intended behavior?

是的。 @Nullable是暗示方法可能返回 null 的好方法如果返回 null，您还应该在自己的代码中使用它在某些路径上因为lint 可以警告可能的 NullPointerExceptions。

如果一个方法可能返回 null您必须将其分配给本地字段并检查字段中的 null值，否则您冒着值可能已更改的风险。

Car car = response.getBody(); // car could be null
if(car != null) {
  // car will never be null
}

不同的选择/方法

我看到您还似乎将您的响应对象包装在一个附加层中。

CarResponseBody body = response.body();
Car car = body.getCar()

如果你想从你的代码中移除复杂性，你应该看看如何移除这个包装 *ResponseBody在较早的阶段。您可以通过注册自己的 Converter 并在那里添加额外的处理来实现。您可以在此 Retrofit talk by Jake Wharton 上看到更多相关信息

另一种完全不同的方法是将 RxJava 与 Retrofit 结合使用，这样就无需自己检查响应。您将获得成功或错误，您可以通过 Rx 方式处理。