c - 为什么当我在学期变量中输入正确的值时，我的程序崩溃了？ C

Question

struct student{
    char *number;
    char *name;
    int semester;
    float *grades;
};

typedef struct student RECORD;

void read_record(RECORD *p);
void print_record(RECORD x);
void init_record(RECORD *p);
void free_record(RECORD x);

main()
{
    int N;
    printf("Please enter the number of students that you want to insert: ");
    scanf("%d",&N);

    RECORD array[N];
    int i;

    for (i=0; i<N; i++)
        init_record(&array[i]);

此时，当我从键盘输入学期值时，我的程序崩溃了

for (i=0; i<N; i++)
{
    printf("Number %d student: \n",i+1);
    read_record(&array[i]);     
}


for (i=0; i<N; i++)
print_record(array[i]);


    for (i=0; i<N; i++)
    {
        free_record(array[i]);
    }   
}

我尝试删除任何不必要的代码，但我留下了最重要的部分，这样任何错误的部分都会更容易被看到

void read_record(RECORD *p)
{
        int i;

    printf("Please give the name: ");
    scanf("%s", p->name);

    printf("Please give AM number: ");
    scanf("%s", p->number);

    printf("Please give semester: ");
    scanf("%d",p->semester);

    printf("Please give grades: ");
    for(i=0;i<5;i++)
    scanf("%f", &(p->grades[i]));
}

在这段代码之后，我有 init_record 函数，其中包含结构体数组的 malloc

void init_record(RECORD *p)
{

    p->name = malloc(sizeof(char)*40);
    if (!p->name)
    {
        printf("Error!");
        exit(0);
    }

    p->number = malloc(sizeof(char)*6);
    if (!p->number)
    {
        printf("Error!");
        exit(0);
    }

    p->grades = malloc(sizeof(float)*5);
    if (!p->grades)
    {
        printf("Error!");
        exit(0);
    }
}

Answer 1

假设您已经为结构成员分配了内存

printf("Please give semester: ");
scanf("%d",p->semester);

是问题所在，因为 p->semester 是一个 int。 %d 需要一个指向 int 的指针，您正在传递一个未初始化的整数值作为指向 scanf 的指针。

正确的调用是

scanf("%d", &p->semester);

通常使用一次malloc，因为需要存在于作用域之外的对象函数，例如链表、树等。而且当你需要时编译器时未知维度的数组。

你的init_record在我看来毫无意义，因为你知道的大小数组，它们甚至很小，您可以轻松地将结构更改为

struct student {
    char number[6];
    char name[40];
    int semester;
    float grades[5];
};

然后您就不需要执行 malloc 调用。

此外，您还需要像这样执行scanf:

int c;

scanf("%39s", p->name);
while((c = getchar()) != '\n' && c != EOF); // clearing the buffer

scanf("%5s", p->number);
while((c = getchar()) != '\n' && c != EOF); // clearing the buffer

...

如果用户输入太大的名称/号码，防止缓冲区溢出用于缓冲区。

<小时/>

编辑

OP asked in the comments

One last thing: I wanted to replace scanf with gets in the case of name, because I want a name and a surname to be saved as one piece(including space). But when I replaced scanf, I got this error: request for member 'name' in something not a structure or union. I searched over here for a similar problem but I didn't find any solution. Can you suggest one?

不要使用gets，这是一个危险的函数，因为它不接受考虑数组的大小以及输入的文本是否大于缓冲区大小，它会溢出并造成很大的损坏。该错误消息是由于语法错误，因为您没有显示代码，所以我无法说出您做错了什么。但是，如果您采纳我的建议，就不会出现此错误。

一般来说，我建议不要使用 scanf 来读取用户的信息，因为 scanf并非旨在这样做。特别是当你想读取具有以下内容的字符串时空格，最好使用 fgets 读取整行并稍后使用其他函数解析该行，例如 strchr、strstr、strtok、sscanf、strtol 等。使用哪个函数取决于您想要做什么从用户处读取。在这种情况下，您正在读取字符串，fgets 会给出更好的结果。所以我会把你的整个阅读过程改成这样:

int read_record(RECORD *p)
{
    char line[1024];

    printf("Please give the name: ");
    if(fgets(line, sizeof line, stdin) == NULL)
    {
        fprintf(stderr, "Could not read name\n");
        return 0; // error
    }

    line[strcspn(line, "\n")] = 0; // removing newline
    strncpy(p->name, line, sizeof p->name);
    p->name[sizeof(p->name) - 1] = 0; // making sure to get a valid string

    printf("Please give AM number: ");
    if(fgets(line, sizeof line, stdin) == NULL)
    {
        fprintf(stderr, "Could not read AM number\n");
        return 0;
    }

    line[strcspn(line, "\n")] = 0; // removing newline
    strncpy(p->number, line, sizeof p->name);
    p->name[sizeof(p->number) - 1] = 0; // making sure to get a valid string

    // this is ok, this can stay like this
    printf("Please give semester: ");
    scanf("%d", &p->semester);

    printf("Please give grades: ");
    if(fgets(line, sizeof line, stdin) == NULL)
    {
        fprintf(stderr, "Could not read grades\n");
        return 0;
    }

    if(sscanf(line, "%f %f %f %f %f",
                p->grades, p->grades + 1, p->grades + 2, p->grades + 3,
                p->grades + 4) != 5)
    {
        fprintf(stderr, "Could not read 5 grades\n");
        return 0;
    }

    return 1; // success
}

我知道你之前的代码要多得多，但是这个代码更多健壮，它可以处理用户输入错误格式的情况，您的代码可以对此使用react，打印错误消息，重试用户输入，等等。和当打印在屏幕或文件上的数据时，错误会自行显现奇怪。

注意我如何复制字符串:

strncpy(p->name, line, sizeof p->name);
p->name[sizeof(p->name) - 1] = 0; // making sure to get a valid string

这里我假设您已经更改了结构以保存数组，就像我在答案的第一部分。如果您没有更改并且仍在使用通过使用硬编码固定大小进行malloc的旧方法，您将不得不使用这里也硬编码固定大小:

strncpy(p->name, line, 40);
p->name[39] = 0; // making sure to get a valid string

你明白为什么我更喜欢结构体有数组了，因为有了 sizeof 我无论尺寸如何，都可以获得尺寸。

这里要注意的另一件事是我使用了 strncpy 而不是 strcpy。strcpy 遇到与 gets 相同的问题，它不占用大小考虑目标缓冲区以及源字符串是否更大比目标缓冲区大，就会溢出缓冲区。

strncpy 的工作方式与 strcpy 类似，只不过您传递的是有多少字节可用于目标缓冲区。如果源大于该数字，那么strncpy将不会在目标中写入字节，从而防止缓冲区溢出。当然，如果 '\0' 终止字节不在其中复制的字节，它不会被写入目标缓冲区。 p->名称[39] = 0;只是确保字符串以 '\0' 结尾，无论字符串有多长来源是。