解放出來的結構數組，它擁有字符指針

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不是在堆上分配。