从二叉搜索树中删除值

Removing a value from a binary search tree

本文关键字:删除 搜索树      更新时间:2023-10-16

我正在编写一个函数,该函数将从具有给定键关联的二叉搜索树中删除条目。到目前为止,我的代码有这个:

template <typename Item, typename Key = Item>
bool BSTree<Item,Key>::remove(const Key& key) {
    bool removed = false;
    Node* ptr = root;
    if(ptr == NULL)
        return removed;
    while(key != ptr) {
        if(ptr == NULL)
            return removed;
        else if(key > ptr) 
            ptr = ptr->right();
        else
            ptr = ptr->left();
    }
    removed = true;
    Item max = max(ptr);
    ptr->data() = max;
    Node* prev = ptr;
    while (ptr != NULL) {
        prev = ptr;
        ptr = ptr->right();
    }
    delete ptr;
    if (prev->left() != NULL) 
        prev = copy(prev->left());
    delete prev;
    return removed;
}

Copy 是我已经编写的另一个函数,它将使用递归方法将所有值从某个节点传输到树的末尾。我相信这个功能应该可以工作,但我不完全确定,并希望得到一些反馈。

我对函数的最后三行也有问题。在它们中的每一个中,"if"、"delete"和"return"都带有下划线,并给我错误"错误:预期的声明"。我不知道这是怎么回事,非常感谢反馈!

我直言,我看到了几件事。

  1. 对于BST移除,你需要一种更好的移植方法,除了你的"副本"之外,至少你应该找到下一个最小的节点,而不是要移除的节点。

  2. PTR->data() = max;我猜你的"data()"方法返回对项目中成员的引用。没有错,只是感觉很奇怪。

  3. 有两个"删除"语句,但您不应该只删除一个项目吗?

  4. 在本节中:

    while (ptr != NULL) {
        prev = ptr;
        ptr = ptr->right();
    }
    delete ptr;
    

    while 退出后,ptr 的值为 NULL,因此您正在删除 NULL,而不是 cool。