c - 为什么在读取文本文件并存储到数组中时出现段错误？

Question

这个问题已经有答案了:

Getting a stack overflow exception when declaring a large array (8 个回答)

已关闭 5 年前。

我需要读入一行文本，并将其存储到一个数组中。当我编译程序时，它可以工作，但是当我执行它时，我收到段错误。我读过其他问题并尝试了他们的建议，但似乎没有任何效果。

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

void main() {

FILE *file;
char text[10000000], *line[10000000];
int i=0;

file = fopen("/home/Documents/xxxxxx_HW01/text.txt", "r");
if(file == NULL) {
    printf("cannot open file");
    exit(1);
}

while (i< 10000 && fgets(text, sizeof(text), file)!= NULL){
    line[i] = strdup(text);
    i++;
}

for (i=0; text[i] != '\0'; i++)
    printf("%c ", text[i]);

fclose(file);

}

Answer 1

继续我的评论，

text[i] = strdup (text);

错了。它尝试将指针(strdup 的结果)分配给 text[i] 有符号的 char 值。您需要使用单独的指针数组(或声明一个指向 char 的指针，例如 char **lines; ，然后分配指针，然后为每一行分配指针)。

您可以做的最重要的事情是聆听编译器告诉您的内容。为了确保您从编译器的帮助中受益，始终在启用警告的情况下进行编译，并且在编译时不发出警告之前不要接受代码。对于 gcc 这意味着至少添加-Wall -Wextra 到您的编译字符串。对于clang，-Weverything就可以了。对于 cl.exe (VS) 添加 /Wall。然后阅读并理解警告。编译器将为您提供出现问题的确切行。

如果您只是读取少于某个数字的行，则可以避免分配指针并仅使用指针数组，但必须跟踪索引(以避免写入超出最后一个元素)

根据您正在尝试的操作，您似乎正在尝试执行类似于以下操作的操作:

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

#define MAX 1000

int  main (void) {

    FILE *file;
    char text[MAX] = "", 
        *lines[MAX] = {NULL};
    int i=0, j;

    file = fopen ("/home/Documents/xxxxxx_HW01/text.txt", "r");

    if(file == NULL) {
        printf("cannot open file");
        exit(1);
    }

    while (i < MAX && fgets (text, sizeof(text), file)!= NULL){
        size_t len = strlen (text);        /* get length */
        if (len && text[len-1] == '\n')    /* check if last is '\n' */
            text[--len] = 0;               /* overwrite with '\0' */
        else {  /* line too long - character remain unread */
            fprintf (stderr, "error: line exceeds %d chars.\n", MAX - 2);
            exit (EXIT_FAILURE);
        }
        lines[i] = strdup(text);
        i++;
    }

    for (j = 0; j < i; j++) {
        printf ("line[%3d]  %s\n", j, lines[j]);
        free (lines[j]);    /* don't forget to free memory */
    }

    fclose(file);

    return 0;   /* main() is type int and therefore returns a value */
}

注意:您还应该通过 fgets 删除 text 末尾包含的尾随 '\n' > -- 上面给出的示例。

仔细检查一下，如果您还有其他问题，请告诉我。