1

我有以下代码（Android 4）：

private HttpURLConnection conn = null;

private synchronized String downloadUrl(String myurl) {
    InputStream is = null;
    BufferedReader _bufferReader = null;
    try {
        URL url_service = new URL(.....);
        System.setProperty("http.keepAlive", "false");
        System.setProperty("http.maxConnections", "5");
        conn = (HttpURLConnection) url_service.openConnection();
        conn.setReadTimeout(DataHandler.TIME_OUT);
        conn.setConnectTimeout(DataHandler.TIME_OUT);
        conn.setRequestMethod("POST");
        conn.setDoInput(true);
        conn.setDoOutput(true);
        conn.setRequestProperty("connection", "close");
        conn.setInstanceFollowRedirects(false);
        conn.connect();
        StringBuilder total = null;
        if (conn.getResponseCode() == HttpURLConnection.HTTP_OK) {
            is = conn.getInputStream();
            _bufferReader = new BufferedReader(new InputStreamReader(is));
            total = new StringBuilder();
            String line;
            while ((line = _bufferReader.readLine()) != null) {
                total.append(line);
            }

        } else {
            onDomainError();
        }

        return total.toString();

    } catch (SocketTimeoutException ste) {
        onDomainError();
    } catch (Exception e) {
        onDomainError();

    } finally {
        if (is != null) {
            try {
                is.close();
            } catch (IOException e) {
                // TODO Auto-generated catch block

            }

        }
        if (_bufferReader != null) {
            try {
                _bufferReader.close();
            } catch (Exception e) {
                // TODO: handle exception
            }
        }

        if (conn != null)
            conn.disconnect();
        conn = null;

    }
    return null;
}

.disconnect()使用时，keep-alive 设置为 false，最大连接数设置为 5。但是，如果SocketTimeout exception发生这种情况，连接不会关闭，并且设备很快就会耗尽内存。这怎么可能？

此外，根据http://developer.android.com/reference/java/net/HttpURLConnection.html，如果 keep-alive 设置为，HttpURLConnection则应关闭连接并在 keep-alive 为时重用它。这些方法都不适合我。有什么想法可能是错的吗？disconnect()falsetrue

4

1 回答 1

0

一种可能性是您没有足够快地设置属性。根据 javadoc，在发出任何 HTTP 请求之前，需要将“keepalive”属性设置为 false。这实际上可能意味着在 URL 协议驱动程序初始化之前。

另一种可能是您的OOME根本不是由此引起的。这可能是由您的应用程序对其下载的内容所做的事情引起的。

您的代码也存在其他一些问题。

  • 变量名url_service,_bufferedReader和myurl都违反了 Java 的标识符命名约定。

  • 该conn变量应该是一个局部变量。使其成为字段使该downloadUrl方法不可重入。（这可能会导致您的问题......如果多个线程共享该对象的一个​​实例！）

  • 您不需要关闭缓冲阅读器和输入流。只需关闭阅读器，它就会关闭流。这对读者来说可能无关紧要，但如果你为缓冲的作者这样做并且你先关闭输出流，你很可能会遇到异常。

更新

所以我们肯定有很多非垃圾HttpURLConnectionImpl实例，而且我们可能有多个线程通过AsyncTask.

如果您尝试连接到一个无响应的站点（例如，TCP/IP 连接请求是黑洞的站点……），那么该conn.connect()调用将阻塞很长时间并最终引发异常。如果连接超时足够长，并且您的代码正在并行执行可能无限数量的这些调用，那么您很可能拥有大量此类实例。

如果这个理论是正确的，那么你的问题与保持活动和连接没有被关闭无关。问题出在另一端……一开始就没有正确建立的连接阻塞了内存，并且每个连接都占用了一个线程/线程堆栈：

  • 尝试减少连接超时。
  • 尝试使用Executor有界线程池运行这些请求。

请注意它在AsyncTaskjavadoc 中的说明：

“AsyncTask 被设计为围绕 Thread 和 Handler 的辅助类，并不构成通用线程框架。AsyncTasks 理想情况下应该用于短操作（最多几秒钟）。如果您需要保持线程长时间运行有时间，强烈推荐你使用 java.util.concurrent 包提供的各种 API，如 Executor、ThreadPoolExecutor 和 FutureTask。”

于 2013-05-22T09:57:03.217 回答