在C ++中的删除操作中出现段错误

Question

我正在为C ++赋值编写代码，它是使用二进制搜索树的字典实现。 我的代码可以编译，但是当我尝试“删除”时，出现段错误。 任何想法为什么会发生。 谢谢

这是我的代码

// this function calls the deleteNode function where the deletion is done
void BST::deleteContent(string *word)
{
    deleteNode(word, root);
}
// a helper fuuntion for the deletecontent function
//uses recursion to find the node to be deleted
void BST::deleteNode(const string *word, Node *&nodePtr)
{
    if(word < nodePtr->word)
        deleteNode(word, nodePtr->left);
    else if(word > nodePtr->word)
        deleteNode(word, nodePtr->right);
    else
        makeDeletion(nodePtr);
}
// a helper function for the deleteNode function
void BST::makeDeletion(Node *&nodePtr)
{
    Node *tempNodePtr;

    if(nodePtr == NULL)
        cout<< "cannot delete empty node. \n";
    // if node has no right child
    else if (nodePtr->right == NULL)
        {
            tempNodePtr = nodePtr;
            nodePtr = nodePtr->left; // reattach child
            delete tempNodePtr;
        }
    else if(nodePtr-> left == NULL)
        {
            tempNodePtr = nodePtr;
            nodePtr = nodePtr->right; // reattach child
            delete tempNodePtr;
        }
    // if node has 2 children
    else
        {
            tempNodePtr = nodePtr->right;
            while (tempNodePtr->left)
                tempNodePtr = tempNodePtr->left;
            tempNodePtr->left = nodePtr->left;
            tempNodePtr = nodePtr;
            nodePtr = nodePtr->right;
            delete tempNodePtr;
        }
}

编辑 ：

谢谢你们！！ 从您的帖子中，我意识到检查节点是否为最后一个节点并且没有子节点是个好主意。 我在deleteNode中添加了此检查

    if((nodePtr->left) && word < nodePtr->word)
    {
        do something
    }

我做了正确的工作，没有抛出任何错误或段错误。 非常感谢！！！！

Answer 1

情况1：空树：

假设您的树为空： root将为nullptr 。 所以deleteContent()将调用deleteNode()与参数nullptr为nodePtr 。

你在那里做的第一件事是比较word与nodePtr->word不先检查nodePtr不nullptr 。 这是您出现分割错误的第一种情况！

情况2：删除不在树中的单词：

在这种情况下，将递归调用deleteNode()直到到达没有后代的叶节点。 由于搜索到的单词在树中不存在，因此它的名称比nodePtr-> word少，或者小于nodePtr-> word，但绝不相等。 deleteNode()然后将调用本身，再次通过论证nullptr为nodePtr ，如情况1.同样，你将有一个分割的错！

情况1和2的解决方案：在deleteNode（）中控制nullptr：

void BST::deleteNode(const string *word, Node *&nodePtr)
{
    if (nodePtr==nullptr) 
        cout << word << " not found in the tree\n";
    else if (word < nodePtr->word)
        ...   // the rest as in your original function 
}

makeDeletion()现在应该叫deleteNode()当且仅当nodePtr不是null word==nodePtr->word 。 摆脱第一个if（）在任何情况下都不再为真。 可以用断言at代替它以验证不变量。

情况3：删除树中的单词：

至少在我查看数据结构图的情况下，所有这三种情况似乎都起作用（甚至具有两个空指针的叶子节点）。

但是，我建议您验证Node::~Node() temNodePtr Node::~Node() ：在所有情况下，您都重新附加子节点，然后删除旧节点（ temNodePtr ），而无需将其子节点设置为nullptr 。 所以，我想知道是否~Node()只是破坏节点，而不考虑其照顾孩子（当时makeDeletion()应该工作），或者如果它是一个递归的析构函数，delete一个节点和它的孩子（当时makeDeletion()是行不通的，因为你会删除您刚刚重新连接的节点，而不会引起注意，从而在第一次出现seg.fault）。

顺便说一句，即使NULL也可以正常工作，对于指针，nullptr也许比NULL更合适。