Question

以下代码显然很危险。问题是：你如何跟踪* this的引用？

using namespace boost;

// MyClass Definition
class MyClass {

public:
   shared_ptr< OtherClass > createOtherClass() {
      return shared_ptr< OtherClass > OtherClass( this ); // baaad
   }
   MyClass();
   ~MyClass();
};

// OtherClass Definition
class OtherClass {

public:
   OtherClass( const *MyClass myClass );
   ~OtherClass();
};

// Call; pMyClass refcount = 1
shared_ptr< MyClass > pMyClass( new MyClass() );

// Call; pMyClass refcount = 1 => dangerous
pMyClass->createOtherClass();

我有答案（在下面发布），我只想让它在stackoverflow上（如果我错了，每个人都可以纠正我。）

Answer 1

关键是扩展enable_shared_from_this<T>并使用shared_from_this()方法将shared_ptr设为*this

For detailed information

using namespace boost;    

// MyClass Definition
class MyClass : public enable_shared_from_this< MyClass > {

public:
   shared_ptr< OtherClass> createOtherClass() {
      return shared_ptr< OtherClass > OtherClass( shared_from_this() );
   }
   MyClass();
   ~MyClass();
};

// OtherClass Definition
class OtherClass {

public:
   OtherClass( shared_ptr< const MyClass > myClass );
   ~OtherClass();
};

// Call; pMyClass refcount = 1
shared_ptr< MyClass > pMyClass( new MyClass() );

// Call; pMyClass refcount = 2
pMyClass->createOtherClass();

Answer 2

有几个问题：
你的代码不能编译!!

您的代码的设计不能阻止滥用/错误使用：

MyClass            x;
SharedPtr<MyClass> y = x.createOtherClass();

现在怎么样？这似乎是一个非常好的用例！

你应该如何使用shared_ptr返回* this？

2 个答案: