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函数，其中包含struct的数组的mallocs

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在评论中提到

最后一件事：我想在名称的情况下用gets替换scanf，因为我希望将名称和姓氏保存为一个整体（包括空格）。但当我更换scanf时，我收到了这个错误：请求会员＆＃39; name＆＃39;在不是结构或联合的东西中。我在这里搜索了类似的问题，但我没有找到任何解决方案。你能建议吗？

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

一般情况下，我建议不要使用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 }会给出更好的结果。因此，我将整个阅读过程改为：

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

你知道为什么我喜欢struct的数组，因为strncpy我无论尺寸如何，都可以获得尺寸。

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

strncpy的工作方式与'\0'类似，只是传递了多少字节可用于目标缓冲区。如果来源大于该数字，然后p->name[39] = 0;将不在目标中写入字节，从而阻止了缓冲区溢出。当然，如果'\0' - 终止字节不在复制的字节，它不会被写入目标缓冲区。 \((?<!k)\w+\) 只要确保字符串是re.sub(r'\((?<!k)\w+\)', '', '(k) i am, us dep economy (applause) (ph)', flags= re.IGNORECASE) - 终止，无论多长时间来源是。

为什么当我在学期变量中放入一个合适的值时，我的程序会崩溃？ C

1 个答案: