我正在尝试创建两个类,它们的实例被一起创建和删除。一个类是另一个类的基础:
class Interface;
class MyClass
{
friend class Interface;
private:
MyClass() {}
public:
static MyClass *NewInstance();
Interface *ControlPanel;
};
class Interface : public MyClass
{
friend class MyClass;
private:
Interface() {}
public:
void Control1() {cout << "control1" << endl;}
void Control2() {cout << "control2" << endl;}
void Control3() {cout << "control3" << endl;}
};应该创建和删除实例的两个成员函数是:
MyClass *MyClass::NewInstance()
{
MyClass *inst = new MyClass;
inst->ControlPanel = new Interface;
return inst;
}
void DeleteMyClassInstance(MyClass *inst)
{
delete inst->ControlPanel;
inst->ControlPanel = 0;
delete inst;
inst = 0;
}我成功地将实例创建过程与创建实例的基类(NewInstance())中的函数的使用联系起来。但是删除函数(DeleteMyClassInstance())不起作用(也就是说,在调用该函数后,我仍然可以同时使用inst1和inst1->ControlPanel ):
int main()
{
MyClass *inst1 = MyClass::NewInstance();
inst1->ControlPanel->Control1();
DeleteMyClassInstance(inst1);
inst1->ControlPanel->Control1();
return 0;
}但是如果我把删除代码放在main函数中,它就能很好地工作( delete语句后面的inst1->ControlPanel->Control1()语句不能工作,这就是我想要的):
int main()
{
MyClass *inst1 = MyClass::NewInstance();
inst1->ControlPanel->Control1();
delete inst->ControlPanel;
inst->ControlPanel = 0;
delete inst;
inst = 0;
inst1->ControlPanel->Control1();
return 0;
}我的问题是:为什么将delete语句直接放在main函数中是可行的,而将它们放在一个单独的函数中并在main中使用它就不行了?为什么我的DeleteMyClassInstance()函数中的代码被编译器忽略?
发布于 2013-03-25 17:50:53
将DeleteMyClassInstance函数更改为。
void DeleteMyClassInstance(MyClass **inst)
{
delete (*inst)->ControlPanel;
(*inst)->ControlPanel = 0;
delete (*inst);
*inst = 0;
}发布于 2013-03-25 17:51:56
主要区别在于,对于main函数中的代码,inst=0将main函数中的变量设置为null。对于DeleteMyInstance中的代码,inst=0行仅将DeleteMyInstance中的局部变量设置为null (没有用处,因为它在该点之后就不用了--启用更多警告,您的编译器可能会提到它)。它不会影响main中完全独立的同名变量。
所以,你的代码
DeleteMyClassInstance(inst1);
inst1->ControlPanel->Control1();具有未定义的行为,因为您尝试使用已删除的对象。UB意味着任何事情都可能发生。如果它看起来是有效的,这可能是因为在您的实现中,函数Control1即使在null (或者无效的)指针上被调用时仍然“有效”,因为该函数没有使用this或任何数据成员。但不应该依赖于实现细节。
注意(如果你还没有意识到)你的代码显示了一些糟糕的C++风格。你不应该写特殊的函数来删除你的类拥有的对象,这就是析构函数的作用。而且你不应该显式地删除析构函数中的对象,这就是智能指针的作用。你不应该在不需要的时候使用动态分配,这就是自动变量和数据成员的用途。一定要让这段代码正确一次,作为学习幕后发生的事情的练习,但这应该是为了尽快正确地完成它。
https://stackoverflow.com/questions/15611780
复制相似问题