2

我有一个任务并且遇到了一些障碍。以下代码应该从文件中获取输入，将其读入我定义的结构中，并且对输入行数没有任何限制。但是，它在第​​ 24行出现段错误：

1 #include <stdio.h>
2 #include <stdlib.h>
3 #include <string.h>
4
5 typedef struct __student {
6         int ID;
7         char fname[33];
8         char lname[33];
9         float grade;
10 } student;
11
12 void partA(FILE *fp) {
13
14         int i, r;
15         i = r = 0;
16         int N = 1000;
17         student **S;
18
19         S = (student **)malloc(sizeof(student *) * N);
20
21
22         while(!feof(fp)){
23                 fscanf(fp, "%d %[^,], %s %f", &S[i]->ID, S[i]->lname, S[i]->fname, &S[i]->grade ); // segfault occurs on this line
24                 printf("%d %s %s %f\n", S[i]->ID, S[i]->fname, S[i]->lname, S[i]->grade);
25                 i++;
26                 printf("Something's happening, at least");
27                 if(i == N){
28                         N *= 2;
29                         S = (student **)realloc(S, sizeof(student) * N);
30                         if(S == NULL) {
31                                 printf("Memory reallocation failed; Fatal error.");
32                                 break;
33                         }
34                 }
35         }
36 }

我之前测试过代码，虽然当时我使用的是静态数组并想更改为动态大小。然而，即使 gdb with 除了行号之外提供的帮助也很少。我是否需要单独 malloc 每个学生结构，或者我在这里错过了一些完全不同的东西？

编辑：当我在while循环中为每个学生分配内存时，代码似乎有效：

S[i] = (student *)malloc(sizeof(student));

所以这似乎解决了这个问题。我会进行一些测试来确定。

4

2 回答 2

1

除了你的 malloc 问题之外，这两行还有很大的问题

while(!feof(fp)){
    fscanf(fp, "%d %[^,], %s %f", &S[i]->ID, S[i]->lname, S[i]->fname, &S[i]->grade ); // segfault occurs on this line

一个问题是您的lname和fname字段是固定大小的 33 字符数组，这意味着如果您的输入此时有超过 32 个字符，您将跑出数组的末尾并损坏内容。另一个问题是您没有检查返回值fscanf来查看是否出现问题（例如 EOF），导致您在到达文件末尾时尝试打印垃圾记录。你想要的是这样的：

while (fscanf(fp, "%d %32[^,],%32s%f", &S[i]->ID, S[i]->lname, S[i]->fname, &S[i]->grade) == 4) {

为您的循环控制。

编辑

问题while(!feof(fp))（以及为什么它几乎总是一个错误）是feof(fp)只有在您尝试读取文件末尾之后才返回 true 。在你读完最后一行之后，feof(fp)仍然返回 false，所以你再次进入循环并尝试读取另一行。该读取失败，但由于您不检查 的返回值fscanf，因此您没有意识到它，而是在您的数组中获得了一个垃圾额外值。

于 2013-02-23T00:14:49.593 回答
0

在第 19 行，您为 N 个学生指针分配了足够的空间，但没有为学生自己的结构分配空间。您需要执行以下操作：

for( int i = 0; i < N; i++) {
     S[i] = malloc(sizeof(struct __student));
}
于 2013-02-22T20:29:30.787 回答