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 回答