写入文件 JavaFx 的问题

Question

所以，我对编程和这个网站还是很陌生。 我一直在尝试制作自己的浏览器只是为了好玩，我正在学习如何编写和读取文本文件以用作日志来保存数据。我正在尝试制作一个收藏夹系统，以便我可以将不同的网站保存到日志中，但我不明白为什么我的部分代码不起作用。我在网上花了大约两个半小时环顾四周，没有发现任何问题。

try(BufferedReader format1 = new BufferedReader(new 
                                         FileReader("favorites.txt"))) {

            String line = format1.readLine(); //Used to look through file
            String found = ""; //Stores the address that was the same

            //Loop searches file, I only plan to have a max of 20 favorites
            for (int counter = 0; counter < 20; counter++) {

                System.out.println(line); //Just used so I can make sure the 
                                            loop works

                if(line.equals(input)) {
                    counter = 20;
                    found = line;
                }

                line = format1.readLine();
            }
            format1.close();

            if(!(found.equals(input))) {
                BufferedWriter format2 = new BufferedWriter(new 
                                         FileWriter("favorites.txt", true));
                format2.write(input);
                format2.newLine();

                format2.close();


            }           
    } catch (IOException e1) {
        System.out.println("ERROR! Favorite not Added.");
    }

我检查了问题所在，实际写入文件的 if 语句似乎是问题所在。我已经看了一百万次，我什么也看不到。也许更有经验的程序员可以看到我的问题？每当我运行这部分代码时，它都会吐出大量错误消息。感谢您的帮助！

Answer 1

我怀疑您的问题源于这个奇怪的“最多 20 个收藏夹”循环，无论文件是否包含它们，它都会尝试读取 20 行（最终结果是它会抛出一个少于 20 行的 NullPointerException，并忽略超过 20 行）。似乎您试图通过仅阅读一定数量的行来简化事情，这样您就不必计算要阅读多少行，但在这样做的过程中实际上使事情变得更加复杂！

在 Java 中逐行读取整个文件有一个相当常见的习惯用法 - 尝试将您的 for 循环（以 counter 作为迭代器的循环）替换为：

String line;
while((line=format1.readLine())!=null) {
    System.out.println(line); //Just used so I can make sure the 
    if (line.equals(input)) {
        found = line;
        break;
    }
}
format1.close();

这会将文件的每一行读入line，而readLine() 的返回值不是null，它会在到达文件末尾时。