lrecord是list容器，元素为含有整型、浮点、string的结构。要求是从一个文件里读取数据。确保文件打开正确，但是在读取第一条之后，往lrecord里添加新元素时就崩溃了

while(!feof(fin))
        {
            lrecord.push_back(nrecord);
            fread(&nrecord,sizeof(record),1,fin);
        }

 

会不会是因为那个结构没有定义复制函数？

或者考虑下内存问题？

请详细说一下？启用cppcheck之后发现定义这个结构的地方有警告，warning: 'rr::id' should be initialized in the member initialization list，会不会跟这有关？但是搜不到解决办法，怎么消掉？

按提示理解 应该是说你的结构体没有被初始化 以致后面的数据没法使用 你可以试着在结构体这块上将其初始化看看效果

怎么初始化啊？以前经常这样用，好像都没问题

这个我不知道怎么说 因为我不知道你的代码是怎样的 我都只是猜测的

#include <iostream>
#include <cstdlib>
#include <cstdio>
#include <list>
using namespace std;
typedef struct rr
{
    int id,num;
    double price;
    string name;
}record;
int main()
{
    record nrecord;
    list<record> lrecord;
    list<record>::iterator rite;
    int total=-1;
    FILE *fin,*fout;
    char look;
    cout<<"请输入要处理的商品数量，100以内，0表示建立新文件:";
    cin>>total;
    while(total<0||total>100)
        cout<<"输出有误，请重新输入:";
    if(total==0)
    {
        fout=fopen("commodity.dat","wb");
    }
    else
    {
        if(!(fin=fopen("commodity.dat","rb")))
        {
            cout<<"对不起，文件不存在。";
            return 0;
        }
        fread(&nrecord,sizeof(record),1,fin);
        while(!feof(fin))
        {
            lrecord.push_back(nrecord);
            fread(&nrecord,sizeof(record),1,fin);
        }
        fclose(fin);
        cout<<"是否查看原来的记录？y/n";
        cin>>look;
        if(look=='y'||look=='Y')
        {
            rite=lrecord.begin();
            if(rite==lrecord.end())
                cout<<"无记录。"<<endl;
            else
                for(;rite!=lrecord.end();++rite)
                {
                    cout<<"ID"<<rite->id<<" 名称"<<rite->name<<" 数量"<<rite->num<<" 价格"<<rite->price<<endl;
                }
        }
        fout=fopen("commodity.dat","wb");
        while(total--)
        {
            cout<<"请输入商品名称:";
            cin>>nrecord.name;
            cout<<"请输入另外三个数据:";
            cin>>nrecord.id>>nrecord.num>>nrecord.price;
            for(rite=lrecord.begin();rite!=lrecord.end()&&rite->id<nrecord.id;++rite);
            if(rite!=lrecord.end()&&rite->id==nrecord.id)
                *rite=nrecord;
            else    lrecord.insert(rite,nrecord);
        }
        for(rite=lrecord.begin();rite!=lrecord.end();++rite)
        {
            fwrite(&(*rite),sizeof(record),1,fout);
            cout<<"ID"<<rite->id<<" 名称"<<rite->name<<" 数量"<<rite->num<<" 价格"<<rite->price<<endl;
        }
        fclose(fout);
    }
    cout<<"文件已保存，谢谢使用。";
    return 0;
}

結構record不要用string，那是不定長的，每個sizeof()的尺寸不一樣。第一次讀文件時，nrecord是空殼，name=""，但讀文件後是文件中第一個記錄的name長度，此時nrecord的尺寸改變，不過此尺寸未必正好是第二個記錄的記錄尺寸，後面就出錯了。如果讀寫文件，穩妥的辦法是record結構用固定長度的數組。注意：string是可變長度的容器，而你用了sizeof()求尺寸。

感谢版主，解答真详细啊。那么用char[]的话可以吧？这样的结构可以直接用insert添加是吗？