C++ vector对象写入文件异常：重复写入与仅最后对象留存咨询

兄弟，我帮你捋捋你代码里的坑，问题其实出在两个核心逻辑错误上，咱们拆解来看：

1. 为什么文件只留存最后一个对象？ #

你每次调用write_boats函数时，用ofstream file("records_file.txt");打开文件——这行代码的默认行为是清空文件原内容，然后写入新数据（对应ios::trunc模式）。

比如你注册了2个Boat对象，save_records循环调用每个对象的write_boats：

• 第一次调用第一个对象的write_boats：清空文件，写入该对象的数据；
• 第二次调用第二个对象的write_boats：再次清空文件，覆盖掉之前的内容，写入第二个对象的数据；
  最终文件里自然只剩最后一次写入的内容。

2. 为什么会重复输出同一条数据？ #

这是因为你的write_boats函数犯了两个致命错误：

• 错误访问成员变量：你在成员函数里遍历全局（或静态）的berths向量，但写入的却是当前调用该函数的对象的owner_name、boat_name等变量，而不是循环变量b的成员！比如调用berths[1].write_boats()时，不管循环遍历多少个b，每次写入的都是berths[1]的数据。
• 双重循环导致重复写入：save_records已经在遍历berths向量，每个元素都调用一次write_boats，而write_boats内部又遍历一次berths——相当于做了嵌套循环，自然会重复输出多次同一内容。

比如你注册1个对象时，如果save_records循环执行了1次，write_boats内部循环又执行1次，就会写入1次？但你说写入两次，大概率是你的save_records循环起始索引写错了（比如从0开始而不是1），或者额外有其他地方调用了write_boats，但核心逻辑问题就是这两个。

修复方案 #

咱们重构代码，把文件操作和对象遍历分开，避免重复打开文件和错误的变量访问：

方案1：用静态函数一次性写入所有对象 #

这种方式最简洁，把写入所有数据的逻辑放在一个函数里，只打开一次文件：

#include <fstream>
#include <vector>

// 假设你的Boat类定义
class Boat {
public:
    std::string owner_name;
    std::string boat_name;
    std::string type;
    double length;
    double draft;

    // 成员函数：负责写入当前对象的数据到已打开的文件
    void write_single(std::ofstream& file) const {
        file << owner_name << "; " << boat_name << "; " << type << "; " << length << "; " << draft << '\n';
    }

    // 静态函数：一次性写入所有Boat对象到文件
    static void write_all(const std::vector<Boat>& berths) {
        // 用ios::out模式会覆盖旧文件，要追加的话改成ios::app
        std::ofstream file("records_file.txt", std::ios::out);
        if (!file.is_open()) {
            // 这里可以加个错误提示，比如cout << "文件打开失败！" << endl;
            return;
        }
        for (const auto& boat : berths) {
            boat.write_single(file);
        }
        file.close();
    }
};

// 保存记录的函数直接调用静态函数即可
void save_records(const std::vector<Boat>& berths) {
    Boat::write_all(berths);
}

方案2：分开文件打开和对象写入 #

如果你想保留成员函数的写法，就把文件打开逻辑放在save_records里，只打开一次，然后遍历每个对象写入：

// 修改Boat类的成员函数，只负责写入自身数据
void Boat::write_single(std::ofstream& file) const {
    file << owner_name << "; " << boat_name << "; " << type << "; " << length << "; " << draft << '\n';
}

// 保存记录的函数：只打开一次文件
void save_records(const std::vector<Boat>& berths) {
    std::ofstream file("records_file.txt", std::ios::out);
    if (!file.is_open()) {
        return;
    }
    for (const auto& boat : berths) {
        boat.write_single(file);
    }
    file.close();
}

关键注意点 #

• 不要重复打开文件：每次打开默认会清空内容，所以应该只打开一次，写完所有数据再关闭。
• 正确访问循环变量：遍历向量时，要写入循环变量的成员（比如b.owner_name），而不是当前对象的成员。
• 避免双重循环：不要外层遍历向量，内层又遍历一次，这会导致重复写入。

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