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;
}

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

原来如此啊，还真不知道。以前没有用过string，因为最近才开始学C++…………那么warning: 'rr::id' should be initialized in the member initialization list怎么解决呢？这个id是int类型的，其他成员也有同样的警告

谢谢大家啦！~