智能指针与类,它指向哪里?

时间:2013-10-12 11:11:46

标签: c++ class smart-pointers

我上了这门课(从我之前的问题开始,我刚改变了一些东西):

class Tree{
private:
    shared_ptr<Tree> Left;
    shared_ptr<Tree> Right;
    int Info;
public:
    Tree() :Info(0) ,Left(nullptr), Right(nullptr) {};
    Tree(int num) : Info(num), Left(nullptr) , Right(nullptr){};
    Tree& operator=(const Tree &src);
    void SetLeft(int num){Left.reset(new Tree(num));};
    void SetRight(int num){Right.reset(new Tree(num));};
    void SetInfo(int num){Info = num;};
    Tree GetLeft(){return *Left;};
    Tree GetRight(){return *Right;};
    int GetInfo(){return Info;};
};
Tree& Tree::operator=(const Tree &src){
    Left = src.Left;
    Right = src.Right;
    Info = src.Info;
    return *this;
}

此类创建一个具有2个叶LeftRight的树(其中包含类树,但在shared_ptr中)。

所以我这样做了:

Tree tr(75);
tr.SetLeft(5);
tr.GetLeft().SetLeft(7);
tr.SetRight(3);
tr.GetRight().SetRight(1);

当我尝试cout the Left-&gt; Left或Right-&gt; Right Info时(这是每棵树的值):

cout << tr.GetRight().GetRight().GetInfo();

我得到一个奇怪的错误,关于智能指针(关于reset函数),编译器在这里创建了一个断点:

void _Reset(const _Ptr_base<_Ty2>& _Other)
    {   // release resource and take ownership of _Other._Ptr
    _Reset(_Other._Ptr, _Other._Rep);
    }

我做错了什么? 谢谢! :)


编辑:

我试着检查一下,因为nullptr和我编译了这个:

shared_ptr<int> i;
int ib=6;
i=nullptr;
i.reset(&ib);
cout << *i;

编译完程序后,我从无处"Debug Assertion Failed!"收到错误。 我不认为它与我的问题有关但是导致了什么?

2 个答案:

答案 0 :(得分:2)

我不知道这是否是造成这一特定错误的原因,但这最终会让你感到痛苦:

Tree GetLeft(){return *Left;};

返回Left的副本。您想要返回参考:

Tree& GetLeft(){return *Left;}

}之后也注意没有分号)

答案 1 :(得分:1)

此代码:

class Tree{
...
    Tree GetLeft(){return *Left;};
    Tree GetRight(){return *Right;};
...
};

分别在返回时创建左右树的副本。 这意味着以下陈述:

tr.GetLeft().SetLeft(7);
tr.GetRight().SetRight(1);

分别设置'tr'树的左右儿童的临时副本的左右树。 您可以使用以下断言轻松地对此进行测试(假设在当前上下文中允许访问Left和Right),这将失败,因为将给定子树的临时副本的地址与原始子地址:

assert(&tr.GetLeft() == &tr.Left.get());
assert(&tr.GetRight() == &tr.Right.get());

正确的版本应为:

class Tree{
...
    Tree& GetLeft(){return *Left;}
    Tree& GetRight(){return *Right;}
...
};

因为您想要返回对实际子树对象的引用,而不是副本。 使用此版本,您的代码应该按照您的预期运行,并且上面的断言不会失败。