新手：iterator的小问题

andyY

有个程序编译不过。。。C++新手，STL新手

1. #include <list>
   
2. #include <set>
   
3. #include <algorithm>
   
4. using std::list;
   
5. using std::set;
   
6. 
   
7. typedef struct
   
8. {
   
9.   int id;
   
10.   list<int> l;
    
11. }mc;
    
12. 
    
13. bool operator == (const mc&o, int id )
    
14. {
    
15.   return o.id == id;
    
16. }
    
17. 
    
18. bool operator < (const mc& o, const mc& n)
    
19. {
    
20.   return o.id < n.id;
    
21. }
    
22. 
    
23. int main()
    
24. {
    
25.   mc s;
    
26.   set<mc> mcset;
    
27. 
    
28.   //init
    
29.   for(int i=0;i<5;i++)
    
30.   {
    
31.      s.id = i;
    
32.      mcset.insert( s );
    
33.   }
    
34. 
    
35.   //想在set中查找符合的，在修改它的list
    
36.   //search
    
37.   set<mc>::iterator it = find( mcset.begin(), mcset.end(), 3);
    
38.   if( mcset.end() == it )   
    
39.     return -1;
    
40. 
    
41.   //
    
42.   it->l.push_back(100);
    
43. }

复制代码

刚学习，猜是iterator用错了

[ 本帖最后由 andyY 于 2006-5-26 17:00 编辑 ]

lenovo

编译不过，什么编译错误？
现在的人是越来越懒了。

莫愁

我是回答问题的新手，我也不知道是怎么回事。

andyY

呵呵，对不住

stest.cpp: In function 'int main()'
stest.cpp:44: passing 'const std::list<int, std::allocator<int> >' as 'this' argument of 'void std::list<_Tp, _Alloc>::push_back(const _Tp&) [with _Tp = int, _Alloc = std::allocator<int>]' discards qualifiers

44行就是
it->l.push_back(100);

converse

改成(*it)->l.push_back(100);试试看

westgarden

int main
{

晕!

还有就是加
#include <algorithm>

lenovo

原帖由 converse 于 2006-5-26 16:30 发表
改成(*it)->l.push_back(100);试试看

我认为返回的是一个const list，
所以不能修改这个list。
要是想修改的话，先删除这个元素，
然后再insert吧。
这个和以前讨论的一个帖子很类似，
当时是要修改map的实现吧，
可惜找不到那个帖子了。

converse

6楼说的对...代码没有写好

converse

原帖由 lenovo 于 2006-5-26 16:40 发表

我认为返回的是一个const list，
所以不能修改这个list。
要是想修改的话，先删除这个元素，
然后再insert吧。
这个和以前讨论的一个帖子很类似，
当时是要修改map的实现吧，
可惜找不到那个帖子了。

当时没有仔细看楼主的帖子,只是从经验上面反应iter应该是智能指针来用的,我以前也犯过这个错误

gooderfeng

我认为返回的是一个const list，
所以不能修改这个list。

我调试以下确实是返回const list
但是不知道原因。
斑竹帮忙找找哪个帖子啊。