C++ポインタ "エラー：ダブル無料または破損（アウト）"

これが私のコードです：C++ポインタ "エラー：ダブル無料または破損（アウト）"

uint16_t * ptemparr = new uint16_t[20]; 
for (int x=0;x<2;x++) 
{ 
    function(ptemparr); 
    ptemparr ++; 

} 
delete[] ptemparr;

私はこのエラーを取得することを行うと：

double free or corruption (out)

EDITED：ありがとうございましたIなぜこのエラーが発生するのか理解していますが、今はこれが良いアイデアだと思いますか？

uint16_t temparr[20]; 
uint16_t * ptemparr = temparr; 
for (int x=0;x<2;x++) 
{ 
    function(ptemparr); 
    ptemparr ++; 

}

このように私はスタック上にポインタを作成し、メモリリークの問題はありません。はまた、上記のコードは、そう私はあなたがnew []によって返されたdelete []に同じアドレスを渡す必要があり、このような状況

出典

2012-03-28 Kam

インデックス作成に角括弧を使用しないのはなぜですか？ – tartar

funtionはptemparrの中に配列を返します。上のコードは説明のためのものです。実際には++はptemparr = ptemparr + 4 – Kam

@Kamです。x（x = 20をコードスニペットに割り当てる）たとえ1秒に1回程度「むしろ」とは言えず、静的割り当てと動的割り当ての違いがその時間に消える傾向があるとしても、静的割り当てをうまく使用します。スタックやヒープを使用するのは味わいです。ブロックの外側でそのベクトルにアクセスする必要がある場合、ヒープは必須です。 – Enzo

のための最高のコーディングの練習に何を知らせる前に、このことを念頭に裸してください、すべての1秒を実行する必要があります。
また、function()が渡されたポインタ上で削除しないようにしてください。deallocate the memory by calling

出典

2012-03-28 04:53:20

私の質問が更新されました:) – Kam

@Kam：2番目のアプローチはより良いです.C++での動的割り当てを避けることは、常に良いアイデアです。実際には、C++コードの中で最も新しいものはnewです。すべて、決して生のポインタではありません。 –

forループで増分したので、ptemparrをそのホームアドレスにリセットする必要があります。だから、私はそれを削除する前に2減らすことをお勧めします。

ptemparr-=2;

出典

2012-03-28 04:53:23 tartar

あまりにもclunky。 –

@JonathonReinhartはかっこ悪いですか？==間違っていますか？ – Kiril

いいえ、それはうまくいくかもしれませんが、保守が難しく、将来のバグを尋ねるだけです。 –

あなたはnewによって返された同じアドレスをdeleteする必要があります。

私はいつも私のループを元のポインタのコピーでこの行にし、mallocまたはnewによって返された元のポインタを変更しません。

出典

2012-03-28 04:57:03

C++ポインタ "エラー：ダブル無料または破損（アウト）"

答えて

関連する問題