C - malloc 用于指向结构的指针

Question

我的代码可以工作，但我还没有弄清楚 malloc 是如何将内存分配给指向 C 中结构的指针的。在我的代码中，我没有将结构的整个大小分配给我的指针，而只是分配大小它的成员，是好是坏？如果它不好，我应该如何优化我的代码。

这是我的代码：

struct Student
{
    char name[25];
    float math, physic;
    char rank;
};



void studentManagement(void)
{
struct  Student *s;
int n,i=1;
float ave;

printf("Number of student = ");
scanf("%d", &n);
getchar();

    while(i<=n)
    {
         s = malloc(25*sizeof(char));

         printf("Enter student name: ");
         fgets(s->name,25,stdin);             
         free(s);

         s = malloc(5*sizeof(float));

         printf("Physics grade = ");
         scanf("%f", &s->physic);
         printf("Math grade = ");
         scanf("%f", &s->math);

         getchar();

         ave = ((s->math )+ (s->physic)) / 2;

         if(ave<5) 
             printf("Rank = D\n");
         if(ave >= 5 && ave <= 7) 
             printf("Rank = C\n");
         if(ave >= 7 && ave <= 8) 
             printf("Rank = B\n");
         if(ave > 8) 
             printf("Rank = A\n");

         free(s);
         i++;
  }
}

Answer 1

在s = malloc(25*sizeof(char)); 中，您正在分配空间并将s 设置为指向该空间。 s 是指向 struct Student 的指针。所以s 应该为struct Student 指向足够的空间。所以正确的分配是：

s = malloc(sizeof(struct Student));

另一种方法是使用s 指向的事物的大小：

s = malloc(sizeof *s);

通常首选后者，因为如果以后更改代码以使s 指向不同类型的对象，它仍然是正确的。

刚得到名字后，你就有了free(s);。这是不正确的。您应该只在使用完毕后释放内存。但是你还想使用s指向的对象，所以现在释放s还为时过早。

同样，您不需要使用s = malloc(5*sizeof(float));。语句s = malloc(sizeof *s); 将（如果成功）为struct Student 的all 分配足够的内存。您分配整个结构，使用它，然后释放它。您不分配结构的各个部分。

一旦你让那段代码正常工作，你应该考虑一下为什么要为struct Student 分配空间，然后尽快释放它。如果您只想使用struct Student 进行一次循环迭代，您可以通过声明struct Student s; 而不是分配内存并指向它来实现。您正在处理的任务可能需要构建一个struct Student 的数组，在这种情况下，您必须考虑如何为整个数组分配内存以及数组中需要多少元素。