map/iteratorはインクリメンタルマップではありません。インクリメンタルではありません。

Question

Driver::~Driver() 
{ 
    AutoCritSec acsDriverList(m_csDriverList,true); 
    DRIVERLIST::iterator it = m_DriverList.begin(); 
    for(;it!=m_DriverList.end();it++) 
    { 
     if (it->second == this) 
     { 
      m_DriverList.erase(it); 
      it = m_DriverList.begin(); 
     } 
    } 
} 

Visual Studio 2003でプログラムをコンパイルすると、プログラムはうまく動作します。しかし、私は2010年に同じことを行うときにアプリケーションを閉じ、その後ながら、私は

Expression:map/set iterator not incrementable 

のようないくつかのエラーを取得し、私はこれを無視して押したとき、私は

Expression:"standard c++ library out of range" && 0 

は、いずれかが何であるか任意のアイデアを持ってい取得しますここに行く：私は誰にでも何か提案があれば、非常に恩恵を受けるだろう。感謝の気持ちと暖かい願い。

Answer 1

thisがリスト内の唯一の要素である場合は、リストの末尾がオーバーランします。

thisをリストから削除すると、it = m_DriverList.begin();がリセットされます。これは問題ありません。ループの式が評価され（forステートメントのi++）、itが範囲の最後を過ぎて進んでいます。

反復子をコンテナの最後まで進めると、プログラムは未定義の動作を示します。最近のバージョンのVisual C++では、プログラムのデバッグビルドで多くの一般的なイテレータエラーが検出され、アサーションを作成して解決するのに役立ちます。

あなたは、ループ式を削除し、else声明にそれを移動することで問題を解決できます。

while (it != m_DriverList.end()) 
{ 
    if (it->second == this) 
    { 
     m_DriverList.erase(it); 
     it = m_DriverList.begin(); 
    } 
    else 
    { 
     ++it; 
    } 
} 

けれども、あなたが要素かなり無駄であるを削除するたびに反復再起動します。

it = m_DriverList.erase(it); 

Answer 2

次のように連想コンテナの正しい消去イディオムは次のとおりです：消去/削除の

for (auto it = container.begin(); it != container.end() /* not hoisted */; /* no inc. */) 
{ 
    if (delete_condition) 
    { 
     container.erase(it++); 
    } 
    else 
    { 
     ++it; 
    } 
}