とSegfaultネストされたループ

Question

にこれは私が実行しているコードである一方、ベクターから消去：

for(auto candidate = candidates.begin(); candidate != candidates.end();) { 
    for(auto inst = candidate->sortedLoads.begin(); inst != candidate->sortedLoads.end(); inst++) { 
     if(...) { 
      candidate = candidates.erase(candidate); 
      break; 
     } 

     else { 
      candidate++; 
     } 
    }   
} 

上記とセグメンテーション違反に実行しています。内側forループを削除すると、segfaultが消えます。あなたは何が間違っているか知っていますか？

Answer 1

EDIT
あなたは何も彼はそのは、UpperBound candidate != candidates.end()を超えないようにしない内側のループ、外側のループからあなたcandidate変数をインクリメントしています。あなたの内側のループにも少なくともその条件を加えてください。あなたは、になってしまいます：

for(auto candidate = candidates.begin(); candidate != candidates.end();) { 
    for(auto inst = candidate->sortedLoads.begin(); candidate != candidates.end() && inst != candidate->sortedLoads.end() ; inst++) { 
     if(...) { 
      candidate = candidates.erase(candidate); 
      break; 
     } 

     else { 
      candidate++; 
     } 
    } 
} 

しかし、一般的に、あなたは/デクリメントをインクリメントするかどう（または任意の値を変更する操作）別のループ内のループ変数は、あなたがどこ変数（最初のループ状態を繰り返す必要があります2番目のループ（どこから変更されていますか）にあります。

Answer 2

問題1：

inst = candidate->sortedLoads.begin(); 

instはcandidate内部の何かにイテレータです。 candidateが消去されている場合は、instとは何ですか？誰も知らない。

問題2：単にcandidateとinstがインクリメントされているとします。

inst != candidate->sortedLoads.end() 

inst第candidateのsortedLoads内のエントリを参照し、別のcandidateにおける他sortedLoadsのendと比較されます。悪いことが起こる前に、出口条件はほぼ確実に満たされません。

問題3：J.Baobyは既にカバーしています。candidateの末尾をオーバーシュートするテストはありません。

あなたのための良い解決策はありません。 instとcandidateの間のロジックを解くことをお勧めします。