将数据插入到 trie 中

Question

所以我正在尝试将 insert 数据放入 trie 中，并且我的代码工作正常。但是后来我稍微改变了我的插入函数，它不再起作用并且还导致内存泄漏。对我来说，insert 的两个版本都做同样的事情，但显然它们不是。有人可以向我解释为什么吗？提前致谢。

这是有效的代码

#include <stdio.h>
#include <stdbool.h>
#include <ctype.h>
#include <stdlib.h>
#include <string.h>

#define SIZE 26

#define hash(c) (tolower(c) - (int)'a')

typedef struct node{
    bool endWord;
    struct node* children[SIZE];
} node;

void freeTrie(node* root){

    if(root == NULL) return;

    for (size_t i = 0; i < SIZE; i++) {
        freeTrie(root->children[i]);
    }

    free(root);
}

node* newNode(){
    node* new = NULL;

    new = (node*) malloc(sizeof(node));

    if(new != NULL){

        new->endWord = false;

        for(int i = 0; i < SIZE; i++)
            new->children[i] = NULL;
    }

    return new;
}

void insert(node* root, const char* data){

    node* temp = root;

    for (size_t i = 0, len = strlen(data); i < len; i++) {

        int index = hash(data[i]);

        if(temp->children[index] == NULL){

            temp->children[index] = newNode();

            if (temp->children[index] /*still*/ == NULL){
                printf("Something went wrong\n");
                return;
            }
        }

        temp = temp->children[index];
    }
    temp->endWord = true;
}

bool search(node* root, const char* data){

    node* temp = root;

    for (size_t i = 0, len = strlen(data); i < len; i++) {

        int index = hash(data[i]);

        temp = temp->children[index];

        if (temp == NULL){
            printf("search end here\n");
            return false;
        }
    }

    return (temp != NULL && temp->endWord);
}

int main() {

    char data[][8] = {"fox", "foo", "dog", "do"};

    node* root = newNode();

    if(root == NULL){
        printf("Something went wrong\n");
        return 1;
    }

    for (size_t i = 0, dataSize = sizeof(data)/sizeof(data[0]); i < dataSize; i++) {
        insert(root, data[i]);
    }

    printf("Check: \n");

    char output[][32] = {"not found", "found"};

    // char s[5];
    // fscanf(stdin, "%s", s);

    printf("%s\n", output[search(root, "fox")]);

    freeTrie(root);

    printf("Done\n");

    return 0;
}

这是让我困惑的insert

void insert(node* root, const char* data){

    node* temp = root;

    for (size_t i = 0, len = strlen(data); i < len; i++) {

        int index = hash(data[i]);

        temp = temp->children[index];

        if(temp == NULL){

            temp = newNode();

            if (temp /*still*/ == NULL){
                printf("Something went wrong\n");
                return;
            }
        }
    }

    temp->endWord = true;
}

PS：我这样做是针对 CS50x 课程的问题集，其中我必须将包含 143091 个单词（按字母顺序）的字典加载到我的 trie 中。当工作人员仅用 0.02 秒和 0.01 秒完成相同的工作时，我的程序加载大约需要 0.1 秒，卸载大约需要 0.06 秒。我不允许查看员工的源代码，但我猜他们使用 trie 存储数据。如何改进我的代码以获得更快的运行时间？如果我将数据存储在数组中然后进行二进制搜索，它会运行得更快吗？

Answer 1

当你写作时

temp = temp->children[index];

您将包含在temp->children[index]（我称之为A）中的值复制到一个名为temp 的完全独立变量中。当您稍后修改temp 时，您只修改temp，而不是A。也就是说，所有新节点都不会插入到 trie 中。