我有以下问题: 在 myClass 中,我想默认初始化一个指向 yourClass 的指针,并使用新的 yourClass 地址。 不幸的是,如果我想在任何时候删除指针,我都会得到一个(核心转储)。
class myClass
{
protected:
yourClass * yc;
public:
myClass() { yc = new yourClass(); }
myClass(yourClass * tyc ) { delete yc; yc = tyc; }
~myClass() { delete yc; yc = NULL; }
void setMyClass (yourClass * tyc) { delete yc; yc = tyc; }
void print () { yc->print(); }
};
int main()
{
yourClass b (//parameter);
myClass * a = new myClass();
a->print();
a->setMyClass(&b)
a->print();
delete a;
return 0;
}
a 的 print() 应该产生两种不同的打印,这取决于//参数。
我考虑过你的类(class);而不是 yourClass* yc,但我想知道这是否可能。
编辑: 我按以下方式重新编写了代码并且它有效。看起来仍然很复杂,智能指针似乎很有前途,但我仍然没有应用“三法则”。 这是代码。谢谢大家。
class myClass
{
protected:
yourClass * yc;
bool dynamic;
public:
myClass() { dynamic = true; yc = new yourClass (); }
myClass (yourClass * tyc )
{
// dynamic init (like default)
if (tyc == NULL ) { dynamic = true; yc = new yourClass (); }
// static use of yc
else { dynamic = false; yc = tyc; }
}
// because only if dynamic is true, we need to erase
~blu () { if (dynamic) { delete yc; dynamic = false; } }
void setMyClass(yourClass* tyc)
{
// leaving unchanged if new-stuff is NULL or like old-stuff
if ( tyc == yc || tyc == NULL ) return;
else // treating dynamic and static differently
{
if (dynamic) // if flag is set, must be deleted
{
delete yc; yc = tyc; dynamic = false;
}
else // must not be deleted, dynamic is still false
{
yc = tyc;
}
}
}
void print () { yc->print(); }
};
最佳答案
那是因为你试图删除太多:
- 您正在第二个构造函数中删除一个未分配的对象(删除
delete yc;
) - 你正试图删除一个堆栈分配的对象,b。
delete a;
将尝试删除指向 b 的指针,b 是堆栈上的一个对象;会发生什么取决于您的操作系统(我希望出现异常/核心转储/其他)
编辑:我发现的另一个问题..
a->setMyClass(NULL)
我建议:
- this post在智能指针上
- this blog post在 RAII 上
- 任何解释堆栈与堆分配(静态与动态?)的 C/C++ 初级读物
关于C++ 默认构造函数,用新对象初始化指针,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/14646696/