C++读取文件解析为结构体数组后遍历输出乱码问题

听起来你已经走了大半路了——能打印当前行说明文件读取和单条解析逻辑是通的，问题大概率出在结构体数据的持久化存储或者内存管理上。我帮你捋几个最常见的坑，再给你一套可运行的参考代码：

先排查核心问题点 #

• 字符串终止符缺失：如果结构体里用char[]存储站点名称，但读取时没手动加'\0'，或者字符串长度刚好等于数组长度，打印时就会越界读内存，出现乱码。
• 野指针陷阱：要是用char* station;做结构体成员，直接把临时缓冲区地址赋值给它，临时缓冲区销毁后，结构体里的指针就指向无效内存，遍历自然乱码。
• 栈数组溢出：如果结构体数组定义在栈上（比如WeatherData data[1000];），但文件行数远超数组大小，会触发栈溢出，覆盖内存导致乱码甚至崩溃。

参考实现代码 #

给你一个正确的结构体定义和解析流程，你可以对照自己的代码找差异：

#include <iostream>
#include <fstream>
#include <sstream>
#include <vector>
#include <cstring>

// 定义匹配文件格式的结构体，用固定大小char数组存储字符串，规避指针问题
struct WeatherRecord {
    char station[32];  // 预留足够空间，避免溢出
    int year;
    int month;
    int day;
    int hour;
    float temperature;
};

int main() {
    std::ifstream input_file("weather_data.txt");
    if (!input_file.is_open()) {
        std::cerr << "Failed to open file!" << std::endl;
        return 1;
    }

    // 用vector动态存储，替代栈上固定数组，自动扩容避免溢出
    std::vector<WeatherRecord> records;
    std::string line;

    while (std::getline(input_file, line)) {
        // 打印当前行，验证读取正确性
        std::cout << "Current line: " << line << std::endl;

        std::istringstream line_stream(line);
        WeatherRecord temp_record;

        // 解析每行数据：直接读取到char数组，istream遇到空格会自动停止
        line_stream >> temp_record.station >> temp_record.year >> temp_record.month 
                    >> temp_record.day >> temp_record.hour >> temp_record.temperature;

        // 手动确保字符串终止（保险操作，避免极端情况）
        temp_record.station[sizeof(temp_record.station)-1] = '\0';

        records.push_back(temp_record);
    }

    input_file.close();

    // 遍历输出验证结果
    std::cout << "\nParsed records:" << std::endl;
    for (const auto& rec : records) {
        std::cout << rec.station << " " << rec.year << " " << rec.month << " "
                  << rec.day << " " << rec.hour << " " << rec.temperature << std::endl;
    }

    return 0;
}

关键细节说明 #

• 固定大小char数组+终止符：结构体里的station用足够大的char[]，读取后手动设置最后一位为'\0'，彻底避免字符串越界问题。
• vector动态存储：替代栈上固定数组，自动适配文件行数，从根源上解决栈溢出风险。
• 规避指针赋值：不要把临时字符串的指针直接存到结构体里，要么用strcpy拷贝，要么直接用istream读取到char数组（像代码里那样）。

如果想更省心，也可以把结构体里的char station[32];换成std::string station;，这样完全不用管终止符和内存管理，代码会更简洁。

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