因资源泄漏导致内存不足的Java堆空间问题

时间:2017-09-26 11:11:02

标签: java file memory-leaks out-of-memory heap

我正在使用下面的代码读取JSON文件并导致java内存错误:

BufferedReader br1 = new BufferedReader(new FileReader(filename));
try {
StringBuilder sb = new StringBuilder();
String line = br1.readLine();
while (line != null) {
  sb.append(line);
  }
result = sb.toString();
} catch (Exception e) {
  e.printStackTrace();
}

我得到以下错误:

java.lang.OutOfMemoryError: Java heap space
        at java.util.Arrays.copyOf(Arrays.java:3332)
        at java.lang.AbstractStringBuilder.ensureCapacityInternal(AbstractStringBuilder.java:124)
        at java.lang.AbstractStringBuilder.append(AbstractStringBuilder.java:448)
        at java.lang.StringBuilder.append(StringBuilder.java:136)

后来我意识到缓冲读卡器存在资源泄漏的警告 所以我添加了代码br1.close();并且警告已经解决。但是,java堆空间的问题被卡住了。

我甚至将文件更改为普通文本文件,只向文件中添加了一个示例行,但问题仍然存在。

1 个答案:

答案 0 :(得分:3)

这不是任何内存泄漏,这是无限循环。您没有在line循环中更新while,因此它永远不会为空。它将循环直到你的内存不足。

您需要将line = br1.readLine()放入循环中:

try (BufferedReader br1 = new BufferedReader(new FileReader(filename))) {
    StringBuilder sb = new StringBuilder();
    String line = br1.readLine();
    while (line != null) {
        sb.append(line);
        line = br1.readLine();
    }
    result = sb.toString();
} catch (IOException | RuntimeException e) {
    // TODO proper exception handling
}

另请注意,如果您的输入可能非常大,您可能需要按顺序处理它,而不是将其全部存储在字符串中。