std :: listの項目を消去するときのバグ

Question

ここで別のリストのコピーであるリストを作成しました。このリストはコピーと呼ばれます。私はコピーされた数字のシャッフルリストを作成しようとしています。ユーザーは最初のリストのサイズ（いくつのカードが必要か）を入力し、リストが作成され、ここではそのリストがシャッフルされます。しかし、何らかの理由でユーザーが20を入力すると、このエラーが発生します。

output(41432,0x7fff7389e000) malloc: *** error for object 0x7fff55638490: 
pointer being freed was not allocated 
*** set a breakpoint in malloc_error_break to debug 
Abort trap: 6 

ここは私のコードのセクションです。 while（！copied.empty（））ループを削除すると、最初のリストから新しいリストに位置 "r"に "it"の値が入るので、うまく動作します。しかし、私はそれぞれの "r"のためにそれを行う必要があります。

list<size_t> copied; 
    copied.insert(copied.end(), cards.begin(), cards.end()); 
    int r = rand()%(((copied.size()+1) - 0) + 1) + 0; 

    list<size_t>::const_iterator it = copied.begin(); 

    while(!copied.empty()){ 
     for(int i = 0; i < r; i++) { 
      it++;} 
     shuffle.push_back(*it); 
     copied.erase(it);} 

Answer 1

問題はcopied.erase(it)はイテレータitを無効にすることであるので、任意のその後の使用は未定義の動作です。 eraseが消去されたものの後の次の要素への有効なイテレータを返しますので、あなたは代わりに

it = copied.erase(it); 
if (it == copied.end()) it = copied.begin(); 

を使用することができます。それはおそらくする必要がありますので、あなたはまた、リストの末尾から実行しているforループにit++を心配する必要があります。

for(int i = 0; i < r; i++) { 
    if (++it == copied.end()) 
     it = copied.begin(); }