C++中使用变量存储文件名时无法删除/重命名文件的问题

你的代码存在多个逻辑和文件操作层面的错误，直接导致remove和rename无法正常执行，具体问题如下：

• 文件打开模式错误
  使用fstream打开文件时未指定正确模式：

  • 读取原用户文件的writedata需要指定ios::in只读模式，默认的in|out模式在文件不存在时会打开失败；
  • 写入临时文件的readdata需要指定ios::out | ios::trunc模式，确保清空文件原有内容并以写入模式打开，否则可能无法正确写入或打开失败。

• 循环内提前执行文件操作
  你在while循环内部找到第一个不等于目标书籍的行就立刻关闭文件、删除原文件并重命名临时文件，这会导致：

  1. 原文件中剩余的行完全没处理；
  2. 若目标书籍是文件第一行，该行不会被写入临时文件，但后续行直接被跳过；
  3. 若目标书籍不存在于文件中，删除、重命名操作完全不会触发。
     正确逻辑是遍历完所有行，将不需要删除的内容全部写入临时文件后，再关闭文件、执行删除和重命名。

• 未检查文件打开状态
  没有判断文件是否成功打开，比如原用户文件不存在时，程序会直接进入循环但无任何操作，看起来像执行失败，也没有错误提示。

• 输入处理的潜在问题
  用cin >> book读取书名会忽略空格，无法读取带空格的书名（如"Harry Potter"）；另外，前面的cin >> l_user会在输入缓冲区留下换行符，后续若改用getline读取书名会先读到空行，需要提前清除缓冲区。

修正后的代码 #

#include <iostream>
#include <fstream>
#include <string>
#include <limits>
using namespace std;

int main (){
    string line;
    string f_user;
    string l_user;
    string book;

    cout << "What is your first name? ";
    cin >> f_user;
    cout << "What is your last name? ";
    cin >> l_user;
    string file_name = f_user + l_user + ".txt";

    cout << "What book would you like to return?" << endl;
    // 清除输入缓冲区残留的换行符
    cin.ignore(numeric_limits<streamsize>::max(), '\n');
    getline(cin, book); // 支持读取带空格的书名

    // 以只读模式打开原文件，检查是否成功
    ifstream writedata(file_name);
    if (!writedata.is_open()) {
        cout << "Error: Could not open user file " << file_name << endl;
        return 1;
    }

    // 以写入模式打开临时文件，清空原有内容，检查是否成功
    ofstream readdata("temp.txt");
    if (!readdata.is_open()) {
        cout << "Error: Could not open temp file" << endl;
        writedata.close();
        return 1;
    }

    bool bookFound = false;
    // 遍历所有行，写入不需要删除的内容
    while (getline(writedata, line)){
        if (line != book){
            readdata << line << endl;
        } else {
            bookFound = true;
        }
    }

    // 先关闭所有文件，避免文件占用导致remove/rename失败
    writedata.close();
    readdata.close();

    if (bookFound) {
        // 删除原文件并将临时文件重命名为原文件名
        if (remove(file_name.c_str()) != 0) {
            cout << "Error: Could not delete original file" << endl;
            return 1;
        }
        if (rename("temp.txt", file_name.c_str()) != 0) {
            cout << "Error: Could not rename temp file" << endl;
            return 1;
        }
        cout << "Book Returned successfully" << endl;
    } else {
        cout << "Book not found in your record" << endl;
        // 清理未使用的临时文件
        remove("temp.txt");
    }

    return 0;
}

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