OkHttp：避免泄漏连接警告答案

【问题标题】：OkHttp: avoid leaked connection warningOkHttp：避免泄漏连接警告
【发布时间】：2016-11-17 17:38:20
【问题描述】：

我正在使用 OkHttp 3，但我不断收到泄露的连接警告：

WARNING: A connection to https://help.helpling.com/ was leaked. Did you forget to close a response body?
Jul 14, 2016 6:57:09 PM okhttp3.ConnectionPool pruneAndGetAllocationCount

每当我收到ResponseBody 时，我要么调用.string()，它应该为我关闭流，或者我在finally 块中显式关闭它，方式如下：

ResponseBody responseBody = response.body();
try (Reader responseReader = responseBody.charStream()) {
    ...
}
finally {
    responseBody.close();
}

我的应用程序大量使用网络，但该警告经常出现。我从未观察到由这种假定的泄漏引起的任何问题，但我仍然想了解 if 和 what 我做错了什么。

有人能解释一下吗？

【问题讨论】：

标签： java memory-leaks okhttp3

【解决方案1】：

通过升级到 OkHttp 3.7，Eclipse 开始警告我潜在的资源泄漏。我发现我的问题出在我写的这个方法上：

public static Response getResponse(HttpUrl url, OkHttpClient client) throws IOException {
    Builder request = new Request.Builder().url(url);
    Response response = client.newCall(request.build()).execute();
    if (!response.isSuccessful()) {
        boolean repeatRequest = handleHttpError(response);
        if (repeatRequest)
            return getResponse(url, client, etag);
        else
            throw new IOException(String.format("Cannot get successful response for url %s", url));
    }
    return response;
}

我假设总是调用getResponse(url, client).body().string()，流会自动关闭。但是，只要响应不成功，就会在执行.string() 之前引发异常，因此流将保持打开状态。

在响应不成功的情况下添加显式关闭解决了问题。

if (!response.isSuccessful()) {
    boolean repeatRequest = handleHttpError(response);
    response.close();
}

【讨论】：

【解决方案2】：

如其他答案中所述，您必须关闭响应。更简洁的方法是在 try 块中声明 ResponseBody，这样它就会自动关闭。

try(ResponseBody body = ....){
....
}

【讨论】：

对于某些人来说，直接关闭 Response 而不是 ResponseBody 可能会更好一些。也就是说，关闭 Response 会关闭底层的 ResponseBody ...对于我的用例，关闭 Response 会更好一些。
我不能使用 try-with-resources 方法，因为我的方法需要返回 Response 实例，如果成功则由另一个方法读取（请参阅我自己的答案）。
如果您使用 Kotlin 和新的 okhttpclient v4，只需使用 client.newCall(..).execute().use {response -> ... } 使用响应