Java删除文件记录异常：原文件未删除、临时文件未重命名问题排查

1. 文件流关闭顺序错误 #

你的流关闭顺序完全颠倒，正确逻辑是从最外层流到最内层流关闭，否则会导致底层流未释放，文件被系统占用，后续删除、重命名操作直接失败。
当前错误顺序：

fr.close();
bw.close();
pw.close();
br.close();
fw.close();

正确顺序：

pw.close(); // 先关最外层打印流
bw.close(); // 再关缓冲写入流
fw.close(); // 最后关文件写入流
br.close(); // 读取流同理，先关缓冲读取流
fr.close(); // 再关文件读取流

更稳妥的方式是使用try-with-resources语法，自动处理流关闭，避免手动顺序错误：

try (FileWriter fw = new FileWriter(tempFile, false);
     BufferedWriter bw = new BufferedWriter(fw);
     PrintWriter pw = new PrintWriter(bw);
     FileReader fr = new FileReader(fileClient);
     BufferedReader br = new BufferedReader(fr)) {
    // 读写逻辑
} catch (Exception e) {
    e.printStackTrace();
}

2. FileWriter误用追加模式 #

你初始化FileWriter时用了new FileWriter(tempFile, true)，第二个参数true表示追加模式。如果临时文件temp.txt之前存在，新内容会追加到旧内容后，导致数据混乱，也会干扰后续重命名操作。应改为覆盖模式：

FileWriter fw = new FileWriter(tempFile, false);
// 或直接省略第二个参数，默认就是覆盖模式
// FileWriter fw = new FileWriter(tempFile);

3. 未校验文件删除结果 #

client.delete()返回boolean值表示删除是否成功，但你未判断结果。如果原文件因被占用、权限不足等原因删除失败，后续renameTo会直接失败（目标文件已存在）。需添加校验：

if (!client.delete()) {
    System.err.println("原文件删除失败，无法完成替换");
    return;
}

4. renameTo方法的局限性与冗余操作 #

你创建File dumy=new File(fileClient)属于冗余操作，直接用newfile.renameTo(new File(fileClient))即可。另外renameTo存在系统兼容性问题（比如Windows下目标文件存在则失败、跨分区无法操作），建议用Java 7+提供的Files.move方法，可靠性更高：

Files.move(newfile.toPath(), new File(fileClient).toPath(), StandardCopyOption.REPLACE_EXISTING);

注意需要导入java.nio.file.Files和java.nio.file.StandardCopyOption。

5. 异常处理过于简略 #

你仅打印System.out.print(e)，只能看到异常类名，无法获取具体错误堆栈，不利于定位问题。应改为打印完整堆栈信息：

catch(Exception e){
    e.printStackTrace();
}

修复后的核心代码片段 #

public void deletClient() {
    int lineRemove = GetLineRemove();
    String tempFile = "temp.txt";
    File newfile = new File(tempFile);
    int line = 0;
    String currentLine;

    try (FileWriter fw = new FileWriter(tempFile, false);
         BufferedWriter bw = new BufferedWriter(fw);
         PrintWriter pw = new PrintWriter(bw);
         FileReader fr = new FileReader(fileClient);
         BufferedReader br = new BufferedReader(fr)) {

        while ((currentLine = br.readLine()) != null) {
            line++;
            if (line != lineRemove) {
                pw.println(currentLine);
            }
        }
        pw.flush();

        // 确保原文件删除成功
        if (!client.delete()) {
            System.err.println("原文件删除失败，无法完成替换");
            return;
        }
        // 使用更可靠的文件移动方法
        Files.move(newfile.toPath(), new File(fileClient).toPath(), StandardCopyOption.REPLACE_EXISTING);

    } catch (Exception e) {
        e.printStackTrace();
    }
}

内容的提问来源于stack exchange，提问作者hodaifa yahia