在二进制搜索树中双删除(?)

时间:2013-04-21 05:50:51

标签: c++ data-structures recursion binary-search-tree

我正在编写一个代码来测试两个二叉搜索树是否相等。

但是,每当我释放内存时,我都会遇到访问冲突错误。通过它,我看到我在我的释放功能上访问内存地址0xfeefee。我在Cell析构函数中获得了访问冲突。

此外,我真的不知道这个功能是否有效,但是我并没有真正寻求帮助 - 尽管仍然可以帮助你。

解除分配功能:

~Cell(void) {
    if (left) { delete left; }
    if (right) { delete right; }
}

功能:

bool BST::isEqualTo(const BST& that) const{
    if(root <= 0 && that.root <= 0){
        return true;
    }else if(root <= 0 || that.root <= 0){
        return false;
    }
    if(root->val != that.root->val){
        false;
    }
    /*Cell* saved_node1 = new Cell(*root);
    Cell* saved_node2 = new Cell(*that.root);*/
    BST a, b, c, d;
    a.root = root->left;
    b.root = that.root->left;
    c.root = root->right;
    d.root = that.root->right;
    if(a.isEqualTo(b) && c.isEqualTo(d)){
        /*a.root = saved_node1;
        b.root = saved_node2;
        c.root = saved_node1;
        d.root = saved_node2;*/
        return true;
    }
    return false;
}

树析构函数:

void BST::destroy(void) {
    length = 0;
    delete root;
    root = nullptr;
}

1 个答案:

答案 0 :(得分:1)

所以,在这部分代码中

BST a, b, c, d;
a.root = root->left;
b.root = that.root->left;
c.root = root->right;
d.root = that.root->right;

您创建BST个对象,一旦该函数完成就会被销毁。在所有内存之后,为root指针分配的内容将被释放。

我建议你写另一个比较函数:

bool _is_equal(Cell* c1, Cell* c2)
{
    if(c1 == nullptr && c2 == nullptr)
        return true;
    else if(c1 == nullptr || c2 == nullptr)
        return false;

    return  (c1 -> val == c2 -> val) &&
            _is_equal(c1 -> left,  c2 -> left) &&
            _is_equal(c1 -> right, c2 -> right);
}

并在你的功能中调用它

bool BST::isEqualTo(const BST& that) const
{
    return _is_equal(root, that.root);
}

当然,你应该重载operator==来比较两个对象。看起来会更漂亮。