for 루프에서 구조체를 사용하는 데 문제가 있습니다.

Question

이것은 숙제 문제입니다. 내 컴파일러는 CodeBlocks입니다. 여기

내 코드입니다 :

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 

struct Address{ 
    char number[5]; 
    char street[30]; 
    char city[30]; 
}; 

struct Employee{ 
    char ID[7]; 
    char name[31]; 
    struct Address *addr; 
}; 

int main(){ 
    int n,i; 
    char temp[7]; 
    printf("Enter number of Employee : "); 
    scanf("%d",&n); 
    struct Employee **p=(struct Employee **)malloc(n*sizeof(struct Employee *)); 

    for (i=0; i<n; i++) 
    { 
     p[i]=(struct Employee *)malloc(sizeof(struct Employee)); 
     p[i]->addr=(struct Address *)malloc(sizeof(struct Address)); 
    } 

    for(i=0; i<n; i++) 
    { 
     printf("Employee #%d\n",i+1); 
     printf("Enter ID : "); 
     gets(p[i]->ID); 
     printf("Enter Name : "); 
     gets(p[i]->name); 
     printf("Enter Home number : "); 
     gets(p[i]->addr->number); 
     printf("Enter Street : "); 
     gets(p[i]->addr->street); 
     printf("Enter City : "); 
     gets(p[i]->addr->city); 
    } 
} 

내 문제는 내가이 코드를 실행하면, 나는 # 1 직원의 ID를 입력 할 수 있다는 것입니다; 그러나 직원 # 2와 # 3의 ID를 입력 할 수 있습니다.

내 문제는 어디에 있습니까?

Answer 1

루프의 첫 번째 패스 전에 콘솔에서 무언가를 읽는 중 gets()에 문제가있는 것 같습니다.

루프를 수정하기 바로 전에 gets(temp);을 추가하면 문제를 해결하는 것처럼 보입니다. 더 좋은 해결책은 gets() 이외의 것을 사용하는 것입니다.

Answer 2

초기 scanf("%d", &n);은 후행 줄 바꿈을 사용하지 않으므로 gets() 호출에 사용할 수 있습니다.

부수적으로, 결코gets()을 사용하십시오. 안전하게 사용할 수 없습니다. 예를 들어, 6 바이트 배열로 읽고 사용자가 10자를 입력하면 버퍼 오버 플로우가 발생합니다. 대신 fgets()을 사용해보십시오 (단, gets()과 달리 버퍼에 '\n' 문자가 남음).

Answer 3

사용자가 입력 한 후 입력 버퍼를 명시 적으로 지워야합니다. 크기 제한기로 입력 내용을 안전하게 만들어야합니다. 그리고 scanf의 반환 값을 사용해야합니다.

scanf("%d",&n);while(getchar()!='\n'); 
... 
scanf("%6[^\n]",p[i]->ID);while(getchar()!='\n'); 
... 
scanf("%30[^\n]",p[i]->name);while(getchar()!='\n'); 
... 
scanf("%4[^\n]",p[i]->addr->number);while(getchar()!='\n'); 
... 
scanf("%29[^\n]",p[i]->addr->street);while(getchar()!='\n'); 
... 
scanf("%29[^\n]",p[i]->addr->city);while(getchar()!='\n');