ベクターを反復処理してその特定の要素を削除する安全な方法は何ですか

Question

繰り返される処理を防ぐために、要素の一部を削除しながら、ベクトルの要素を1つのステップで反復することはできますか？ （私はフラグメントは偽ですけど、私が欲しいものを示す）

std::vector<MyRecord*>* BLV = GetVector(); 
for (std::vector<MyRecord*>::iterator it = BLV->begin(); it != BLV->end(); ++it) 
{ 
    MyRecord* BL = *it; 
    if(Selected) 
    { 
    delete BL; 
    BLV->erase(it); 
    } 
} 

Answer 1

songyuanyaoにループを変更するには、すでにあなたに素敵な答えを与えた：一言で言えば、あなたはeraseによって返された値に、あなたのイテレータをリセットする必要があります。

ただし、該当する場合は、unique_ptrとイレーズ・イディオムのベクトルを使用することをお勧めします（これは、要素を1回だけ移動するので効率的です）。

std::vector<std::unique_ptr<MyRecord>>* BLV = GetVector(); 
BLV->erase(
    std::remove_if(BLV->begin(), BLV->end(), 
     [&](const std::unique_ptr<MyRecord>& rec) { return Selected; } 
    ), 
    BLV->end() 
); 

Answer 2

注std::vector::eraseを消去する要素にイテレータを無効になり、その後、++it;はUBにつながります。

end（）イテレータを含め、イレースポイントと参照ポイントを消去ポイント以降で無効にします。

戻り値はerase()です。

最後に削除された要素に続くイテレータ。イテレータposが最後の要素を参照する場合、end（）イテレータが返されます。

その後、

for (std::vector<MyRecord*>::iterator it = BLV->begin(); it != BLV->end();) 
{ 
    MyRecord* BL = *it; 
    if(Selected) 
    { 
    delete BL; 
    it = BLV->erase(it); 
    } else 
    { 
    ++it; 
    } 
}