我在网上的insert函数中遇到了段错误:

current->isWord = true;

一切都可以正常编译,没有警告或错误(g++ -Wall -Wextra)。我的main函数仅调用insert函数一次,将无法工作。这是我的代码;它是我的.h.cpp文件之间的混合体:
const int alphabetSize = 26;

struct Node
{
    bool isWord;
    Node* child[alphabetSize];
};

Dictionary::Dictionary()
{
    initNode(head); //Node* head; is defined in my .h file under private:
}

bool Dictionary::isPrefix(string s)
{
    Node* current = endOfString(s, false);
    if (current == NULL)
    {
        return false;
    }
    else
    {
        return true;
    }
}

bool Dictionary::isWord(string s)
{
    Node* current = endOfString(s, false);
    if (current == NULL)
    {
        return false;
    }
    else
    {
        return current->isWord;
    }
}

void Dictionary::insert(string s)
{
    Node* current = endOfString(s, true);
    current->isWord = true; //segfault here
}

//initializes a new Node
void Dictionary::initNode(Node* current)
{
    current = new Node;
    current->isWord = false;
    for (int i = 0; i < alphabetSize; i++)
    {
       current->child[i] = NULL;
    }
}

//returns a pointer to the Node of the last character in the string
//isInsert tells it whether it needs to initialize new Nodes
Node* Dictionary::endOfString(string s, bool isInsert)
{
    Node* current = head;
    Node* next = head;
    for (unsigned int i = 0; i < s.length(); i++)
    {
        if (isalpha(s[i]) == true)
        {
            int letter = (tolower(s[i]) - 'a');
            next = current->child[letter];
            if (next == NULL)
            {
                if (isInsert == false)
                {
                    return NULL;
                }

                initNode(next);
                current->child[letter] = next;
            }
            current = current->child[letter];
        }
    }

    return current;
}

最佳答案

initNode创建一个新的Node并将其初始化,但随后将其丢弃。因为current是按值传递的,所以在函数内部对其进行修改时,更改不会传播到initNode之外。最直接的解决方法是使其通过引用传递:

void Dictionary::initNode(Node*& current)

关于c++ - C++字典特里实现,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/15892922/

10-09 17:15