应该“删除这个”从成员方法中调用？

Question

我刚刚读了这篇文章 并希望得到大家的建议：

问：应该从成员方法中调用 delete this; 吗？

Answer 1

通常这是一个坏主意，但有时很有用。

只要删除后不使用任何成员变量，并且只要调用此方法的客户端了解它可能会删除该对象，那么它就是完全安全的。

一个很好的例子说明这何时有用，如果你的类使用引用计数：

void Ref() {
  m_References++;
}

void Deref() {
  m_References--;
  if (m_References == 0) {
    delete this;
  }
}

Answer 2

我认为这里确实有两个问题

可以从成员方法中有效调用删除它吗？

是的。只要您非常小心使用，这是合法的。

应该在成员方法中使用删除它吗？

在非常特殊的情况下这是必要的。例如，某些类型的智能指针使用delete this模式来终止指针。示例：CComPtr<> 样式。

不过，除了智能指针之外，应该避免使用它，除非您有充分的理由这样做。即便如此，我也会仔细地重新考虑我的场景，看看是否有解决办法。

Answer 3

是的，您可以，这里很好地解释了何时以及为何

Answer 4

是的，有一些情况是常见的。

引用计数：

void release() 
{
  cnt--;
  if (cnt == 0) 
    delete this;
}

GUI 编程。在某些框架中，当用户关闭窗口时，窗口通常会删除自身。

Answer 5

为反对票做好准备。

应该：不。
技术上可以：可以
这是个好主意：绝对不是。
在什么情况下有用：当然。如果你是 C++，foo 就非常强大。但大多数人都没有那么好。因此，只有当您有一个与您一起工作的团队能够进行良好的代码审查时，才可以这样做。

原因：
对象无法知道它已被动态分配（因此需要删除）还是普通对象（因此不得删除），因此它如何决定是否应该删除它。因此，如果一个对象正在删除自身，那么在我看来，设计就存在严重错误。

如果您有一个需要管理的对象，那么您应该编写一个单独的对象来进行管理（因此是智能指针）。让对象做它擅长的事情，然后将对象的管理分离到另一个对象中。

Answer 6

这样做并非没有充分的理由。

问题是，当您在成员函数中调用 delete this 时，您会产生令人讨厌的副作用 - 调用者仍然拥有对您的实例的引用，但现在该引用完全无效。

这可能不是预期的行为，因此很容易导致严重的错误。

话虽如此，有时这是合适的（我见过一些内存管理方案，使用某些库，您可以在类中显式创建删除自身的方法 - 主要是为了语言互操作性）。但总的来说，我认为这是不好的做法。

Answer 7

一些线程库在线程终止时实现自动销毁时使用它。

void Thread::threadFunc()
{
    doRun();

    if(this->destroyOnExit == true)
        delete this;
}

Answer 8

这在 MFC 时代经常被使用。 IIRC 窗口收到的最后一条消息是 WM_NCDESTROY，此时您可以调用 delete this，当然假设您是某种形式的虐待狂（尽管 MFC 本身有时会这样做）我认为。）

Answer 9

是的。就像所有答案所说的那样，如果您 100% 确定在调用 delete this 后不会使用该类的数据。

例如，在一个项目中：

void Test()
MyClass * Someclass = new MyClass;
SomeClass->DoYourThing();
SomeClass->KillYourself();
return;

void MyClass::DoYourThing() { return; }
void MyClass::KillYourself() {delete this;}

非常简单的解释，该项目使用 delete this; 作为该类型对象的内存管理的一部分；它们的构造函数将它们添加到正在使用的该类型的类的私有列表中，并在它们被销毁时将其自身从该列表中删除，然后将其自身删除（这不在析构函数中）。当程序到达其端点时，该类中任何未删除自身的对象都会从静态成员函数 CleanYourselves() 中调用其 KillYourself() 等效项

Answer 10

虽然与该线程没有直接关系，但我想澄清这一点。
有人问我一个针对特定情况的问题：

int* a = new int ;
int* b = a ;
delete a;

现在下一个语句安全吗？

cout<<*b ;

我的答案：
删除 a 后，a 指向的位置已被标记为删除，并且在任何时间点都可以将其分配给其他对象。因此，使用 b 访问该值并不安全，因为它在分配给其他对象后可能会被修改。

注意：请不要投反对票，这只是一个澄清

Answer 11

你可以做到这一点，只要它是成员函数中的最后一个元素，并且在返回后你会忘记该对象曾经存在过......但是，是的，就像那篇文章所问的......你为什么要这样做？我不知道标准说什么，但它确实给了我一种有趣的感觉：P

我想这有点像你是否应该使用 GOTO 语句，我个人有时会使用 GOTO 来清理 C 中的资源，尤其是在特殊的条件。

我想知道共享状态的含义是什么（我知道的模糊陈述）：P

Answer 12

1. 不能从非成员函数调用 delete this :)
2. 在您了解其后果之前，这样做是一个坏主意。