ラムダからのアクセス違反がときどき発生します

Question

std::vectorからオブジェクトを消去しようとするとアクセス違反が発生するラムダがあります。イテレータは正しいオブジェクトの実行時間を指していない可能性があるため、削除するオブジェクトのコピーをイテレータの代わりにラムダに渡します。

ベクトルから特定のオブジェクトを削除するラムダを設定するより良い方法はありますか？そして、ここでアクセス違反の正確な原因は何ですか？このエラー時にデバッグすると、ローカル変数とオート変数がnull /壊れているのでデバッグが難しくなります。

... std::vector<CustomLayout> customLayouts; // private class member variable 

void App::loadCustomLayouts() { 
    CustomLayout customLayout; 
    // ... create HWND 

    onMessage(WM_RBUTTONDOWN, [customLayout, this]() { 

     auto it = std::remove(customLayouts.begin(), customLayouts.end(), customLayout); 
     customLayouts.erase(it); // occasionally causes access violation 
    }); 
} 

Answer 1

のstd :: customLayoutに等しいすべての値が削除された範囲の終了イテレータを返す取り除きます。実際には何も削除されません。要素の位置が変更されるだけで、新しい範囲の最後のイテレータの後に「削除された」要素が配置されます。それにもかかわらず、 "削除された"要素の状態は不特定であり、それらを参照することは問題を引き起こすかもしれない。

次に、STL end iteratorでは、最後の要素自体ではなく、範囲の最後の要素の次の要素を指します。そのため、イテレータの終了（あなたの場合はit）の操作は無効です。

customLayoutsには、customLayoutに等しい要素が含まれていない場合が考えられます。次に、itは決して存在しなかった要素を指し、eraseはそれを破壊しようとします。

eraseを新しい（より短い）範囲の最後の要素にする場合は、customLayouts.erase(it - 1);と書く必要があります。

Answer 2

あなたは適切なremoveイディオムに従っていません。それは簡単です：

vec.erase(std::remove_if(vec.begin(), vec.end(), elem), vec.end()); 

何かを引いたり、カスタムブランチを行う必要はありません。