我的Java代码经常陷入“ while（（line = rd.readLine（））！= null）{”循环中。 如何优化代码

Question

我的Java代码经常陷入循环“ while（（line = rd.readLine（））！= null）{”

我猜这是因为Java垃圾回收没有收回一些内存。 变量“ line”将在每次迭代中指向新内容，并且由于某些原因，java GC不会回收“ line”的旧内容的内存。

那么如何优化代码来解决该问题呢？

代码如下：

for (String url : urlList) { // a loop that process ~1000 urls
      HttpResponse response = fetchSomeUrl(url);
      // I already confirmed that response code is 200 OK, which means HttpResponse is generated successfully.

      BufferedReader rd = new BufferedReader (new InputStreamReader(response.getEntity().getContent()));
      StringBuilder raw_content = new StringBuilder();
      String line;
      int i = 0;
      System.out.println("=====start processing rd.readLine()");
      while ((line = rd.readLine()) != null) {
         // my code often stuck here when there are more than 1000 lines in rd
         System.out.println("=====processing " + i + "th line");
         raw_content.append(line);
         raw_content.append("\n");
         i++;
      }
      System.out.println("=====finished processing rd.readLine()");
  }

Answer 1

我的Java代码经常陷入循环“ while（（line = rd.readLine（））！= null）{”

这个词是“被阻止的”。 它被阻止，等待另一条线从对等方到达，或等待对等方关闭连接。

我猜这是因为Java垃圾回收没有收回一些内存。

它与垃圾回收，Java或内存无关。 它与对等方在做什么或不在做什么有关。

变量“ line”将在每次迭代中指向新内容，并且由于某些原因，java GC不会回收“ line”的旧内容的内存。

这是胡说八道，即使是事实，也不能解释这种现象。

那么如何优化代码来解决该问题呢？

这不是编码问题，如果是的话，没有任何“优化”可以解决。 您的程序正在按编码方式工作。 该循环将迭代直到对等方断开连接，并且在没有完整的行或流的末尾要读取时将阻塞。

您应该在到达每条线时对其进行处理，而不是尝试在对等体断开连接时组装它们并对其进行全部处理。

编辑您可以考虑设置读取超时以捕获有问题的情况。

Answer 2

您的循环直到到达编写代码的null时才会停止。您应该添加一个条件，该条件将在程序达到1000时告诉程序停止。而且，您的内容可能超过了预期的1000，这就是它被卡住的原因。