使用工厂朋友类分配新类,好习惯

use factory friend class to allocate new class, good practice?

本文关键字:新类 好习惯 分配 工厂 朋友      更新时间:2023-10-16

我有一个复杂的类,我想强制用户使用它使用 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。

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

提前谢谢。

不,这不是一个好的做法:

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

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