Question

I just started to play C and I bump into this problem. Here’s my code:

#include<stdio.h>
#include<conio.h>

struct person {
    int i;
    char name[100];
};

int main() {
    struct person p[2];

    clrscr();
    for(int i=0;i<2;i++) {
        printf("Enter i:\n");
        scanf("%d",&p[i].i);

        printf("Enter name:\n");
        gets(p[i].name);
    }
    for(int j=0;j<2;j++) {
        printf("ID: %d, Name: %c\n", p[j].i,p[j].name);
    }
    getch();
    return 0;
}

Here’s a sample ouput:

The problem is, all char members are not being asked for a value.

UPDATE:

btw, I am using Turbo C++ version 3 compiler.

Answer 1

Two things:

1. You need to clear the input buffer to keep it from eating the newline.
2. Secondly, you need to change the format string to %s.

Here’s the corrected code:

int main() {
    struct person p[2];

    for(int i=0;i<2;i++) {
        printf("Enter i:\n");

        scanf("%d",&p[i].i);

        //  Flush input buffer
        int ch;
        while ((ch = getchar()) != '\n' && ch != EOF);


        printf("Enter name:\n");
        gets(p[i].name);
    }
    for(int j=0;j<2;j++) {
        printf("ID: %d, Name: %s\n", p[j].i,p[j].name);
    }
    getch();
    return 0;
}

%c expects a char, but you’re trying to pass in a string. It’s undefined behavior to have mismatching types.

Output:

Enter i:
1
Enter name:
asdf
Enter i:
2
Enter name:
zxcv
ID: 1, Name: asdf
ID: 2, Name: zxcv