C++中ifstream无法正确加载多个文件的问题求助

核心问题 #

你的代码有两个关键问题导致输出不符合预期：

1. 重复初始化随机种子：每次读取文件都调用srand((unsigned)time(NULL))，由于time(NULL)返回秒级时间戳，短时间内多次调用会让随机种子重复，导致rand()生成相同的随机数。
2. lines容器未清空：每次读取新文件时，你都把内容追加到lines里，但取随机行时用的是当前文件的行数范围（rand() % total_lines），这个索引对应的是lines容器的前N个元素——而这些元素是上一个文件的内容，不是当前文件的。比如读姓文件时，total_lines是姓文件的行数，但lines里已经存了所有名字，你取的lines[random_number]还是名字的内容。

修复后的代码 #

把重复逻辑封装成函数，只初始化一次随机种子，每个文件用独立的局部容器存储内容：

#include <iostream>
#include <fstream>
#include <vector>
#include <string>
#include <cstdlib>
#include <ctime>

// 读取指定文件并返回随机一行内容
std::string getRandomLine(const std::string& filename) {
    std::vector<std::string> lines;
    std::ifstream file(filename);
    std::string line;

    if (!file.is_open()) {
        std::cerr << "无法打开文件: " << filename << std::endl;
        return "";
    }

    while (getline(file, line)) {
        lines.push_back(line);
    }
    file.close();

    if (lines.empty()) {
        return "";
    }

    int random_idx = rand() % lines.size();
    return lines[random_idx];
}

int main() {
    // 仅初始化一次随机种子，放在程序开头
    srand((unsigned)time(NULL));

    std::string first_name = getRandomLine("englishfirstname.txt");
    std::string surname = getRandomLine("englishsurname.txt");
    std::string address = getRandomLine("addresses.txt");

    std::cout << "Full name: " << first_name << " " << surname << std::endl;
    std::cout << "Address: " << address << std::endl;

    return 0;
}

关键优化点 #

• 封装复用逻辑：把读取文件+随机选行的代码做成函数，避免重复冗余，后续新增数据源只需调用函数即可。
• 单次随机种子初始化：放在main开头，确保随机数序列真正具有随机性，不会因为短时间重复调用种子导致随机数重复。
• 局部容器隔离数据：每个文件读取时用独立的局部lines容器，彻底避免不同文件内容的混淆问题。

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