解放出来的结构数组，它拥有字符指针

Question

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
typedef struct 
{ 
int id; 
char *name; 
char *lastname; 
} student_type; 
typedef struct 
{ 
student_type list[15]; 
} student_list_type; 
void get_students(FILE *input,char *filename,student_list_type *student_list); 
int main() 
{ 
printf("Hello world!\n"); 
student_list_type std_list; 
student_list_type *std_list_p=&std_list; 
FILE *input; 
get_students(input,"students.txt",std_list_p); 
return 0; 
} 
void get_students(FILE *input,char *filename,student_list_type *student_list) 
{ 

int i=0; 
int j=0; 
input=fopen(filename,"r"); 
printf("filename is %s",filename); 
while(fscanf(input,"%d",&student_list->list[i].id)==1) 
{ 
    student_list->list[i].name=(char *) malloc(15); 
    student_list->list[i].lastname=(char *) malloc(15); 
    fscanf(input,"%s",student_list->list[i].name); 
    fscanf(input,"%s",student_list->list[i].lastname); 
    i++; 
} 

for(j=0; j<i+1; j++) 
{ 
    free(student_list->list[i].name); 
    free(student_list->list[i].lastname); 
} 
free(student_list->list); 
fclose(input); 
} 

我觉得我在这个code.I的释放一部分未能了解到，我应该释放elemenets第一则整个数组但是这个代码没有按我可能错了解到way.Anyways给出错误信息，但是在从txt获取所有学生列表后，有时会崩溃。所有的

Answer 1

首先，你不检查，如果你的fopen成功，如果失败的fscanf仍然试图打开变量输入，但在这种情况下，这将是NULL。

你们为什么声明“文件*输入;”主要？如果你在下一刻将它传递给你的函数“get_student”。为什么不直接在函数中声明它？

此代码是更故障安全

void get_students(char *filename,student_list_type *student_list) 
{ 
    int i=0; 
    int j=0; 

    FILE * input; 

    input=fopen(filename,"r"); 
    printf("filename is %s",filename); 

    if((input = fopen("students.txt","r")) == NULL) 
    { 
     fprintf(stderr, "\nFile could not be open\n"); 
    } 
    else 
    { 
     while(fscanf(input,"%d",&student_list->list[i].id)==1) 
     { 
      student_list->list[i].name=(char *) malloc(15); 
      student_list->list[i].lastname=(char *) malloc(15); 
      fscanf(input,"%s",student_list->list[i].name); 
      fscanf(input,"%s",student_list->list[i].lastname); 
      i++; 
      } 

      for(j=0; j<i+1; j++) 
      { 
       free(student_list->list[i].name); 
       free(student_list->list[i].lastname); 
      } 

      fclose(input); 
     } 
} 

，并在结束时，你并不需要释放此

free(student_list->list); 

因为student_list不是在堆上分配。