Question

我开始使用MIT OCW入门课程学习C（课程可以找到here）。我正在完成一项任务，你必须编写函数，使用给定的模板（在名为prob2.c的文件中找到here）将trie结构实现为英语到法语的字典。基本上，该程序应该要求翻译单词，在trie中查找，并给出翻译。我自己首先尝试实现它，但我一直遇到一个错误，结果似乎是打印“结束”本身。这是一个正在发生的事情的例子，因为它解释起来有点奇怪：

Enter word to translate: Adam
,Adamo> Adam
Enter word to translate: word
,parole mot
Enter word to translate: hello
,eh oh-> salut
Enter word to translate: tree
tree -> arbre

对于这些输入，它应该如下所示，以便翻译仅以逗号分隔：

Enter word to translate: Adam
Adam -> Adam,Adamo
Enter word to translate: word
word -> mot,parole
Enter word to translate: hello
hello -> salut,allo,eh oh
Enter word to translate: tree
tree -> arbre

放弃并查看解决方案（找到here后，请参阅问题6.2，同时包含完整的问题描述），我的代码与给出的内容没有太大差别。无论如何，我使用在OCW上发布的解决方案再次编译，似乎没有变化。我尝试用valgrind运行程序，看看是否有任何明显的内存问题可以解释这一点，但是在查找单词时没有出现任何问题，只是在最后释放trie的一些问题。

这是我怀疑可能是主要问题的功能（这是我之前在帖子中链接的解决方案）：

/* add word to trie, with translation
   input: word and translation
   output: non-zero if new node added, zero otherwise
   postcondition: word exists in trie */
int add_word(const char *word, char *translation) {
    struct s_trie_node *pnode = proot;
    int i, len = strlen(word), inew = 0;
    unsigned char j;
    for (i = 0; i < len; i++) {
        j = word[i];
        if ((inew = !pnode->children[j]))
            pnode->children[j] = new_node();
        pnode = pnode->children[j];
    }
    if (pnode->translation) {
        /* concatenate strings */
        char *oldtranslation = pnode->translation;
        int oldlen = strlen(oldtranslation), newlen = strlen(translation);
        pnode->translation = malloc(oldlen + newlen + 2);
        strcpy(pnode->translation, oldtranslation);
        strcpy(pnode->translation + oldlen, ",");
        strcpy(pnode->translation + oldlen + 1, translation);
        free(oldtranslation);
    } else
        pnode->translation = strcpy(malloc(strlen(translation) + 1), translation);
    return inew;
}

如果有人能向我解释为什么会发生这种情况以及如何解决这个问题，我将不胜感激！

Answer 1

您的翻译是以CRLF结尾输入的，但解析代码正在寻找LF终止符。因此，每个翻译都有一个CR，就在add_word添加逗号之前。因此，当您打印它们时，CR会使它们在屏幕的左边缘打印另一个。

在add_word中添加测试以确认此操作。

if (pnode->translation[oldlen-1] == '\r') ...

字符串内存问题：使用printf打印字符串似乎会覆盖自身

1 个答案: