我正在编写一个程序,用于计算每个单词在文本文件中出现的次数。我收到一个运行时错误,说明:分段错误(核心转储)。我理解这与尝试访问尚未分配的内存有关。
另外,我收到关于我对getline的争论的警告,我不确定我是否正确使用它。任何建议表示赞赏。
我的代码是:
#include <stdio.h>
#include <string.h>
#include <ctype.h>
#include <stdlib.h>
#define MAXWORDS 5000
#define MAXLINE 1000
#define MAXWORDLEN 100
int count = 0;
struct wordcount *wordPtr[MAXWORDS];
typedef struct wordcount *wordcountptr;
typedef struct wordcount {
char word[50];
int count;
} wordcount_t;
main()
{
int wordfound = 0;
int len;
char line[MAXLINE];
int printcount;
while ((len = getline(line, MAXLINE, stdin))> 0)
{
int i = 0;
int j = 0;
for( ; i<len; i++)
{
if(line[i] != isalnum(line[i]) && line[i] != 32)
line[i] = 32;
else
line[i] = tolower(line[i]);
}
for( ; j<len; j++)
{
char currentword[MAXWORDLEN];
if(line[j] != 32)
{
for(i=0; line[j] != 32; i++)
currentword[i] = line[j];
}
if(line[j] == 32)
{
for(i=0;i<MAXWORDS; i++)
{
if(strcmp(currentword, (*wordPtr[i]).word) == 0)
{
(*wordPtr[i]).count++;
wordfound = 1;
}
}
if(wordfound == 0)
{
wordPtr[i] = (wordcount_t*)malloc(sizeof(wordcount_t));
strcpy((*wordPtr[i]).word, currentword);
(*wordPtr[i]).count = 1;
count++;
}
}
wordfound = 0;
}
}
for(printcount = 0; printcount < count; printcount++)
printf("There are %d occurances of the word %s\n", (*wordPtr[printcount]).count, (*wordPtr[printcount]).word);
for(printcount = 0; printcount < MAXWORDS; printcount++)
{
free((void*)wordPtr[printcount]);
wordPtr[printcount]= NULL;
}
}
答案 0 :(得分:2)
getline()
期望第一个agrument属于char **
类型,因此您应该这样称呼它:
getline(&line, MAXLINE, stdin)
因为line
是一个char数组,它等同于char *
而不是必需的char **
。
引用this SO answer使用char **
而非char *
的原因是:
您需要传入一个char **(即一个指向char的指针),以便该函数能够更新它指向的char *的值。
答案 1 :(得分:1)
ssize_t getline(char **lineptr, size_t *n, FILE *stream)
;这是getline
原型,第一个参数是char **
,而不是char *
,您可以这样使用:getline(&line, MAXLINE, stdin)