输出不可读

时间:2013-12-08 20:34:30

标签: c graph tree

我正在编写一个程序,通过遵循翻译路径来翻译给定的单词。 给定单词的每个字母代表一个特定节点。

输出应该导致所有要翻译的单词后跟相应的翻译,但我收到的是输出而不是:

a: 4��t�    xp� t����
an: 4��t�   xp� t����
ant: 4��t�  xp� t����
at: 4��t�   xp� t����
atom: 4��t� xp� t����
no: 4��s�   xp� t����
not: 4��s�  xp� t����
tea: 4��q�  xp� t����
ten: 4��q�  xp� t����

main.c:

    int main()
    {
        struct Trie* trie = trie_alloc();
        trie_insert_from_file(trie, "dictionary.txt");
        trie_print_mappings(trie);
        trie_free(trie);

        return 0;
    }

trie.c:

   int trie_insert(Trie* trie, const char* key, const char* value)
    {
        ...
    }

    void trie_insert_from_file(Trie* trie, const char* file_name)
    {
        FILE* file = fopen(file_name, "r");
        if (file == NULL)
        {
            fprintf(stderr, "Unable to open %s for reading: %s\n",
                    file_name, strerror(errno));
            return;
        }
        while (!feof(file))
        {
            char key[64];
            char value[64];
            int nb_matched = fscanf(file, "%63[a-z] : %63[a-z]\n", key, value);

            if (nb_matched == 2)
            {
                  trie_insert(trie, key, value);
            }
            else
            {
                fprintf(stderr, "Syntax error while reading file\n");
                fclose(file);
                return;
            }
        }
        fclose(file);
    }

    static char* str_append_char(const char* str, char c)
    {
        size_t len =  strlen(str);
        char* new_str = malloc(len + 2);
        strcpy(new_str, str);
        new_str[len] = c;
        new_str[len + 1] = '\0';
        return new_str;
    }

    static void trie_print_mappings_aux(Trie* trie, const char* current_prefix)
    {
        if (trie->value != NULL)
            printf("%s: %s\n", current_prefix, trie->value);
        int i;
        for (i = 0; i < TRIE_NB_CHILDREN; i++)
        {
            Trie* child = trie->children[i];
            if (child != NULL)
            {
                char* child_prefix =
                    str_append_char(current_prefix, trie_get_child_char(i));
                trie_print_mappings_aux(child, child_prefix);
                free(child_prefix);
            }
        }
    }

    void trie_print_mappings(Trie* trie)
    {
        trie_print_mappings_aux(trie, "");
    }

trie.h:

#define TRIE_NB_CHILDREN 26

typedef struct Trie
{
    char* value;
    struct Trie* children[TRIE_NB_CHILDREN];
} Trie;

当我使用函数 trie_insert 手动插入数据而不使用 insert_from_file 读取.txt文件时,不会发生这种情况。

Eg. trie_insert(trie, (const char*)&"ten", (const char*)&"tien");
    trie_insert(trie, (const char*)&"no", (const char*)&"nee");
    trie_insert(trie, (const char*)&"not", (const char*)&"niet" );
...

经过一番研究,我相信这可能与我写的超出允许的内存位置有关。但我不知道哪里出错了。

函数 insert_from_file str_append_char trie_print_mapping_aux * 应该正常工作,因为它们是给我的。所以错误可能在我实现的 trie_insert 中。

非常感谢任何帮助。

1 个答案:

答案 0 :(得分:2)

您正在将trie->value指针(在trie_insert中)设置为存储在堆栈中的字符串(在trie_insert_from_file - char value[64]中)。

当函数返回时,存储在堆栈中的变量“丢失” - 这意味着你的trie指向了它不应该的位置(在同一指针地址上会有完全不同且无关的数据)。

解决方案是将value字符串复制到堆:

if(*key == '\0')
{
    trie->value = (char*)malloc(sizeof(char)*64);
    strcpy(trie->value, value);
    return 1;
}

请确保在不需要时释放已分配的内存。