C：ポインタを持たないときに構造体にメモリを解放するには？

Question

私は構造を作り、新しい構造自体を返す関数を持っています。構造体には数字の配列と、そこにいくつの数字があるかを示す整数が含まれます（これは配列のように機能します）。 私は一時的な構造体を作成し、それを返すmakeStructを持っています。 mainではこの関数を呼び出してmyNumbers構造体に割り当てます。 これは、関数内に構造体のインスタンスが作成され、myNumbersにコピーされてから削除されるという効果があります。私の仮定は正しいのですか？

これは正しくコンパイルされているようですが、私の構造を指すポインターがないため、割り振られたメモリーを解放する方法がわかりません。どうすればメモリを解放できますか？ はもうそれへのポインタを持っていないとき、構造体へのメモリを解放する方法

#include <stdlib.h> 
#include <stdio.h> 

typedef struct{ 
    int* numbers; /*array of numbers*/ 
    int crowd; /*crowd of numbers*/ 
} magicNums; 

magicNums makeStruct(int crowd){ 
    magicNums tempStruct, *struct_ptr=&tempStruct; /*my temp struct and its pointer*/ 
    struct_ptr=(magicNums*)malloc(sizeof(magicNums)); /*allocating memory for struct*/ 
    tempStruct.numbers=(int*)malloc(sizeof(int)*crowd); /*allocating memory for struct's numbers*/ 
    return tempStruct; 
} 

int main() { 
    magicNums myNumbers; 
    myNumbers=makeStruct(10); 
    /*magicNums *myNumbers_ptr=&myNumbers; This won't work    */ 
    /*free(myNumbers_ptr);     This won't work    */ 
    /*free(&myNumbers);      This won't work    */ 
    return 0; 
} 

Answer 1

？

幸い！それがどこにあるのかわからない場合、どうすればそれを解放することができますか？解決策：どこに置くかを覚えておいてください！

Answer 2

正しいですが、有効なポインタがないのでメモリを解放することはできません。メモリリークが正常に作成されました。

この関数から構造体ポインタを返す必要があり、値による構造体は返しません。

なぜこれのためにmalloc（）を使用していますか？

Answer 3

私が見る限り、最初にmallocを入力する必要はありません。makeStructです。後でint main(void)で次に

（！）あなたの代わりにポインタを返す試し

free(myNumbers.numbers); 

Answer 4

を行うことができます。

magicNums* makeStruct(int crowd){ 
    magicNums *tempStruct = (magicNums*)malloc(sizeof(magicNums)); /*allocating memory for struct*/ 

    tempStruct->numbers=(int*)malloc(sizeof(int)*crowd); /*allocating memory for struct's numbers*/ 
    return tempStruct; 
} 

int main() { 
    magicNums *myNumbers; 
    myNumbers = makeStruct(10); 

    free(myNumbers->numbers); 
    free(myNumbers); 
    return 0; 
} 

Answer 5

構造体のコピーをメインプログラムに戻しながら、関数自体でクリーンアップを行うことがあります。その後、そのコピーへの参照があり、使用後にメモリを解放するために解放することができます。

編集：問題の多くがここにあります正確に何をしたいかもしれませんマルティンスの答えは、:)

Answer 6

を参照してください。 1つは、質問に答える - あなたはできません。しかし、それはここでは本当にあなたの問題ではない：

私はあなたの代わりにこれをしたいことを推測するつもりだ。そして、

magicNums* makeStruct(int crowd){ 
    magicNums *struct_ptr = 
     (magicNums*)malloc(sizeof(magicNums)); /*allocating memory for struct*/ 
    struct_ptr->crowd = crowd; 
    struct_ptr->numbers=(int*)malloc(sizeof(int)*crowd); /*allocating memory for struct's numbers*/ 
    return struct_ptr; 
} 

の主は、あなたが割り当てる必要はありません

int main() { 
    magicNums *myNumbers; 
    myNumbers = makeStruct(10); 

    free (myNumbers->numbers);   
    free (myNumbers); 

    return 0; 
} 

Answer 7

だろう構造体自体のメモリは、あなたのmakeStructは次のようになります：

magicNums makeStruct(int crowd) { 
    magicNums tempStruct; 
    tempStruct.numbers = (int *) malloc(sizeof(int) * crowd); 
    tempStruct.crowd = crowd; 
    return tempStruct; 
} 

tempStructが作成されていますが関数が返ってきたらスタックは自動的に解放され、関数はtempStructのコピーを作成して返します。

、あなたはこのようにそれを使用することができます：あなたの質問に答えるためには、「どのようにもうそれへのポインタを持っていないとき、構造体へのメモリを解放するために？」

int main() { 
    magicNums myNumbers; 
    myNumbers = makeStruct(10); 
    free(myNumbers.numbers); 
} 

Answer 8

。

できません。あなたのコードはメモリをリークします。

も意味がありません、あなたのコードの中しかし、多くのエラーがあります

magicNums tempStruct, *struct_ptr=&tempStruct; 
struct_ptr=(magicNums*)malloc(sizeof(magicNums)); 

上記二行最初のセットstruct_ptr =＆tempStruct。後でそのポインタをmallocからの戻り値に置き換えます。 2行はまったく同じ結果を達成します：

magicNums tempStruct; 
magicNums *struct_ptr; 
struct_ptr=(magicNums*)malloc(sizeof(magicNums)); 

これは、何が起こっているかをより明確にするかもしれません。

return tempStruct;

ここでtempStructを返すと、struct_ptrへのポインタが失われているため、解放することができず、メモリがリークします。

ただし、tempStruct.numbersへのポインタは失われていません。あなたのメインでそれを解放することができます

magicNums myNumbers; 
myNumbers=makeStruct(10); 
free(myNumbers.numbers); 

magicNumsは、ポインタではありません、あなたがすることはできませんし、それを解放する必要はありません。あなたは構造体を値で返しているので、大丈夫です。そこには動的なメモリはありません。

Answer 9

ここには、自分の考えていることの説明に一致するコードがあります。それがあなたがしたいことであるかどうか、私は知らない。

#include <stdlib.h> 
#include <stdio.h> 

typedef struct{ 
    int* numbers; /*array of numbers*/ 
    int crowd; /*crowd of numbers*/ 
} magicNums; 

magicNums makeStruct(int crowd){ 
    magicNums tempStruct; 
    tempStruct.crowd = crowd; 
    tempStruct.numbers = malloc(sizeof(*tempStruct.numbers)*crowd); 
    return tempStruct; 
} 

int main(void) { 
    magicNums myNumbers = makeStruct(10); 
    free(myNumbers.numbers); 
    return 0; 
}