C-单词列表问题的 strlen() 中的段错误

Question

我尝试创建一个可扩展的最多 20 个字母的单词列表。在添加到列表之前，我使用“char word[20]”作为每个新单词的主机。对于长度为 20- 的单词输入，程序运行良好。如果我添加长度超过 20 的单词，我的程序应该忽略并继续。

但是在下一次，我在 if-else 语句的 strlen(word) 的某个地方遇到了段错误。

奇怪的是，有时我的程序可以克服这个错误，但大多数时候，它会崩溃。

一些尝试: - 如果我使用 'char word[1000]' ，它在任何时候都工作正常(到目前为止)。我猜想 strlen() 的参数指针在“继续”之后错误地指向错误的地方，所以我用 &word[0] 跟踪它并编写 strlen(&word[0]) 并使用 'char word[20]'。这并不能解决问题，并且当发生段错误时 &word[0] 保持不变。

char *wordList[] = {""};
char word[10];
char * p, c;
int i = -1;
int size = 10;
int strlength = -1;

while(strlength != 0){
    //get word by using loop of getchar() and detect \0
    printf("Enter word :");
    scanf("%s",word);
    printf("Taken word: %s at %p\n",word,&word[0]);

    //detect word length to avoid 4-letter words and words with length 20+
    if (strlen(word) == 0){
        printf("Break due to length 0\n");
        break;
    }else if(strlen(word) > 20){
        printf("Ignore due to length 20+\n");
        continue;
    }
    printf("End checking length");

    //add word to wordList by extending memory for one new word and assign each char to memory
    i++;
    wordList[i] = malloc(sizeof(char)*strlen(word)-1);
    strcpy(wordList[i],word);
    printf("[");
    for(int j = 0; j <= i; j++){
        printf(" %s,",wordList[j]);
    }
    printf("]\n");

Answer 1

请记住，在 C 语言中，字符串只是内存中连续的字符集，以已知地址(如果您愿意，可以从 char* 或 char[])开始，以空字节 \0 结束。 你程序员有责任确保你不会尝试访问超出分配给你的字符串的内存。当你这样做时，你会遇到段错误。

scanf("%s",&word[0]);

您告诉 scanf 从标准输入读取一个字符串并将其存储在 word 中( &word[0] 只是 word )。但是你没有告诉 scanf 最多读取多少个字符，所以你不会阻止 scanf 访问 world[20] ，这是 10 个字符的字符串中太远的一个字符(C 当然是基于 0 的偏移量，所以第一个有效字符world 是 world[0]，最后一个是 world[19] 。)。输入超过 19 个字符(多留一个字符作为终止符 \0 )，您将超出内存分配，并且可能发生任何事情(如果您的程序在该内存地址处有有效数据，您将覆盖它。如果那里的数据碰巧是一个指针，下次尝试取消引用时，您将覆盖地址并出现段错误。如果程序的内存到此结束，您将直接出现段错误。

所以你要做的就是告诉 scanf 最多读取多少字节。最简单的方法是使用 scanf 的可选“字段宽度说明符”:

scanf("%9s",word)

这告诉scanf读取一个字符串并将其存储在word中，但最多只能存储9个字符(第10个是终止\0字符)。

If I add a word with length 20+, my program should ignore, and continue

您的逻辑存在缺陷 - 您不知道该单词是否有 10 个字符长，直到您从输入中读取全部，然后然后从 word 读取字符到第一次出现 '\0'。在您可以检查 strlen 和可选的 continue 之前，您已经通过将第 11 个字符写入 word[10] 内的 scanf 来犯了错误。请记住，它无法仅从 word 知道在该地址分配了多少字节。

单词列表代码也必须修复。

char *wordList[] = {""};`
...
//add word to wordList by extending memory for one new word and assign each char to memory

好吧， wordList[] 正好是 1 char* 长，并且该字符串是 "" ，您定义为未指定长度 [] 的静态定义列表的唯一内容的空字符串，其大小在编译时确定为 1。相反，您应该像定义 word 一样定义静态列表。

char* wordList[20];

您还需要知道当前列表大小，与下一个列表元素的偏移量相同。你已经在处理这个问题了；从 -1 开始并在操作之前递增有点不正统，但它有效。

现在你实际上有 wordList[i] 来分配，直到 i >= 20 为止。

unsigned int wordListLen = 0;
...
if(wordListLen >= 20){
   // list full!
   break;
} else {
   unsigned int wlen = strnlen(word,10)+1;
   wordList[wordListLen] = malloc( wlen * sizeof(char) );
   strncpy(wordList[wordListLen], word, 10);
   wordListLen++;
   ...print...
}

在分配 '\0' 的副本时，不要忘记 word 的位置!