Java BufferedReader FileReader 问题

Question

所以我有以下代码：

import java.io.*;

public class Plagiarism {

    public static void main(String[] args) {

        Plagiarism myPlag = new Plagiarism();

        if  (args.length == 0) {
            System.out.println("Error: No files input");
        }
        else if (args.length > 0) {
            try {
                for (int i = 0; i < args.length; i++) {
                    BufferedReader reader = new BufferedReader (new FileReader (args[i]));
                    simplify (reader);
                    reader.close();
                }
            }
            catch (Exception e) {
                System.err.println ("Error reading from file");
            }
        }
    }

    public static void simplify(BufferedReader input) throws IOException {
        String line = null;

        line = input.readLine();
        while (line != null) {
            line = line.replaceAll ("[^a-zA-Z0-9 ]", "");
            line = line.toLowerCase();
        }    
    }

}

此代码的问题在于它可以编译，但是当我运行它并在命令行中添加 2 个参数时，例如。 Java 抄袭 text1.txt text2.txt。编辑：当我运行它时，它什么也不做，甚至没有完成，就像它卡在某个地方一样。

感谢您的帮助。

Answer 1

您不是一次读取文件（您需要使用线程来执行此操作）。

问题出在您的 simplify 方法中。

line = input.readLine();
while (line != null) {

...应该变成：

while ((line = input.readLine()) != null)

原因是你只调用了一次readLine，否则只迭代第一行的值。

使用正确的while 循环，您可以在将readLine 调用的值分配给line 变量后将其传递给非null 条件。

然后，您可以按照您在评论中的建议，使用您在 while 循环中操作的 line String 做任何您想做的事情，例如将其添加到 array 或 Collection。

例如：

public static List<String> simplify(BufferedReader input) throws IOException {
    String line = null;
    List<String> result = new ArrayList<String>();

    while ((line = input.readLine()) != null) {
        result.add(line.replaceAll ("[^a-zA-Z0-9 ]", "").toLowerCase());
    }    
    return result;
}

...然后在您的main 方法中...

List<String> foo = simplify(reader);

Answer 2

编辑：当我运行它时，它什么也不做，甚至没有完成，就像它卡在某个地方一样。

这是完全正常的。看看你的while 循环：

    while (line != null) {
        line = line.replaceAll ("[^a-zA-Z0-9 ]", "");
        line = line.toLowerCase();
    }

line 在这里永远不会为空。你应该：

while ((line = input.readLine()) != null)
    // etc 

您有一个更根本的问题：如果您的目标是替换输入文件中的行，您的程序将永远无法正常工作...

试试这个：

private static final Pattern PATTERN = Pattern.compile("[^a-zA-Z0-9 ]+");

private static void simplify(final String fileName)
    throws IOException
{
    final Path path = Paths.get(fileName);
    final Path tempfile = Files.createTempFile(fileName.getFileName(), "tmp");
    try (
        final BufferedReader reader = Files.newBufferedReader(path);
        final BufferedWriter writer = Files.newBufferedWriter(tempfile);
    ) {
        String line;
        while ((line = reader.readLine()) != null) {
            line = PATTERN.matcher(line).replaceAll("").toLowerCase();
            writer.write(line);
            writer.newLine();
        }
        writer.flush();
    }
    Files.move(tempfile, path, StandardCopyOption.REPLACE_EXISTING);
}