リストの要素を消去する際の問題

Question

struct ScheduleTaskInfo 
{ 
unsigned int nTaskID; 
    __time64_t timeStartTime; 
__time64_t timeEndTime; 
}; 

typedef list<ScheduleTaskInfo> SchedulerList; 

SchedulerList::iterator itrSchedulerList;  
for(itrSchedulerList = gSchedulerList.begin();itrSchedulerList != gSchedulerList.end();itrSchedulerList++) 
{ 
    SYSTEMTIME st; 
    GetLocalTime(&st); 

    CTime ctSyatemTime(st); 

    if (itrSchedulerList->timeEndTime == ctSyatemTime.GetTime()) 
    { 

     itrSchedulerList = gSchedulerList.erase(itrSchedulerList);   
    } 

} 

これを実行すると、forループがクラッシュしています。これはerase（）; 私はここで何か間違ったことをやっている？あなたのようにelse-blockでitrSchedulerList++を記述する必要があり、この

Answer 1

で私を提案してください：

for(itrSchedulerList = gSchedulerList.begin(); 
        itrSchedulerList !=gSchedulerList.end();) 
{ 
    SYSTEMTIME st; 
    GetLocalTime(&st); 

    CTime ctSyatemTime(st); 

    if (itrSchedulerList->timeEndTime == ctSyatemTime.GetTime()) 
    { 

     itrSchedulerList = gSchedulerList.erase(itrSchedulerList);   
    } 
    else 
     itrSchedulerList++; 
} 

これは、あなたがリストの終わりを超えて、それをインクリメントしていないことを保証します。 erase()は、あなたのバージョンのforループでインクリメントされるエンドイテレータを返すことがあります。

Answer 2

戻り値をeraseに再割り当てするときは、イテレータをインクリメントしないでください。これは、効果的に1つの要素をスキップするためです。余分なelseブロックにそれを置く：

if (itrSchedulerList->timeEndTime == ctSyatemTime.GetTime()) 
{ 

    itrSchedulerList = gSchedulerList.erase(itrSchedulerList);   
} 
else 
{ 
    ++itrSchedulerList; 
} 

そして、forループの外にそれを取ります。