c++ - 二进制搜索树中的Double Delete(？)

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

但是，每当我释放内存时，都会遇到访问冲突错误。通过它，我看到我正在访问释放函数上的内存地址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;
}

最佳答案

因此，在这部分代码中

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==来比较两个对象。看起来会更漂亮。