c++ - 二叉搜索树删除（Inorder Pred 方法）C++

Question

好的，所以我认为它已修复，但我得到的结果完全不一致。我从头开始重写它，这是我的结果。我没有错误，没有崩溃，只是没有删除它们。它完全把树弄乱了，给了我更多的叶子，把所有的东西都混在一起了。不知道还能去哪里

template <class T>
void BST<T>::remove(struct Node<T>*& root, const T& x)
{
   Node<T>* ptr = root;
   bool found = false;
   Node<T>* parent;


   while (ptr != NULL && !found)
   {
       if (x < ptr->data)
       {
           parent = ptr;
           ptr = ptr->left;
       }
       else if (x > ptr->data)
       {
           parent = ptr;
           ptr = ptr->right;
       }
       else
           found = true;
   }

   if (found == false)
       return;
   else
   {
       if(ptr->left != NULL && ptr->right != NULL)
       {
           Node<T>* inOrderPtr = ptr->left;
           parent = ptr;
           while (inOrderPtr->right != NULL)
           {
               parent = inOrderPtr;
               inOrderPtr = inOrderPtr->right;
           }

           ptr->data = inOrderPtr->data;
           ptr = inOrderPtr;
       }
    Node<T>* subPtr = ptr->left;
    if (subPtr == NULL)
        subPtr = ptr->right;

    else if (parent->left == ptr)
        parent->left = subPtr;

    else
        parent->right = subPtr;

    delete ptr;
    }

Answer 1

实际发生的情况是可能的搜索被反转了，所以它实际上会继续正确，但数据并没有真正正确匹配，所以它似乎会撞墙。

if (root->data < x)
        remove(root->left, x);
    else 
        remove(root->right, x);

本来应该

if(x < root->data)
remove(root->left, x);
else
remove(root->right, x);

Answer 2

你不应该remove()在第三种情况下递归调用（你的“不确定这是否正确”的评论是）。在要删除的节点有两个孩子的情况下，您要做的是找到左孩子的最右边的孩子（正如您所做的那样；结果节点存储在 中parent）。该节点没有右孩子 - 使其右孩子成为要删除的节点的右孩子。然后只需将root变量更改为其左孩子；无需更改data任何节点中的成员或remove递归调用。

在图片中：

前：
         r <- 这里的根点
       / \
      / \
     抗体
    / \ / \
   xcyy
      / \
     xd
        /
       X

后：
      a <-- 这里的根点
     / \
    xc
       / \
      xd
         / \
        xb
           / \
          年年

Answer 3

在树中找到的每个 T 都是唯一的吗？看起来它们来自您的代码...

看起来这应该有效：

在其他情况下删除根节点：

Node<T> *tmp_r = root->left;
Node<T> *parent = root;
while (tmp_r->right != NULL)
{
    parent = tmp_r;
    tmp_r = tmp_r->right;
}
Node<T> *tmp_l = tmp_r;
while (tmp_l->left != NULL)
    tmp_l = tmp_l->left;

tmp_l->left = root->left;
tmp_r->right = root->right;
parent->right = NULL;

parent = root;
root = tmp_r;
delete parent;