C++从文件读取整数存入数组时输出与文件内容无关异常值的排查

核心错误点说明 #

• 文件流位置错位，无有效数据可读取
  当你通过distance(istream_iterator<int>(fs1), istream_iterator<int>())计算文件整数个数时，遍历过程会直接消耗流中的所有数据，计算完成后fs1、fs2的读写指针已经移动到文件末尾，同时流的eof失败状态位被置位。后续你试图从流中读值到数组时，没有任何剩余可读内容，数组中存储的全是栈内存的随机脏值，这就是输出全是无意义大数的根本原因。
• 读写逻辑完全串线
  你声明了a1、a2两个数组用于分别存储两个文件的内容，但后续写的读入循环判断条件依赖fs1的状态，实际操作却是从fs2读值写入a2；同时由于fs1计算完长度后已经处于eof状态，这个循环从头到尾一次都不会执行，两个数组从来没有被写入过任何有效数据——你看到file2.txt本该输出4、实际输出16，就是因为16是数组内存块上残留的随机值，和文件内容完全无关。此外循环里的计数器n没有绑定数组长度边界，就算循环能正常执行，也会很快出现数组越界写入的问题。
• 使用非标准语法的变长数组
  int a1[s1], a2[s2] 属于C语言的变长数组扩展，并非标准C语法，仅GCC编译器兼容支持，换用其他标准C编译器会直接报语法错误，这种写法也不适合在正式开发中使用。
• 错误的EOF判断范式
  while(!fs.eof()) 是C++文件操作的经典错误写法：eof标记只有在读操作失败时才会被系统置位，提前判断eof很容易导致循环多执行一次，往数组中写入无效值。

修正实现思路 #

1. 不需要先遍历一遍文件计算长度，直接使用标准库的vector<int>存储读入的整数，容器会自动根据实际读取的内容扩容，从根源上避免流位置错位、变长数组不兼容的问题。
2. 读文件时直接把读操作本身作为循环判断条件（while(文件流 >> 临时变量)），读成功就把值存入容器，读失败（到达文件末尾、格式错误）就自动退出循环，完全规避eof判断的坑。
3. 严格匹配文件流和对应的存储容器，不要出现读写交叉的逻辑错误。

修正后可正常运行的代码示例 #

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

int main()
{
    string f1, f2;
    cout<<"Please enter the names of the files you would like to merge, file extensions included."<<endl;
    cout<<"File 1 Name: ";
    cin>>f1;
    cout<<"File 2 Name: ";
    cin>>f2;

    ifstream fs1(f1), fs2(f2);
    if (!fs1.is_open() || !fs2.is_open())
    {
        cout<<"No file exists with that name."<<endl;
        return 0;
    }

    vector<int> a1, a2;
    int temp_val;
    // 读取file1内容存入a1
    while (fs1 >> temp_val)
    {
        a1.push_back(temp_val);
    }
    // 读取file2内容存入a2
    while (fs2 >> temp_val)
    {
        a2.push_back(temp_val);
    }

    // 测试输出验证读取结果
    cout << "File1读取的整数序列：" << endl;
    for (int num : a1)
    {
        cout << num << " ";
    }
    cout << endl << "File2读取的整数序列：" << endl;
    for (int num : a2)
    {
        cout << num << " ";
    }
    cout << endl;

    return 0;
}

如果你坚持要保留“先统计长度、再开数组读内容”的逻辑，在调用distance计算完长度之后，必须先调用fs1.clear()、fs2.clear()清除流的eof失败状态，再用fs1.seekg(0, ios::beg)、fs2.seekg(0, ios::beg)把读写指针重置回文件开头，否则即使修正了循环的读写匹配逻辑，也因为流处于错误状态、指针在文件末尾，完全读不到有效内容。

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