1

我正在尝试编写一个程序，该程序从定向到标准输入的文件中读取，然后在文件中找到两个最常用的单词。我几乎所有东西都能正常工作，但现在我的 while 循环出现了奇怪的分段错误

char *word=readWord();
int end=0;
while(end==0&&word!=NULL){
    printf("word readn %s\n",word);
    list=addToList(list,word);
    printf("added to list\n");
    word=readWord();
    if(word==NULL){
        end=1;
        printf("word is null\n");
    }
}
printf("done while loop");

当使用包含单词一二三四的文件运行时，此输出如下所示。

单词读了一个
添加到列表
单词读了两个
添加到列表
读三字
添加到列表
单词读四>
添加到列表
词为空
分段故障

readWord功能在其他文件中工作正常；

char * readWord(){

    //temporary char array to read string
    char c,word[BUFFER_SIZE];

    if(scanf("%c",&c)==EOF){
        return NULL;
    }
    while(!((c>='a'&&c<='z')||(c>='A'&&c<='Z'))){
        if(scanf("%c",&c)==EOF)
            return NULL;
    }
    int i;
    for( i=0;(c>='a'&&c<='z')||(c>='A'&&c<='Z');i++){
        word[i]=c;
        scanf("%c",&c);
    }
    word[i+1]='\0';

    //dynamic allocation of just enough memory to store the word
    char * str= malloc(strlen(word) +1);
    strcpy(str,word);
    return str;
}
4

1 回答 1

1

首先是 for 循环末尾的 i 值比单词的长度多一个，所以这是不正确的：

word[i+1]='\0'

它应该是：

word[i]='\0'

其他几条评论：

  • 使用 isalpha(c)
  • 将 malloc/strcpy 折叠成 strdup
  • 写 end==0&&word!=NULL 为 (end == 0) && (word != NULL)
于 2013-03-30T02:44:11.913 回答