0

我正在尝试创建两个类,它们的实例一起创建和删除。一个类是另一个类的基础:

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()inst1inst1->ControlPanel

int main()
{
  MyClass *inst1 = MyClass::NewInstance();

  inst1->ControlPanel->Control1();

  DeleteMyClassInstance(inst1);

  inst1->ControlPanel->Control1();

  return 0;
}

但是如果我将删除代码放在主函数中,它可以完美运行(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()函数中的代码会被编译器忽略?

4

2 回答 2

2

主要区别在于函数中的代码将main函数inst=0中的变量设置main为null。使用 in 中的代码DeleteMyInstance,该行inst=0仅将局部变量 in 设置DeleteMyInstance为 null (无用,因为在那之后它未使用 - 启用更多警告并且您的编译器可能会提到它)。它不会影响main.

所以,你的代码

DeleteMyClassInstance(inst1);
inst1->ControlPanel->Control1();

具有未定义的行为,因为您尝试使用已被删除的对象。UB 意味着任何事情都可能发生。如果它看起来有效,那可能是因为在您的实现中,Control1即使在空(或其他无效)指针上调用该函数仍然“有效”,因为该函数不使用this或任何数据成员。但不应依赖该实施细节。

请注意(如果您还没有的话)您的代码显示了一些糟糕的 C++ 风格。您不应该编写特殊函数来删除您的类拥有的对象,这就是析构函数的用途。而且您不必在析构函数中显式删除对象,这就是智能指针的用途。而且您不应该在不需要时使用动态分配,这就是自动变量和数据成员的用途。一定要把这段代码弄好一次,作为一个学习练习,幕后发生的事情,但这应该是为了尽快正确地完成它。

于 2013-03-25T09:51:55.997 回答
1

将您的 DeleteMyClassInstance 函数更改为。

void DeleteMyClassInstance(MyClass **inst)
{
  delete (*inst)->ControlPanel;
  (*inst)->ControlPanel = 0;
  delete (*inst);
  *inst = 0;
}
于 2013-03-25T09:50:53.007 回答