2

このメソッドは中止エラーを引き起こします:失敗"map/set iterator not incrementable." した後、if消去する必要がある有効なイテレーターが決定された (そして現在ある)ため、有効なオブジェクト/ポインターではない++_iterため、failsを介してマップ内の次のイテレーターに進みます。_iter

マップを反復処理し、全体を通して個々のアイテムを削除する機能を持つための正しい手順は何ですか?

typedef std::map<std::string, BITMAP*> MapStrBmp;
typedef MapStrBmp::iterator MapStrBmpIter;
\\...
void BitmapCache::CleanCache() {
    //Clean the cache of any NULL bitmaps that were deleted by caller.
    for(MapStrBmpIter _iter = _cache.begin(); _iter != _cache.end(); ++_iter) {
        if(_iter->second != NULL) {
            if((_iter->second->w < 0 && _iter->second->h < 0) == false) continue;
        }
        _cache.erase(_iter);
    }
}
4

4 に答える 4

6

もう少し注意する必要があります。

void BitmapCache::CleanCache() {
    //Clean the cache of any NULL bitmaps that were deleted by caller.
    for(MapStrBmpIter _iter = _cache.begin(); _iter != _cache.end(); ) {
        if(_iter->second != NULL) {
            if((_iter->second->w < 0 && _iter->second->h < 0) == false)
            {
                ++_iter;
                continue;
            }
        }

        _cache.erase(_iter++);
    }
}
于 2012-01-11T22:15:45.967 に答える
2

map::erase(iterator)消去後にマップ内の次の要素 (存在する場合) を指す反復子を提供します。したがって、次のことができます。

for(MapStrBmpIter _iter = _cache.begin(); _iter != _cache.end(); ) {
    if(_iter->second != NULL) {
        if((_iter->second->w < 0 && _iter->second->h < 0) == false) {
           ++_iter;
           continue;
        }
    }
    _iter = _cache.erase(_iter);
}
于 2012-01-11T22:19:42.587 に答える
1

連想コンテナの標準的な消去ループ:

for (auto it = m.cbegin(); it != m.cend() /* not hoisted */; /* no increment */)
{
    if (delete_condition)
    {
        m.erase(it++);
    }
    else
    {
        ++it;
    }
}
于 2012-01-11T22:19:08.790 に答える
0

反復中に反復子を安全に消去する標準的な方法は、次の結果を使用することですcontainer::erase。

void BitmapCache::CleanCache() {
    //Clean the cache of any NULL bitmaps that were deleted by caller.
    MapStrBmpIter _iter = _cache.begin();
    while (_iter != _cache.end()) {
        bool erase_entry= true;
        if(_iter->second != NULL) {
            if((_iter->second->w < 0 && _iter->second->h < 0) == false) 
                erase_entry= false;
        }

        if (erase_entry)
            _iter= _cache.erase(_iter);
        else
            ++_iter;
    }
}
于 2012-01-11T22:19:34.283 に答える