Question

我有一个相对简单的程序，在从文件中读取整数后会一直崩溃。它在执行fclose线时崩溃。我已将错误本地化为此函数。

// Read array from text file
int * fileRead() {
    FILE *file;
    file = fopen("test.txt", "r");

    // Check if the file exists
    if(file == NULL){
        printf("There was a problem opening the file");
        exit(1);
    }

    // Count number of lines in file
    int count = 0;
    char c;
    for (c = getc(file); c != EOF; c = getc(file)){
        if (c == '\n') { 
            count = count + 1;
        }
    }

    // Reset to top of file
    int t = fseek(file, 0, SEEK_SET);

    // Read each line and save it to temp
    int *temp = malloc(sizeof(int)*count);
    int num, i;
    for (i = 0; i<=count; i++){
        fscanf(file, "%d\n", &temp[i]);
        printf("%d\n", temp[i]);
    }

    fclose(file);
    printf("Hello World\n");
    return temp;
}

Hello World是向我自己证明它在fclose完全崩溃。该函数从一个文件中读取整数，该文件仅包含单独行（未知数量）的整数，并将它们保存到数组中，然后返回该数组。提前感谢您的帮助，这是我第一次使用c，所以我不知道从哪里开始寻找bug。

Answer 1

备注：

c中的索引从0开始。因此，如果要保存count整数，则必须迭代到count-1。

i.e.

i < count

i <= count-1

你的阅读是错误的，因为你假设你的整数是一位数。

#include<stdio.h> #include<stdlib.h> int * fileRead() { FILE *file; file = fopen("PATH.txt", "r"); // Check if the file exists if(file == NULL){ printf("There was a problem opening the file"); exit(1); } // Count number of lines in file int count = 0; char pc = '\n'; char c; while (c = fgetc(file), c != EOF) { if (c == '\n' && pc != '\n') count++; pc = c; } // Reset to top of file fseek(file, 0, SEEK_SET); // Read each line and save it to temp int *temp = malloc(sizeof(int)*count); int num, i; for (i=0; i<count; i++) { fscanf (file, "%d", &temp[i]); } fclose(file); return temp; } int main() { int *t = fileRead(); printf("%d\n", t[0]); printf("%d\n", t[1]); }

文件：

452 55

<强>输出：

542 55

总结一下：

fclose（）崩溃我的程序。

没有。它不是fclose你试图访问未分配的内存。

Answer 2

您的代码中存在多个问题：

fgetc(fp)返回int值，该值可以包含unsigned char类型的所有值和特殊值EOF。您必须将其存储到int类型的变量中，以使文件结束测试可靠。

for (i = 0; i<=count; i++){

会导致缓冲区溢出，因为分配的块中允许的最大索引为count-1。这肯定会导致未定义的行为，并可能很好地解释观察到的行为。

请改用：
```
for (i = 0; i < count; i++) ...
```
您必须将已分配数组的长度传递给调用者。
计算行数并不能准确确定文件中的条目数。最好解析文件两次或只解析一次并动态重新分配数组。后一种方法对于不可寻找的输入流（例如终端和管道）是必要的。

以下是您的代码的修改版本：

#include <errno.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int *fileRead(int *countp) {

    // Check if the file exists
    FILE *file = fopen("PATH.txt", "r");
    if (file == NULL) {
        fprintf(stderr, "Cannot open input file PATH.txt: %s\n",
                strerror(errno));
        exit(1);
    }

    // Count number of lines in file
    int num, count = 0;
    while (fscanf(file, "%d", &num) == 1) {
        count++;
    }

    // Reset to top of file
    fseek(file, 0, SEEK_SET);

    // Read each line and save it to temp
    int *temp = calloc(sizeof(int), count);
    if (temp != NULL) {
        int i;    
        for (i = 0; i < count; i++) {
            if (fscanf(file, "%d", &temp[i]) != 1) {
                fprintf(stderr, "error reading element number %d\n", i);
                break;
            }
        }
        *countp = i;  // number of entries successfully converted
    }
    fclose(file);
    return temp;
}

int main(void) {
    int count;
    int *t = fileRead(&count);
    if (t != NULL) {
        for (int i = 0; i < count; i++) {
            printf("%d\n", t[i]);
        }
        free(t);
    }
    return 0;
}

fclose（）崩溃c程序

2 个答案: