要素を削除するときにSTLマップのイテレータを使用するのに問題がある

Question

要素を削除してクリーンアップするためにSTLマップをループしています。残念ながら、私のプログラムは+ + iter行でクラッシュします。私はマップ上でどのようにループしているかに間違ったことをしていますか？

std::map<string,BoneHierarchy* >::iterator iter = boneMap.begin(); 
while(iter != boneMap.end()) { 
    string playerName = iter->first; 
    boneMap[playerName]->clear(); 
    boneQueue->push(boneMap[playerName]); 
    boneMap.erase(iter); 
    ++iter; 
} 

Answer 1

は、（それが消去されていますので、どのようにあなたがそれをインクリメントすることを期待できますか？）それを削除した後、イテレータをインクリメントしないでください。反復子のコピーを消去：

boneMap.erase(iter++); 

またはC++ 11を使用している場合

iter = boneMap.erase(iter); 

を行います。

（もRemoving elements from a C++ map through a for-loop参照）

Answer 2

boneMap.erase(iter); 
++iter;      //incorrect 

のように記述する必要があります：map::erase() returns iterator following the erased itemので

iter = boneMap.erase(iter); //correct (in C++11) 
//no need to increment! 

。