Question

这可能是一个“最佳实践”问题，但我想确保以正确的方式解决这个问题。

我有以下课程：

typedef boost::shared_ptr<MyClass> MyClassPtr;

class MyClass final : public boost::enable_shared_from_this<MyClass> {
public:
    /* ctors/dtor ommitted */

    MyClassPtr method1() {
         // does something
         return shared_from_this();
    };

    MyClassPtr method2() {
         // does something
         return shared_from_this();
    };

}; // eo class MyClass

这是因为我想轻松地将呼叫链接在一起：

MyClassPtr ptr(myClassFactory.createMyClass());

ptr->method1()->method2()->methodX();  // etc...

这适用于shared_from_this()成语吗？是否有任何我不知道的陷阱，或者有更好的方法来做到这一点吗？

Answer 1

函数调用链通常通过返回对象的引用来完成：

MyClass& method1() {
     // does something
     return *this;
};

使用shared_ptr表示您希望MyClass仅使用shared_ptr将其包含在MyClass中。实际上，如果有人创建了具有自动存储持续时间的shared_ptr并调用了您的某个方法，那么您将遇到问题，因为当返回的delete被销毁时，它会尝试{{1你的对象。当然，您不能delete具有自动存储持续时间的对象。

我通常认为enable_shared_from_this只应在您的类提供静态工厂函数以生成shared_ptr并且其构造函数是私有的时才使用。这可以防止任何人创建具有自动存储持续时间的此类对象。

尽管如此，如果他们要进行功能链接，大多数人都会期望对该对象的引用。那么至少他们可以做到这一点：

MyClass foo;
foo.method1().method2();

而不是：

MyClass foo;
foo.method()->method2();

链接与共享指针一起调用

1 个答案: