OkHttp: tránh kết nối bị rò rỉ cảnh báo

Question

Tôi đang sử dụng OkHttp 3, và tôi tiếp tục nhận được rò rỉ cảnh báo kết nối:

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 

Mỗi lần tôi nhận được một ResponseBody, tôi hoặc gọi .string() được cho là đóng dòng đối với tôi, hoặc tôi một cách rõ ràng đóng nó trong một khối finally, theo cách sau:

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

ứng dụng My tận dụng cường độ cao của mạng, nhưng cảnh báo rằng xuất hiện thường xuyên. Tôi chưa bao giờ quan sát thấy bất kỳ vấn đề nào do rò rỉ được cho là này, nhưng tôi vẫn muốn hiểu nếu và những gì Tôi đang làm sai.

Có ai có thể làm sáng tỏ điều này không?

Answer 1

Bằng cách nâng cấp lên OkHttp 3.7, Eclipse bắt đầu cảnh báo tôi về rò rỉ tài nguyên tiềm ẩn. Tôi thấy vấn đề của mình là theo phương pháp này tôi đã viết:

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; 
} 

Tôi giả định rằng bằng cách luôn gọi getResponse(url, client).body().string() luồng sẽ tự động đóng. Tuy nhiên, bất cứ khi nào một phản ứng không thành công, một ngoại lệ sẽ tăng trước khi thực hiện .string(), do đó luồng sẽ vẫn mở.

Thêm gần đúng trong trường hợp phản hồi không thành công đã giải quyết được sự cố.

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

Answer 2

bạn phải đóng cho cơ thể() thử mã dưới đây

ResponseBody body = resp.body(); 
try { 
    ... 
} finally { 
body.close(); 
} 

Answer 3

Như đã đề cập trong câu trả lời khác, bạn phải đóng các phản ứng. Một cách tiếp cận hơi sạch sẽ là khai báo ResponseBody trong khối thử, để nó sẽ tự động bị đóng.

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