C 프로그래밍의 연결된 목록

Question

이 코드의 문제점을 모르겠습니다. "ADD"에서 입력 한 항목을 "표시"할 때 많은 불필요한 항목이 표시됩니다. 왜 이런 일이 일어나는 걸까요?

temp= (struct list*)malloc(sizeof(struct list)); 

을하지만 당신은 당신의 구조의 값을 초기화하지 :

#include <stdio.h> 
#include <conio.h> 
#include <string.h> 
#include <ctype.h> 
#include <stdlib.h> 
struct list 
{ 
    char name[20]; 
    int age; 
    char gender[10]; 
    struct list *next; 
}; 

void main(void) 
{ 
    struct list *HEAD = NULL; 
    struct list *temp, *trav; 

    char choice; 


    while(1) 
    { 
     clrscr(); 
     printf("MENU\n"); 
     printf("A) ADD\n"); 
     printf("B) DISPLAY\n"); 
     printf("X) EXIT\n"); 

     scanf("%c", &choice); 
     switch(toupper(choice)) 
     { 
      case 'A': 
       temp= (struct list*)malloc(sizeof(struct list)); 
       printf("Fill-Up the following:\n"); 
       printf("Name:"); 
       fflush(stdin); 
       gets(temp->name); 
       printf("Age:"); 
       fflush(stdin); 
       scanf("%d",&temp->age); 
       printf("Gender:"); 
       fflush(stdin); 
       gets(temp->gender); 


       if(HEAD == NULL) 
       { 
        HEAD = temp; 

       } 
       else if(HEAD!=NULL) 
       { 
        for(trav=HEAD; trav->next != NULL; trav= trav->next); 
        trav->next=temp; 
       } 
       else 
       { 
        printf("Not Enough Memory!\n"); 
       } 


       break; 
      case 'B': 

       if(HEAD==NULL) 
       { 
        printf("Linked List is Empty!\n"); 
        getch(); 
        break; 
       } 
       if(HEAD!=NULL){ 
        for(trav=HEAD; trav != NULL; trav=trav->next) 
        { 

         printf("Name: %s\n", trav->name); 
         printf("Age: %d\n", trav->age); 
         printf("Gender: %s\n", trav->gender); 
         getch(); 
        } 
       } 

       break; 
      case 'X': 
       free(HEAD); 
       free(trav); 
       free(temp); 
       exit(1); 

       break; 
     } 
    } 
} 

Answer 1

는이 같은 온도를 초기화된다. 그래서 그들은 쓰레기를 포함합니다. 모든 malloc()은 힙에서 메모리 덩어리를 반환합니다. 당신은 care와 feeding에 대한 책임이 있으므로 모든 데이터 멤버를 NULL과 같은 알려진 값으로 설정하십시오. 이것은 디버거를 사용하고 mallloc() 할당 후 temp 값을 검사 할 때 매우 분명합니다 (영상).

따라서, 귀하의 malloc 구조의 데이터 멤버를 항상 초기화하는 것이 좋습니다.