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

const_iterator のエラー:

 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)

助言がありますか？

4

1 に答える 1

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);。

于 2012-06-17T19:03:59.530 に答える