如何为指针分配内存

#include #include #include #include struct date { int year; int month; int day; }; struct person{ char name[64]; struct date birthday; }; struct aop { int max; struct person **data; }; struct aop *create_aop(int max) { struct aop *new = malloc(sizeof(struct aop)); new->data = malloc(max*sizeof(struct person)); for (int i=0; idata[i] = NULL; } new->max = max; return new;} void destroy_aop(struct aop *a) { free(a->data); free(a); } int add_person(struct aop *a, char *name, struct date birthday) { if (a->data[a->max-1] != NULL) { return -1; } struct person *new_person = malloc(sizeof(struct person)); strcpy(new_person->name, name); new_person->birthday = birthday; for (int i=0; imax; i++) { if (a->data[i] == NULL) { a->data[i] = new_person; break; } } free(new_person); return 0; } 

我对我写的代码有一些疑问。 首先，我需要在create_aop中添加额外的代码来初始化人的名字和生日吗？ 我发现在add_person中的free（new_person）之后，我无法访问a-> data [0] – > name。 如何在不使用其他指针的情况下更改a-> data [i]？

 struct aop *birthdays(const struct aop *a, int month) { int m = a->max; struct aop *n = create_aop(m); int j = 0; for (int i=0; idata[i] != NULL) { if (a->data[i]->birthday.month == month) { n->data[j] = a->data[i]; j++; } } else { break; } } if (j == 0) { return NULL; } return n; } 

每次运行上面的函数时，都会出现一些错误内存。 我一直在考虑它几个小时，但不知道这个有什么问题。

Answer 1

这段代码中有两个大错误。 首先，在struct aop中分配数据数组是有缺陷的。

 new->data = malloc(max*sizeof(struct person)); 

你不希望它指向大小最大倍数的人类结构的内存，对吗？ 因为它是指向指针的指针 ，所以它的大小只需要是指针的最大长度，即

 new->data = malloc(max*sizeof(struct person*)); 

您也可以让数据直接指向struct person。 然后第一行是正确的，每次创建新人时都不必分配内存。 相反，您只需使用数据指向的内存：

 (aop->data[i]).name = ... 

等等。

其次，您在创建后立即释放您的人员结构。

 free(new_person); 

现在aop-> data [i]是一个悬空指针，因为它指向的地址可能随时被覆盖（因为它不再被malloc锁定）。 相反，你必须在你的销毁function中释放它。 它可能看起来像这样：

 void destroy_aop(struct aop *a) { int i; for(i = 0; i < a->max; i++) { if(a->data[i] != NULL) { free(a->data[i]); } } free(a->data); free(a); }