6

reallocポインタ変数に再度使用する前にメモリを解放する必要がありますか。次のうち正しいのはどれ?

for(i = 0; i < n; i++){
   myArray = (int *)realloc(myArray, i*sizeof(int));
}

for(i = 0; i < n; i++){
   myArray = (int *)realloc(myArray, i*sizeof(int));
   free(myArray);
   myArray = NULL;
}
4

4 に答える 4

14

の具体的な有用性reallocは、それを使用する前に必要がないfreeことです。すでに割り当てられているメモリを増やすために存在します。

したがって、これは必須ではなく、一般的ではありません。NULLポインターを渡すと、 のようにrealloc動作しmallocます。呼び出す前に使用している場合はfree、使用している可能性がありますmalloc

エラー処理を省略したため、どちらの例も正しくありません。すべてのアロケーターが戻る可能性があり、この点NULLでの使用reallocは少し注意が必要です。ドキュメントと例を注意深く読んでください。具体的には、 ifが失敗して を返す場合、参照が失われ、メモリがリークしているだけptr = realloc(ptr, ...なので、常に悪い考えです。代わりに tmp 変数を使用してください。reallocNULL

tmp = realloc(ptr, newSize);
if (tmp != NULL)
    ptr = tmp;
else handle_error();
于 2012-07-18T19:29:17.180 に答える
5

どちらでもない。NULL を返す realloc を常にチェックする必要があります。そうしないと、メモリリークが発生します。また、アロケーター関数の戻り値をキャストしていますが、これはお勧めできません。また、後で配列/ポインターの型を変更し、ここでも型を変更するのを忘れた場合、追跡が困難な segfault に直面することになるためsizeof(*myArray)、明示的な構造の代わりに使用することをお勧めします。 sizeof(type)/またはメモリ リーク エラー。総括する:

for(i = 0; i < n; i++){
    int *tmp;
    tmp = realloc(myArray, i*sizeof(*myArray));
    if (tmp == NULL)
    {
        free(myArray);
        /* And handle error */
    }
    myArray = tmp;
}

さて、あなたの質問に実際に答えるには、free() を使う必要はありません。

于 2012-07-18T19:31:22.333 に答える
4

これは、何をしたいかによって異なります。 realloc()具体的には、既存の割り当てを取得してそのサイズを変更し、できるだけ多くのデータを保持するように設計されています。あなたが提供した両方の例は、コンパイルされるという意味では正しいですが、異なることを行います。

realloc(NULL, n)は と同じなmalloc(n)ので、2 番目のケースは次と意味的に同等です。

for(i = 0; i < n; i++){
   myArray = (int *)malloc(i*sizeof(int));
   free(myArray);
   myArray = NULL;
}

最初の例では、 が指すデータmyArrayが保持されます。2 番目の例では、既存の割り当てが破棄され、初期化されていない新しい割り当てに置き換えられます。元の割り当てが指すデータはすべて破棄されます。


は、割り当てが失敗した場合realloc()に返されることに注意してください。ただし、ポインタ以外を渡した場合、その割り当ては解放されません。1 つのポインターを渡してその同じポインター変数に直接結果を代入すると、元の割り当てがまだ存在するため、再割り当てが失敗した場合にメモリ リークが発生する可能性があります。これを行う正しい方法は、一時ポインター変数を使用することです。NULLNULLrealloc()

于 2012-07-18T19:30:02.130 に答える
2

いいえ、なぜですか?要点はrealloc、既存のメモリ ブロックを再割り当てし、そのメモリ ブロックに既に格納されている値を保持することです。アイデアは、場合によっては、完全に新しいメモリ ブロックを割り当てて古い値をそこにコピーする代わりに、既存のメモリ ロケーションを再利用する可能性があるということです。

古いブロックの内容を保存したくない場合は、まったく必要ないかもしれませんrealloc

また、適切な引数値が与えられた場合、 の機能はおよびreallocの機能もカバーすることに注意してください。コードの 2 番目のサイクルで行っていることは、基本的に純粋モードで使用しています。mallocfreereallocmalloc

于 2012-07-18T19:28:57.810 に答える