为什么我的循环没有到达文件末尾？

Question

我正在 Windows 8.1 上的 Visual Studio 2012 中构建一个应用程序，该应用程序使用线性回归来计算/预测特定年份的世界人口。正在从文本文件中读取数据，程序使用该数据计算线性回归。

文本文件内容：

0.001 0.200
1.000 0.310
1.500 0.450
1.650 0.500
1.750 0.791
1.800 0.978
1.850 1.262
1.900 1.650
1.927 2.000
1.950 2.519
1.955 2.756
1.960 2.982
1.965 3.335
1.970 3.692
1.975 4.068
1.980 4.435
1.985 4.831
1.990 5.263
1.995 5.674
2.000 6.070
2.005 6.454
2.008 6.707
2.009 6.800

我遇到的问题是，当我遍历文件以将数据存储在结构数组中时，在完全读取文件之前出现异常。

这是我遇到问题的功能：

int ReadFile(char* filename) {
    char line[20];
    int i = 0;
    int recordCount = 0;
    FILE* file = NULL;
    Population* wp[] = {0};

    file = fopen(filename, "r"); /* open text file for reading */
    if (file != NULL) {
        printf("FILE OPENED FOR READING\n");
        while (fgets(line, 20, file) != NULL) {
            fscanf(file, "%s", line);
            recordCount++;
        }
        fclose(file);
        printf("There are %d records.\n\n", recordCount);
        //*wp = (Population*)malloc(sizeof(Population) * recordCount);
        file = fopen(filename, "r");
        for (i = 0; i < recordCount; i++) {
            wp[i] = (Population*)malloc(sizeof(Population));
            fscanf(file, "%5f", &wp[i]->year);
            printf("%f ", wp[i]->year);
            fscanf(file, "%5f", &wp[i]->population);
            printf("%f\n", wp[i]->population);
        }
    }
    fclose(file);
    return 1;
}

最后一个 for 循环（我为结构分配空间的那个）是使我的程序崩溃的循环。在第 4 次崩溃之前，我可以获得大约 3 次迭代。

抛出异常：

First-chance exception at 0x7741D7E0 (ntdll.dll) in worldpopulation.exe: 0xC0000005: Access violation reading location 0x871EADD7.
Unhandled exception at 0x7741D7E0 (ntdll.dll) in worldpopulation.exe: 0xC0000005: Access violation reading location 0x871EADD7.

该异常似乎是由文件指针损坏引起的，但我不知道为什么。谁能帮我弄清楚为什么最后一个循环会导致崩溃？谢谢！

Answer 1

while (fgets(line, 20, file) != NULL) {
    fscanf(file, "%s", line);
    recordCount++;

您正在阅读两次并数一次。首先使用fgets，然后使用fscanf。我猜你想摆脱fscanf。

另外一个问题是：

Population* wp[] = {0};
[...]
wp[i] = (Population*)malloc(sizeof(Population));

访问wp[0] 以外的任何内容都是非法的。同样，我猜你想分配类似的东西：

Population** wp = malloc(recordCount * sizeof *wp);

---

您两次读取文件的方式效率不高。您应该第一次使用fscanfs，并根据需要使用realloc 扩展wp。