Java删除车辆记录时文件无法删除、临时文件无法重命名问题求助

问题根源与对应修复 #

1. 原cars.csv无法删除

原因：执行删除操作时，读取原文件的FileReader和BufferedReader仍处于打开状态，文件被Java进程锁定，操作系统拒绝删除请求。
修复：先关闭所有读取原文件的流，再执行删除操作。

2. temp.csv重复写入数据

原因：创建FileWriter时启用了追加模式（第二个参数传了true），每次运行都会在temp文件原有内容后追加新数据，而非清空重写。
修复：改为覆盖模式，去掉第二个参数或传false。

3. temp.csv无法重命名为cars.csv

原因：

• 写入temp文件的流未关闭，文件被进程占用；
• 原文件删除失败（问题1导致），目标路径已有文件存在，renameTo方法无法覆盖现有文件。
  修复：先关闭所有写入temp文件的流，确认原文件删除成功后再执行重命名。

修复后的完整代码 #

private static void removeCar(String vin, String make, String model, int year, int mpg, double price) {
    CarRespository carRespository = new CarRespository();
    String tempFile = "src/temp.csv";
    String filePath = "src/cars.csv";
    File oldFile = new File(filePath);
    File newFile = new File(tempFile);
    int line = 0;
    int deleteLine = -1;
    String currentLine = "";

    // 定位要删除的行号
    for (int i = 0; i < carRespository.getListOfCars().size(); i++) {
        Car car = carRespository.getListOfCars().get(i);
        if (car.getVinNumber().equalsIgnoreCase(vin)
                && car.getMake().equalsIgnoreCase(make)
                && car.getModel().equalsIgnoreCase(model)
                && car.getYear() == year
                && car.getMpg() == mpg
                && car.getCarPrice() == price) {
            deleteLine = i + 1;
            System.out.println("要删除的行号：" + deleteLine);
            break; // 找到匹配后直接跳出循环，避免无效迭代
        }
    }

    if (deleteLine == -1) {
        System.out.println("未找到匹配的车辆信息！");
        customerWalkedIn();
        return; // 提前返回，避免执行后续无用操作
    }

    BufferedReader br = null;
    PrintWriter pw = null;
    try {
        // 修复点1：覆盖模式创建FileWriter，清空temp文件后写入
        pw = new PrintWriter(new BufferedWriter(new FileWriter(tempFile, false)));
        br = new BufferedReader(new FileReader(filePath));

        // 读取原文件并写入temp（跳过要删除的行）
        while ((currentLine = br.readLine()) != null) {
            line++;
            if (deleteLine != line) {
                pw.println(currentLine);
            }
        }

        // 修复点2：先关闭所有文件流，释放文件占用
        if (br != null) br.close();
        if (pw != null) {
            pw.flush();
            pw.close();
        }

        // 修复点3：确保原文件删除成功后再重命名
        if (oldFile.delete()) {
            if (!newFile.renameTo(oldFile)) {
                System.out.println("重命名文件失败，请检查文件权限");
            }
        } else {
            System.out.println("删除原文件失败，请检查文件是否被占用");
        }

    } catch (Exception e) {
        System.out.println("操作异常：" + e.getMessage());
        e.printStackTrace();
    } finally {
        // 确保流最终被关闭，避免资源泄漏
        try {
            if (br != null) br.close();
            if (pw != null) pw.close();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}

额外优化点 #

• 找到匹配车辆后添加break，减少无效循环；
• 未找到匹配时提前return，避免浪费资源；
• 使用finally块强制关闭流，防止资源泄漏；
• 增加操作结果提示，方便调试排查。

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