1

我正在尝试从 txt 文件（书）中读取，然后将其每一行添加到链表中。但是，当我运行代码时，我在l.add(line);. 你能告诉我这段代码我做错了什么吗？或者，有没有更好的方法来存储字符串值而不是 LinkedList？

非常感谢！

public Book (String bookname) throws java.io.IOException{
    f = new FileReader(bookname);
    b = new BufferedReader(f);
    l = new LinkedList<String>();
    String line = b.readLine();
    while (line != null) {
        l.add(line);
    }
    b.close();
}
4

6 回答 6

6

正如其他人指出的那样，您已经创建了一个无限的、消耗内存的循环。从 BufferedReader 读取的常见习惯用法是：

String line;
while ( ( line = b.readLine() ) != null) {
    l.add(line);
}

我想这本书的内容可能太大而无法一次全部放入记忆中。您可以使用 Xmx 参数增加 JVM 可用的内存，即：

java -Xmx1G MyClass

其默认值为 64 Mb，这在当今并不多。

于 2011-03-04T11:53:05.350 回答
2

您一遍又一遍地添加同一行，直到内存用完：

String line = b.readLine();
while (line != null) {
    l.add(line);
}

看？line变量在循环外被读取，并且在循环内永远不会改变。

于 2011-03-04T11:54:31.487 回答
2

可能你应该更换

while (line != null) {
    l.add(line);
}

和

while (line = b.readLine()) {
    l.add(line);
}
于 2011-03-04T11:56:52.420 回答
1

While 循环永远不会退出，因为变量 line永远不会为空。试试这个：

String line = "";
while ((line = b.readLine())!= null)
{
   l.add(line);
}
b.close();
于 2011-03-04T11:55:55.297 回答
0

很简单，存储字符串（以及程序中的所有其他内容）所需的内存超过了堆上可用的总可用内存。

其他列表的开销会略有不同，但实际上结构本身的内存需求与其包含的数据相比可能微不足道。换句话说，切换到不同的列表实现可能会让你在跌倒之前多读几行，但这并不能解决问题。

如果您没有增加 java 应用程序的堆空间，它可能会以相当低的默认值运行。在这种情况下，您应该考虑为您的调用提供以下命令行参数java：

-Xmx512m

（这512m意味着 512 兆字节的堆空间；您可以使用 eg-Xmx2g或您认为合适的任何其他内容。）

另一方面，如果你已经在运行一个大堆（比你想在内存中保存的字符串的总大小大得多），这可能指向其他地方的内存问题。书中有多少人物？存储所有行至少需要两倍的字节数，并且可能比开销多 20% 左右。如果您的计算表明您当前的堆空间应该能够保存所有这些数据，那么其他地方可能会出现一些问题。否则，您现在知道需要将堆增加到最低限度。

（顺便说一句，尝试将大量输入作为单个批次处理通常会遇到内存问题 - 如果您想处理 8GB 文本文件怎么办？通常最好在某种流中按顺序处理较小的块。对于例如，如果您想将每个字符大写并将其写回不同的文件，您可以一次执行一行，而不是先将整本书读入内存。）

于 2011-03-04T11:54:05.590 回答
0

我同意 mjg123。并注意内存不足的预期。我要求您必须查看此博客以了解有关如何处理此类情况的更多详细信息 单击此处

于 2011-03-04T13:39:11.293 回答