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 
printf("Enter word :");
scanf("%s",word);
printf("Taken word: %s at %pn",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 0n");
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[])开头，并以空字节结尾C 字符串本质上不知道它们的大小;程序员负责确保不会尝试访问分配给字符串的内存之外的内存。 当你这样做时，你会得到一个段错误。

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

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

因此，您要做的是告诉scanf最多读取多少字节。 最简单的方法是使用 scanf 的可选"字段宽度说明符"：

scanf("%9s",word)

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

如果我添加一个长度为 20+ 的单词，我的程序应该忽略，并继续

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

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

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

好吧，wordList[]正好是一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...
}

分配word副本时不要忘记''的地方！