使用while(true)循环读取文件时，执行break后仍多一次迭代的问题

问题根源 #

myfile.eof()的触发逻辑是只有当读取操作失败（比如已读到文件末尾，再读无内容）时，才会返回true，而非文件指针刚到达末尾就返回true。

对应你的场景：

• 成功读取最后一个数字4后，文件指针已到末尾，但此时eof()仍为false，循环会继续执行。
• 下一次循环中，尝试读取新数值时失败，但存储数值的变量还保留着上一次的4，你将这个旧值再次存入数组，之后才触发eof()为true并执行break，最终数组里多了一个重复的4。

关于break语句 #

break本身完全适合用于退出这个循环，问题出在你判断退出的时机不对，和break没有关系。

正确的写法 #

把读取操作和退出判断结合，确保只有读取成功时才将数值存入数组：

#include <fstream>
#include <vector>
#include <iostream>

using namespace std;

int main() {
    vector<int> arr;
    ifstream myfile("numbers.txt");
    int num;

    // 写法1：直接将读取操作作为循环条件
    while (myfile >> num) {
        arr.push_back(num);
    }

    // 写法2：保留while(true)结构，调整判断时机
    /*
    while (true) {
        myfile >> num;
        // 判断读取是否失败（含eof、格式错误等情况）
        if (!myfile) {
            break;
        }
        arr.push_back(num);
    }
    */

    // 输出验证
    for (int n : arr) {
        cout << n << " ";
    }
    return 0;
}

这两种写法都会在读取失败的第一时间退出循环，不会重复存入旧值，最终输出为1、2、3、4。

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