分段错误(核心转储)错误

时间:2013-12-04 06:31:48

标签: c segmentation-fault malloc getline

我正在编写一个程序,用于计算每个单词在文本文件中出现的次数。我收到一个运行时错误,说明:分段错误(核心转储)。我理解这与尝试访问尚未分配的内存有关。

另外,我收到关于我对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;
    }
}

2 个答案:

答案 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)