8

我正在尝试从文本文件中删除一行文本而不复制到临时文件。我正在尝试通过使用 Printwriter 和 Scanner 并让它们同时遍历文件来做到这一点，作家写下 Scanner 读取的内容并用相同的东西覆盖每一行，直到它到达我希望的行删除。然后，我推进扫描仪而不是写入器，并像以前一样继续。这是代码：

但首先，参数： 我的文件名是数字，所以这将读取 1.txt 或 2.txt 等，因此 f 指定文件名。我在文件的构造函数中将其转换为字符串。Int n 是我要删除的行的索引。

public void deleteLine(int f, int n){
 try{
 Scanner reader = new Scanner(new File(f+".txt")); 
 PrintWriter writer = new PrintWriter(new FileWriter(new File(f+".txt")),false); 
 for(int w=0; w<n; w++)
   writer.write(reader.nextLine()); 
 reader.nextLine(); 
 while(reader.hasNextLine())
   writer.write(reader.nextLine());
 } catch(Exception e){
   System.err.println("Enjoy the stack trace!");
   e.printStackTrace();
 }
}

它给了我奇怪的错误。它在堆栈跟踪中显示“NoSuchElementException”和“找不到行”。它指向不同的线；似乎任何 nextLine() 调用都可以做到这一点。是否可以通过这种方式删除一行？如果是这样，我做错了什么？如果不是，为什么？（顺便说一句，以防万一你想要这个，文本文件大约有 500 行。不过，我不知道这是否算大甚至重要。）

4

3 回答 3

24

正如其他人指出的那样，如果您的程序在中途崩溃的风险很小，那么使用临时文件可能会更好：

public static void removeNthLine(String f, int toRemove) throws IOException {

    File tmp = File.createTempFile("tmp", "");

    BufferedReader br = new BufferedReader(new FileReader(f));
    BufferedWriter bw = new BufferedWriter(new FileWriter(tmp));

    for (int i = 0; i < toRemove; i++)
        bw.write(String.format("%s%n", br.readLine()));

    br.readLine();

    String l;
    while (null != (l = br.readLine()))
        bw.write(String.format("%s%n", l));

    br.close();
    bw.close();

    File oldFile = new File(f);
    if (oldFile.delete())
        tmp.renameTo(oldFile);

}

（注意对编码、换行符和异常处理的草率处理。）

但是，我不喜欢用“我不会告诉你怎么做，因为无论如何你都不应该这样做。 ”来回答问题。（例如，在其他一些情况下，您可能正在处理一个比硬盘驱动器大一半的文件！）所以这里是：

您需要使用 aRandomAccessFile代替。使用此类，您可以使用相同的对象读取和写入文件：

public static void removeNthLine(String f, int toRemove) throws IOException {
    RandomAccessFile raf = new RandomAccessFile(f, "rw");

    // Leave the n first lines unchanged.
    for (int i = 0; i < toRemove; i++)
        raf.readLine();

    // Shift remaining lines upwards.
    long writePos = raf.getFilePointer();
    raf.readLine();
    long readPos = raf.getFilePointer();

    byte[] buf = new byte[1024];
    int n;
    while (-1 != (n = raf.read(buf))) {
        raf.seek(writePos);
        raf.write(buf, 0, n);
        readPos += n;
        writePos += n;
        raf.seek(readPos);
    }

    raf.setLength(writePos);
    raf.close();
}
于 2011-06-25T13:05:38.557 回答
2

你不能这样做。FileWriter 只能附加到文件，而不是在文件中间写入 - 如果要在中间写入，则需要 RandomAccessFile。你现在做什么 - 你第一次写入文件时覆盖文件（它变空 - 这就是你得到异常的原因）。您可以创建 FileWriter 并将附加标志设置为 true - 但这样您将附加到文件而不是在文件中间写入。

我真的建议写入一个新文件，然后在最后重命名它。

于 2011-06-25T13:02:16.163 回答
1

@shelley：你不能做你想做的事，更重要的是，你不应该做。您应该出于多种原因读取文件并写入临时文件，一方面，可以这样做（与您尝试执行的操作相反），另一方面，如果进程损坏，您可以打包不丢失原始文件。现在您可以使用 RandomAccessFile 更新文件的特定位置，但这通常在处理固定大小的记录而不是典型的文本文件时完成（根据我的经验）。

于 2011-06-25T12:54:24.283 回答