0

这个函数的任务相当简单。给定一个 char* 数组、一个指向文件的指针和一个最大字长，它会读取文件并将每个字一个接一个地复制到 char* 数组中。由于文件中每行一个单词，因此用作单词之间的分隔符是有意义\n的。所以考虑到这一点，代码应该很容易解释：

void loadDictionary(char* strDictionary[], FILE* filePointer, int nMaxLetters)
{
    int nNameCount= 0, nCursor = 0;
    char* strCurrent;
    char cCurrent;

    //allocate space for a word
    strCurrent = malloc(nMaxLetters * sizeof(char));

    while ((cCurrent = fgetc(filePointer)) != EOF) {

        if(cCurrent != '\n')
        {
            strCurrent[nCursor] = cCurrent;
            nCursor++;

        } else { //then we've reached the end of the line (word)

            //add null termination to string
            strCurrent[nCursor] = '\0'; //SEG FAULT

            //copy string to dictionary
            memcpy(strDictionary[nNameCount], strCurrent, strlen(strCurrent)+1);

            //increment count
            nNameCount++;

            //reset the cursor
            nCursor = 0;
        }
    }
}

此代码在我调用的行处生成分段错误strCurrent[nCursor] = '\0';。我不知道为什么，因为从表面上看，这个操作似乎应该与我调用的另一个块中的操作没有什么不同strCurrent[nCursor] = cCurrent;。strCurrent 应该分配了足够多的空间来存储所有必要的字符。所以，我有些不知所措。帮我解决这个问题，伙计们。

注意：我想我可能会更容易使用fgets而不是fgetc完成这项任务。我很可能会转向那个；但是，由于我遇到了一个我不理解的错误，所以在我理解它之前，我不想让它不管。

编辑：

有人指出，memcpy操作中可能会出现错误，可能strDictionary是分配不当造成的。这是分配的main块。strDictionary也许我犯了一个错误：

int main(int argc, char* argv[])
{
    char** strDictionary;
    FILE* filePointer;
    int nResults = 0, nLines = 0, nNumLines, nMaxChars, i;

    filePointer = fopen("dictionary.txt", "r");

    //obtain the number of lines and the maximum word size of the dictionary
    countLines(filePointer, &nNumLines, &nMaxChars);

    //allocate memory for strDictionary
    strDictionary = malloc(nNumLines * nMaxChars * sizeof(char));
    printf("%d words in dictionary. Longest word is %d letters\n",
            nNumLines, nMaxChars);
    //Output here correctly prints: 1000 and 21

    //reset the file pointer (not sure if this is a necessary step, but oh well)
    filePointer = fopen("dictionary.txt", "r");

    //load dictionary into memory
    loadDictionary(strDictionary, filePointer, nMaxChars);
    for (i=0; i<10; i++)
        printf("%dth element of dictionary: %s\n", i, strDictionary[i]);

    return 0;
}

编辑2：

好的，我决定使用fgets()而不是fgetc()大大简化我的功能。我也做了我认为正确的malloc()操作strDictionary。但是，我仍然遇到段错误。这是更新的代码：

void loadDictionary(char* strDictionary[], FILE* filePointer, int nMaxLetters)
{
    printf("Call to loadDictionary. nMaxLetters = %d\n", nMaxLetters);
    int nWordCount= 0, nCursor = 0;
    char* strCurrent;
    char cCurrent;

    strCurrent = malloc(nMaxLetters); //allocate space for a word


    while (fgets(strCurrent, nMaxLetters, filePointer) != NULL)
    {
        memcpy(strDictionary[nWordCount], strCurrent, strlen(strCurrent)+1);
        nWordCount++;
    }
}

int main(int argc, char* argv[])
{
    char** strDictionary;
    FILE* filePointer;
    int nResults = 0, nLines = 0, nNumLines, nMaxChars, i;
    filePointer = fopen("dictionary.txt", "r");

    //count the lines in the file (works fine)
    countLines(filePointer, &nNumLines, &nMaxChars);

    //allocate space for the dictionary
    strDictionary = malloc(nNumLines * sizeof(char*));
    for (i = 0; i<nLines; i++)
        strDictionary[i] = malloc(nMaxChars * sizeof(char));
    printf("%d words in dictionary. Longest word is %d letters\n",
            nNumLines, nMaxChars);

    //load dictionary into array
    filePointer = fopen("dictionary.txt", "r");
    loadDictionary(strDictionary, filePointer, nMaxChars);
    for (i=0; i<10; i++)
        printf("%dth element of dictionary: %s\n", i, strDictionary[i]);

    return 0;
}
4

1 回答 1

1

这里：

char cCurrent;
...
while ((cCurrent = fgetc(filePointer)) != EOF) {

您正在将fgetc()type 的值截断int为char。这可能会导致 while 条件无法正确识别EOF。cCurrent必须是int。

这里：

//allocate space for a word
strCurrent = malloc(nMaxLetters * sizeof(char));

nMaxLetters必须考虑一个额外的字符来表示字符串 NUL 终止符。是否入账？

顺便说一句，sizeof(char)始终为 1。

现在，这个参数声明：

char* strDictionary[]

相当于：

char** strDictionary

或者，IOW，一个指向指向 a 的指针的指针char。这是因为在 C 中，数组永远不会作为参数传递，只有指向它们的第一个元素的指针才是，尽管括号中的欺骗性语法暗示某些东西是数组。

这一行：

memcpy(strDictionary[nNameCount], strCurrent, strlen(strCurrent)+1);

将把nNameCount'th 指针指向 achar并写入它指向的字符数据。

但是调用函数分配的字符串缓冲区（长度nMaxLetters）是否至少与文件中的行数一样多？在将这个数组传递给 char 之前，它是否会用指向这些缓冲区的指针填充一些指向 char 的指针数组loadDictionary()？IOW，此代码期望调用者执行以下操作：

#define nMaxEntries 1000

char* dictionary[nMaxEntries];
int i;
FILE* f;

...

for (i = 0; i < nMaxEntries; i++)
  dictionary[i] = malloc(nMaxLetters);

loadDictionary(dictionary, f, nMaxLetters);

内存分配失败必须在上面的代码中检查。另外，我强烈建议传入nMaxEntries或使用它，loadDictionary()这样如果文件的行数多于nMaxEntries. nNameCount不应该超越nMaxEntries。

更新到更新的问题...

这里：

char** strDictionary;
...
strDictionary = malloc(nNumLines * nMaxChars * sizeof(char));

您没有按照上述分析char的loadDictionary()预期创建一个指针数组，而是创建了一个char. 因此，段错误很可能不会发生在这一行：

 strCurrent[nCursor] = '\0'; //SEG FAULT

但是在下一个，这在调试器中可能不明显，直到您放大并查看代码的反汇编：

 //copy string to dictionary
 memcpy(strDictionary[nNameCount], strCurrent, strlen(strCurrent)+1);

更新2：

我不明白你为什么现在为nNumLines指针分配空间：

strDictionary = malloc(nNumLines * sizeof(char*));

但是在这些nNumLines指针中，您初始化了nLines指针（nLines如果我正确阅读了您的最新代码，则永远不会变成 0 以外的任何东西）：

for (i = 0; i<nLines; i++)
    strDictionary[i] = malloc(nMaxChars * sizeof(char));

有什么诀窍？错字？

于 2012-10-10T21:17:11.487 回答