ダブルフリーまたは破損オブジェクト（別の...）を削除中

Question

以下のコードは、アレイ

for (i = j; i > 0; i--) { 
    myBlk *tmp = blks[i]; 
    blks[i] = blks[i-1]; 
    blks[i-1] = tmp; 
    delete tmp; 
} 

の先頭に要素を移動する、私が取得：

*** glibc detected *** double free or corruption (out): 0x00007fffd556ad10 *** 

この文を削除しても問題はありません。しかし、私はメモリがリークするのを望んでいません...

Answer 1

回答のコメントを促進する。

メモリ割り当てをポインタコピーと混同しているようです。あなたのループでは、あなたはメモリ割り当てをしていません。ポインタをコピーするだけで、メモリを割り当てません。

だから、deleteを取り除く必要があります：メモリの割り当てがある場合

for (i = j; i > 0; i--) { 
    myBlk *tmp = blks[i]; 
    blks[i] = blks[i-1]; 
    blks[i-1] = tmp; 
} 

deleteだけ呼ばれて - あなたがのどれを持っていません。 （少なくともループ内にはありません）

Answer 2

このコードは非常に奇妙に見えます。

私の推測は（あなたのコメントに基づいて "ポインタはループ内に作成され、ループの最後には削除されます。"）私はdeleteが不必要であると思う。

delete tmpに電話するとtmpとblks[i-1]の両方が同じメモリを指しているので、blks[i-1]が解放されます。ループの最後に有効なポインタが含まれ続けることが期待される場合は、deleteは確かに余計です。

Answer 3

tmpは、配列の既存の要素を指しています。あなたはnewでtmpを割り当てていません。したがって、tmpを削除する必要はありません。私は、元の配列要素が割り当てられ、どこか他の場所で解放されていると仮定しています。

Answer 4

毎回同じ要素を削除していますか？

削除した要素をi-1に移動してからiを減らします。