2012-06-17 60 views
1

我看过类似的问题,但我根本看不出什么问题。 我试过:: const_iterator。但是gcc不会以任何方式进行编译。标准:地图不匹配运营商=

它是关于:i = allFunctions.erase(i);

void removeEventListener(const std::string &type, function listener){ 
    if(!hasEventListener(type)) 
     return; 

    std::map<int, std::list<function> > &allFunctions = eventHandlerList[type]; 
    std::map<int, std::list<function > >::iterator i; 

    for(i=allFunctions.begin(); i!=allFunctions.end(); ++i) 
    { 
     i->second.remove(listener); 

     if(i->second.empty()) 
     { 
      i = allFunctions.erase(i); 

     } 
    } 

    if(allFunctions.empty()) 
     eventHandlerList.erase(type); 
} 

误差常量性:

Error: passing const std::list<void (*)(const st::event::Event&)> as 
this argument of void std::list<_Tp, _Alloc>::remove(const 
value_type&) [with _Tp = void (*)(const st::event::Event&), _Alloc = 
std::allocator<void (*)(const st::event::Event&)>, std::list<_Tp, 
_Alloc>::value_type = void (*)(const st::event::Event&)] discards qualifiers [-fpermissive] 

Error: no matching function for call to std::map<int, std::list<void 
(*)(const st::event::Event&)> >::erase(std::map<int, std::list<void 
(*)(const st::event::Event&)> >::const_iterator&) 

误差的迭代器:

no match for operator= in i = (& allFunctions)->std::map<_Key, _Tp, _Compare, _Alloc>::erase [with _Key = int, _Tp = std::list<void (*)(const st::event::Event&)>, _Compare = std::less<int>, _Alloc = std::allocator<std::pair<const int, std::list<void (*)(const st::event::Event&)> > >, std::map<_Key, _Tp, _Compare, _Alloc>::iterator = std::_Rb_tree_iterator<std::pair<const int, std::list<void (*)(const st::event::Event&)> > >](i) 

有什么建议?

+0

什么是eventHandlerList定义为? –

+0

std :: map >> eventHandlerList; 虽然函数是一个函数指针的typedef – Sidar

+1

这是在C++ 11吗? –

回答

3

试试这个一般关联容器擦除循环:

for (std::map<int, std::list<function> >::iterator it = allFunctions.begin(); 
    it != allFunctions.end(); /* no increment! */) 
{ 
    it->second.remove(listener); 

    if (it->second.empty()) { allFunctions.erase(it++); } 
    else     { ++it;      } 
} 

C++ 11改变了签名和返回的成员 - erase功能类型,然后你可以有一个单行it = allFunctions.erase(it);

+0

奇怪的是,对于std :: lists,它工作得很好。 但你的代码似乎工作。谢谢。也许我应该用一段时间来代替。 – Sidar