c++ - 这是一个好的代码吗(在阅读同事的代码时遇到的)

Question

文件a.hpp:

class a;
typedef boost::shared_ptr<a> aPtr

class a{
public:
    static aPtr CreateImp();
    virtual void Foo() = 0 ;
    ....
};

文件aImp.hpp:

class aImp : public a{

    virtual void Foo();

};

文件aImp.cpp:

aPtr a::CreateImp()
{
     return aPtr(new aImp());
}

void aImp::Foo(){}

客户端必须使用CreateImp 获取指向a 的指针，不能使用a 其他方式。您如何看待此实现？您如何看待这种实现方式？

Answer 1

如果 Factory Method 这看起来像一个正常的实现设计模式。 boost::shared_ptr 的返回只是让程序员在内存管理和异常安全方面更容易使用此 API，并防止简单的错误，例如调用函数和忽略返回值。

编辑:

如果这是基类的唯一实现，那么作者的目标可能是 pimpl idiom隐藏实现细节和/或减少编译时依赖性。