4

我有一个复杂的类，我想强制用户为其使用 std::unique_ptr，我认为最好的方法是创建一个返回 std::unique_ptr 的工厂方法并隐藏 ctor。

问题是，我添加的工厂方法越多，标题和代码开始看起来就越脏。

所以我想也许如果我创建一个包含所有工厂方法的新 ClassFactory 类，它看起来会更干净，每次我想添加一个新的工厂方法时，我只需编辑工厂类并完成它（保留原来的清洁类）。

例如：

class Widget
{
private:
    friend class WidgetFactory;

    Widget(){}
    Widget(const Widget& other) = delete;
    Widget& operator = (const Widget& other) = delete;

    std::string _msg;

public:
    void printMessage(){ std::cout << _msg << std::endl; }
};

class WidgetFactory
{
public:
    static std::unique_ptr<Widget> create(){ auto w = std::unique_ptr<Widget>(new Widget);
                                             w->_msg = "hello world!"; return w; }
};

然后是用户：

int main(int argc, char** argv)
{
    auto w = WidgetFactory::create();
    w->printMessage();
}

另一种方法是使用私有令牌使 ctor 显式公开，以便我可以使用 std::make_shared/unique。

但我不记得以前见过这种模式，所以我的问题是，这是一种好的做法吗？你介意从你的同事那里看到这样的代码吗？

提前致谢。

4

1 回答 1

6

不，这不是一个好习惯：

  1. 您将无法为单元测试创​​建此类对象 - 您必须使用工厂
  2. 使用std::unique_ptrfor everything 让人想起“当你有一把新锤子时，一切看起来都像钉子”这句话。很好用，但不能解决所有问题
  3. 代码auto看起来不错，但你不能用它来创建成员变量
  4. 对于大型（或至少是中型）项目，使用一个工厂处理所有事情会变得非常麻烦，并且会破坏它的目的

工厂模式很好（我个人喜欢它），但我不认为实际上每个对象都需要使用它来创建。

于 2013-10-08T18:50:55.227 回答