一个list读取文件的简单问题

[arden1019]

我用下面的函数来将一个formatted文件按行读到一个list中：

bool readftl(char *m_filename,list <char *> m_flist)
{
char rowdata[256];
//int rowindex=0;
fstream listfile;
listfile.open(m_filename,ios::in);

while(!listfile.eof())
{
listfile.getline(rowdata,256,'\n');
//cout<<rowdata<<endl;
m_flist.push_back(rowdata);
//cout<<"\nflist : "<<m_flist.front()<<endl;
}
for (flistitor=m_flist.begin();
flisritor!=m_flist.end();
++flistitor) {
// dereference the iterator to get the element
cout << *flistitor << endl;
}
return true;
}

其中的flist ： list<char*> flist;
flistitor: list<char*>::iterator flistitor;

为什么我的cout << *flistitor << endl; 不能正确显示出文件的内容？？？

我用把flist和flistitor 都按参数传进来么？我已经在整个程序的区域声明了他们。

[oo]

listfile.getline(rowdata,256,'\n');
//cout<<rowdata<<endl;
m_flist.push_back(rowdata);
========================================
rowdata是个局部数组，你所有的节点保存的都是一样的，
而且退出这个函数后，里面的内容是随机的。

应该这样：
把前面的rowdata定义去掉；
char *rowdata = new char[256];
listfile.getline(rowdata,256,'\n');
//cout<<rowdata<<endl;
m_flist.push_back(rowdata);

另外要注意，你在把list删除前，别忘了 delete[]；

[goodluckyxl]

楼上的说的应该是关键

只能用重新申请的办法

静态的都不能用

[freefalcon]

前面已经说了

这里再提一点，STL容器都是值语义的，即容器是用来装对象，而不是用来装指针，尽管后者可行，但你务必要小心谨慎，自己做好内存管理

既然是基于对象，那么这里如果用string来作为list的元素就不存在任何问题，关键代码
list<string> flist;
string line;
getline(listfile, line);
flist.push_back(line);

[arden1019]

让我来试试....看

[Squall1009]

转指针的话强烈建议使用vector效率比list高

[freefalcon]

晕,先写好了,在提交时又没了
这次说简单点

原因就是，你只申请了一块内存，list存的是指向这块内存的指针，你每次push的都是同一个指针，于是list里的元素都指向同一块内存，此乃问题1。
问题2在于你没次读取文件用的仍然是同一块内存，所以后读取的数据就覆盖了前面的值，这样不会有问题吗？

[carylin]

同意，用list<string>管理好了

[arden1019]

oooooo哦..........我明白了，哈哈大彻大悟哈哈哈哈爽。

[arden1019]

to ：freefalcon 能不能让我把你放到我的msn名单中？
我的是[e-mail]

[arden1019]

还按错的来说：

我 list<char*> flist;
list<char*>::iterator flistitor; 以后，要取得一个list中的数据，我*flistitor后应该得到的是 char* ，对不？ 那我改为 **flistitor 能不能得到char*指向的数据呢？

那个char rowdata[256]; 我是应该把它改成 char *rowdata=new char[256]；我改过来后，cout<<**flistitor 仍然是空的。
如果我把//cout<<"\nflist : "<<m_flist.front()<<endl;的注释去掉，它倒是能显示出正确的数据。
这能说明我已经把文件的数据读到list中了吧？
为什么？难道
for (flistitor=m_flist.begin();
flisritor!=m_flist.end();
++flistitor) {
// dereference the iterator to get the element
cout << *flistitor << endl;
有问题？不该这样遍历一个list么？