我上了这门课(从我之前的问题开始,我刚改变了一些东西):
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个叶Left
和Right
的树(其中包含类树,但在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!"
收到错误。
我不认为它与我的问题有关但是导致了什么?
答案 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;}
...
};
因为您想要返回对实际子树对象的引用,而不是副本。 使用此版本,您的代码应该按照您的预期运行,并且上面的断言不会失败。