C++ fstream文件在读取结束时打印垃圾

我写了一个程序来练习这种语言，但是我从代码中得到了一些非常奇怪的输出，这对我来说似乎是正确的

守则：

#include <iostream>
#include <fstream>
#include <list>
struct Car
{
    std::string make;
    std::string model;
    int partNo;
    double price;
    int quantity;
    std::string partname;
};
void AddItem();
void _Update(int PartNo, int quantity);
void UpdateList(std::list<Car>& _Car);
int main()
{
    std::list<Car> _Car;
    UpdateList(_Car);
    for(std::list<Car>::iterator iter = _Car.begin(); iter != _Car.end(); iter++)
     {
         std::cout << iter->make << " " << iter->model << " " << iter->partNo << " " << iter->price << " " << iter->quantity << " " << iter->partname << std::endl;
     }
}

void UpdateList(std::list<Car>& _Car)
{
    std::ifstream File("CarParts.txt");
    if(!File.is_open())
        std::cerr << "Bad file input....... closing....";

    while(!File.eof())
    {
        Car tempObj;
        File >> tempObj.make >> tempObj.model >> tempObj.partNo >> tempObj.price >> tempObj.quantity;
        getline(File,tempObj.partname);
        _Car.push_back(tempObj);
    }
    File.close();

}

记事本文件：

Pajero NA1H25 1 3.65 11 BLADE W/S WIPER
HondaSivic R34gFk 2 4.97 15 ENGINE CHANGE

---

我想要打印出来的两行下面的三个数字是什么？我真的很困惑。。。如果你能帮忙，谢谢你

对于初学者，将程序的名称空间设置为std，以避免讨厌的std:：string etc语法

在文件顶部附近的“导入”下，插入： 使用名称空间std

看起来目标是从文件中填充列表

看看这部分：

 File >> tempObj.make >> tempObj.model >> tempObj.partNo >> tempObj.price >> tempObj.quantity;
 getline(File,tempObj.partname);

---

我认为更好的解决方案是使用getline解析每一行，然后使用空格delimeter或逗号相应地解析数据。空格分隔符可能很精细

> P>这是C++中读取文件数据时遇到的一个常见问题。问题是您如何使用

eof

。该标志仅在尝试读取数据失败后设置

因此，在阅读了前两行之后，它仍然没有到达文件的末尾。它一直读取到它，但尚未设置

eof

。然后它将第三次循环，尝试读取2行，然后退出。问题是，在将第三个循环的结果推到汽车列表中之前，您从未检查过eof

在您的情况下，您可以在调用

getline

之后将

eof

检查移动到，或者使用

getline

返回值

例如：

while(true)
{
    Car tempObj;
    File >> tempObj.make >> tempObj.model >> tempObj.partNo
         >> tempObj.price >> tempObj.quantity;
    if (!getline(File,tempObj.partname)) break;
    _Car.push_back(tempObj);
}

---

这将检查数据在推送之前是否已成功读取。

在文件顶部的“导入”下，插入：using namespace std

这是一个非常糟糕的建议，可能会引入bug。使用getLine（）callz，您将整行放入变量tempObj.partname中。为什么使用std名称空间是一个糟糕的做法？如果您没有实现std库中没有包含的任何内容，它将创建更干净的代码和一个通用名称空间。我建议永远不要使用它。不，使用“讨厌的”

std:：

语法。你很快就习惯了，它不需要太多的打字，而且它使代码更具可读性（我一眼就能看出某个类型是来自标准库还是某个自制库。它还避免了名称冲突。如果你真的想，你可以在.cpp文件的顶部放置一个

，使用namespace std

，但决不在头文件中。即使在.cpp文件中，我发现没有它也会更好。哦，没有成功作为C++中的“导入”，它不仅仅是一个不同的名字，它所做的不同于我编译和运行你所发布的代码的“导入库”，它运行得很好。与杰西好一样，输出与记事本文件中的输出相同。

while(true)
{
    Car tempObj;
    File >> tempObj.make >> tempObj.model >> tempObj.partNo
         >> tempObj.price >> tempObj.quantity;
    if (!getline(File,tempObj.partname)) break;
    _Car.push_back(tempObj);
}