删除二叉搜索树节点的问题

Question

我正在用 C++ 实现一个二叉搜索树，但是代码有错误，当删除一个有 2 个子节点的节点时结果树连接不好，删除一个有 0 或 1 个子节点的方法效果很好.此代码基于此答案：https://stackoverflow.com/a/31698320/11053027。

编辑： 例如，如果按以下顺序添加值：17、12、25，然后我尝试删除作为根的 17，它被删除，所以当我尝试显示所有元素时：12、25，其中 25 为根。 但是如果现在我尝试删除12它并没有被删除，那么问题是在我第一次删除17时引起的。没有显示错误消息。

请你帮帮我。提前致谢。代码如下：

    Node<E> *getSuccesor(Node<E> &node) const {
       auto *Succesor = node.right;
       while (Succesor != nullptr) {
          if (Succesor->left == nullptr) return Succesor;
          Succesor = Succesor->left;
       }
       return Succesor;
    }

    void remove(Node<E> &node) {
       if (&node == nullptr) return;
       int nChildren = numChildren(node);
       Node<E> *child;

       if (nChildren == 2) {
        child = getSuccesor(node);
        remove(*child);
        child->parent = node.parent;
        child->left = node.left;
        child->right = node.right;
        if (&node == root)
            root = child;
        else {
            if (&node == node.parent->right)
                node.parent->right = child;
            else node.parent->left = child;
        }
    } else {
        child = (node.left != nullptr ? node.left : node.right);

        if (child != nullptr)
            child->parent = node.parent;

        if (&node == root)
            root = child;
        else {
            if (&node == node.parent->left)
                node.parent->left = child;
            else node.parent->right = child;
        }
    }
    Size--;
 }

Answer 1

很难确定，因为你没有给我们minimal complete example，但我认为问题出在：

if (nChildren == 2) {
  child = getSuccesor(node);
  remove(*child);
  child->parent = node.parent;
  child->left = node.left;
  child->right = node.right;

所以现在child 知道它的孩子了。但他们不知道child 现在是他们的父母。他们的parent 指针仍然指向失效的节点（在本例中为'17'），这会在下一次删除时弄乱逻辑。

在这些行之后，添加以下内容：

if(child->left)
  child->left->parent = child;
if(child->right)
  child->right->parent = child;

编辑：同样的错误也出现在 if-else 条件的另一个分支中，但我相信您也可以在那里看到如何修复它。