2

鉴于以下情况：

#include <iostream>
using namespace std;

class A
{
public:
    void func() {delete this;}
    A() : x(5) {cout << "ctor A" << endl;}
    ~A() {cout << "dtor A" << endl;}
    int x;
};

int main() {
    A a;
    cout << "The X value: " << a.x << endl;
    a.func();  // calling 'delete this' ->going to the dtor #1
    cout << "The X value: " << a.x << endl;
    return 0;  
}

输出是：

ctor A
The X value: 5
dtor A
The X value: 5
dtor A

有没有delete this;什么严重的影响？

4

5 回答 5

5

在这种情况下（您没有通过 new 分配 A 对象），您正在调用未定义的行为。未定义的行为通常是一件非常非常糟糕的事情。基本上，任何事情都可能发生。

可以在“删除这个；”的地方编写代码。不过会好的。但我不记得曾经这样做过。尽量避免它。尝试通过将此责任委托给其他对象（例如，智能指针）来避免完全手动调用 delete（无论您是否使用 new ）。

于 2011-08-28T10:19:39.897 回答
3

我将回顾一下您正在调用delete堆栈分配的对象，并检查delete this一般的用法。

有一种思想流派说“这不是一个好主意”。但是，我已经看到它在引用计数对象的实现中多次使用。

在COM中，框架要求所有对象都由工厂方法创建，然后通过调用“Release”再次释放。

  class MyRefCountedObject : public IUnknown
  {
      private:
           // Making the constructor and destructor private
           // so that the object can only be allocated as a pointer

           MyRefCountedObject() {}

           ~MyRefCountedObject() {}

           MyRefCountedObject(const MyRefCountedObject& mrco) {}

            int _refCount;
      public:
           static MyRefCountedObject* CreateInstance()
           {
                MyRefCountedObject* pObject = new MyRefCountedObject();
                pObject->_refCount = 1;
                return pObject;
           }

           void Release()
           {
                if(--_refCount == 0)
                {
                    delete this;
                }
           };
           void AddRef()
           {
               ++_refCount;
           }
  }

注意 - 这不是一个完整的实现，只是给出逻辑的想法。但是，通过将构造函数设为私有，我可以确保它只会在堆上分配。

于 2011-08-28T10:24:50.210 回答
3

运算符调用对象的delete析构函数然后释放对象使用的底层内存。内存必须由new操作员分配。在delete this中，删除的对象只是当前对象。构造没有什么特别之处，它只是普通的 C++。

在您的示例中，内存来自堆栈，因此您进入了未定义行为的领域（因为您在delete未通过运算符分配的对象上调用运算new符）。

调用它通常被认为是糟糕的设计，delete this因为一个对象不应该对它自己的生命周期负责。通常认为最佳实践是让对象的创建者对其销毁负责。

但是，有一种情况我个人觉得它非常有用。当您通过发送消息进行线程通信时，就会发生这种情况。在这种情况下，如果消息负责它自己的生命周期，那么它实际上比让原始线程负责它更安全、更容易编写。

于 2011-08-28T10:29:20.547 回答
2

C++ FAQ Lite: "Is it legal (and moral) for a member function to say 'delete this'?"

That said, you should rarely if ever have a use for this in properly organized code.

于 2011-08-28T10:38:56.817 回答
0

你不应该这样做。这是不好的编程习惯。使用new的对象应该是使用delete的对象。否则你会陷入混乱并最终导致内存泄漏等。

于 2011-08-28T10:19:14.167 回答