C++中ifstream读取Double后状态异常触发错误的原因

我正在编写一个从文件读取数据的方法，文件格式如下：

[String] [Integer] [Double]
[String] [Integer] [Double]

我使用以下循环从文件读取每一行，并将其存储为自定义类型Car的vector：

while(ifs.good()){
    Car car;
    ifs >> car.carName;
    ifs >> car.yearBuilt;
    if(!ifs.good()) throw std::invalid_argument("Year must be a integer");
    ifs >> car.price; 
    if(!ifs.good()) 
      throw std::invalid_argument("Price must be a double");
    to_return.push_back(car);
  }

然而，尽管文件格式正确、字段符合要求，函数仍会抛出“Price must be a double”的错误。我使用的测试文件内容如下：

Mercedes 1998 2364.12
Audi 2012 32645.79

如果删除判断读取double后的状态的if语句，文件就能被正确解析，vector也能正常工作。我困惑的是为何会触发该错误，以及读取double后处于fail()状态时为何文件状态会自行恢复。

问题根源 #

1. 循环条件逻辑错误：while(ifs.good())是在读取操作前检查流状态。当文件中所有有效数据读取完成后，流的good()状态仍为true（只有尝试读取超出文件末尾的内容时，才会设置eofbit，导致good()返回false），这会让循环多执行一次。
2. 最后一次循环的失败连锁：进入最后一次循环后，尝试读取carName失败，流状态变为失败，但代码未检查carName的读取结果，继续执行yearBuilt和price的读取操作——这些操作全部失败，最终触发price读取后的状态判断，抛出异常。
3. 删除判断后的“假象正常”：删除price的状态判断后，最后一次循环中虽然读取全部失败，但代码仍执行了push_back(car)，此时car是未初始化的垃圾值。只是你只关注有效数据，可能没注意到vector中多了一个无效元素；或者某些情况下流的错误状态被隐式处理，但这完全不可靠。

正确实现方式 #

把读取操作直接放在循环条件中，利用>>运算符返回流对象的特性，只有所有字段读取成功才进入循环体：

std::vector<Car> to_return;
Car car;
// 三个字段全部读取成功才进入循环
while (ifs >> car.carName >> car.yearBuilt >> car.price) {
    to_return.push_back(car);
}

// 如果需要单独处理每个字段的错误，可拆分读取并检查：
/*
while (true) {
    Car car;
    if (!(ifs >> car.carName)) {
        if (ifs.eof()) break; // 正常文件结束，不抛异常
        throw std::invalid_argument("Failed to read car name");
    }
    if (!(ifs >> car.yearBuilt)) {
        if (ifs.eof()) break;
        throw std::invalid_argument("Year must be an integer");
    }
    if (!(ifs >> car.price)) {
        if (ifs.eof()) break;
        throw std::invalid_argument("Price must be a double");
    }
    to_return.push_back(car);
}
*/

额外说明 #

• 流的good()状态仅在eofbit、failbit、badbit都未设置时为true，用它做循环条件极易导致多执行一次循环。
• 每次读取后都应检查流状态，尤其在需要精确错误提示的场景下，要区分正常文件结束（eofbit）和读取失败（failbit），避免误抛异常。

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