成员指针在函数返回时被覆盖?

时间:2012-02-14 08:24:47

标签: c++ pointers data-structures binary-search-tree memory-corruption

我一直在练习我的C ++,因为它从大学开始就变得有点生疏了,而且我有一个奇怪的问题,一旦我的函数返回,成员值就会被覆盖。

template <class T>
class BstNode
{
    public:
        T value;
        BstNode<T>* left;
        BstNode<T>* right;
        BstNode<T>* parent;

        BstNode()
        { left = right = parent = NULL; }
        BstNode(T value)
        { this->value=value; left=right=parent=NULL;}
        BstNode(T value, BstNode<T>* parent)
        { this->value=value; this->parent=parent; left=right=NULL;}
};

template <class T>
class BinarySearchTree 
{
    protected:
        BstNode<T>* root;

        void removeNode(BstNode<T>* node);
        void addChild(T value, BstNode<T>* node);
        BstNode<T>* find(T value, BstNode<T>* node);
    public:
        BinarySearchTree()
        { root = NULL; }
        ~BinarySearchTree()
        { removeNode(root); }

        BinarySearchTree<T> insert(T value);
        bool contains(T value);
        BinarySearchTree<T> remove(T value);

        void print();

        BstNode<T>* getRoot() {return root;}

};

template <class T>
BinarySearchTree<T> BinarySearchTree<T>::insert(T value)
{
    if (root == NULL)
    {
        root = new BstNode<T>(value);       
    }
    else
    {
        addChild(value, root);
    }
    cout << "VAL: " << root->value << endl << "LEFT: " << root->left << endl << "RIGHT: "<< root->right << endl << "ADDR: " << root <<endl;
    return *this;
}
template <class T>
void BinarySearchTree<T>::addChild(T value, BstNode<T>* node)
{

    if (value > node->value)
    {
        cout <<"\tgt"<<endl;
        if (node->right == NULL)
        {
            node->right = new BstNode<T>(value, node);
        }
        else
        {
            addChild(value, node->right);
        }
    }
    else
    {
        cout<<"\tlte"<<endl;
        if (node->left == NULL)
        {
            node->left = new BstNode<T>(value, node);
        }
        else
        {
            addChild(value, node->left);
        }
    }
}

// [other member functions]


int main()
{
    BinarySearchTree<int> tree;
    BstNode<int> *n;
    n = tree.getRoot();
    cout << "ADDR: " << n <<endl<<endl;
    tree.insert(5);
    n = tree.getRoot();

    cout << "VAL: " << n->value << endl << "LEFT: " << n->left << endl << "RIGHT: "<< n->right << endl << "ADDR: " << n << endl;
    return 1;
}

我的功能输出是:

$ ./bst
ADDR: 0

VAL: 5
LEFT: 0
RIGHT: 0
ADDR: 0xa917c8

VAL: 11085080
LEFT: 0xa917a8
RIGHT: 0
ADDR: 0xa917c8

我不明白为什么根节点中的值发生了变化,但指针仍指向同一位置。我唯一能想到的是根节点是在堆栈上创建而不是在堆中分配的,但是new是否确保在C ++中正确分配了内存?

1 个答案:

答案 0 :(得分:3)

我认为问题在于您的insert方法按值返回BinarySearchTree,但您没有定义复制构造函数。因此,这会生成BinarySearchTree的浅表副本,返回它,并导致副本的析构函数触发。然后删除存储为根的BstNode,但由于复制的BinarySearchTree与原始树共享BstNode,因此您将在原始树中丢弃内存。您获得的错误是当您尝试再次访问该节点时访问释放的内存。

要解决此问题,要么让insert函数返回对树的引用(因此不进行复制),要么定义复制构造函数或赋值运算符。理想情况下,两者都做。 : - )

希望这有帮助!