获取 C++ 中工厂类的用户输入

Question

在工厂设计中，当我使用工厂类创建新对象时，我会要求用户输入他/她的输入，然后在工厂类中从用户那里获取输入，然后使用这些输入创建对象。

在工厂类中获取用户输入是否可行？ 我应该如何在工厂类中获取用户输入？

工厂类在下面；

   Type *Factory::create_type(int Type){

switch(Type){

    case 1:{
        return new A(this->getUserTime(),this->getUserValue());
    }
    case 2:{
        float min = this->getUserMin();
        float max = this->getUserMax();

        if(this->validMinMax(min,max))
            return new B(this->getUserSpeed(),this >getUserValue(),min,max);
        else 
            return NULL;
    }
    case 3:{
        float min = this->getUserMin();
        float max = this->getUserMax();

        if(this->validUserMinMax(max,min))
            return new C(this->getUserSpeed(),this->getUserValue(),max,min);
        else 
            return NULL;
    }
    case 4:{
        return new D(this->getUserDistance(),this->getUserSpeed(),this->getUserValue());
    }
}}

工厂类的输入函数之一；

float Factory::getUserValue(){
float m;
std::cout<<"\n enter value:";
std::cin>>m
return m; }

Answer 1

简答：不，不要在工厂类中获取用户输入。

长答案：如果没有更多上下文（例如项目规模会很有趣），很难为单个课程的一部分提供建议。但是请考虑Single Responsibility Principle，这是SOLID 原则之一。

它告诉我们给一个模块/类/函数完全一个责任。因此，如果您有一个类是工厂（一项职责）并从用户那里读取输入（第二项职责），则违反了该原则。

相反，您应该考虑类似的事情

Type *Factory::create_type(int Type, const UserInput &input)
{
    switch (Type)
    {
        case 1:
            return new A(input.getTime(), input.getValue());
        case 2:
            float min = input.getMin();
            float max = input.getMax();

            /* and so on */

    }
}

还有一些

class UserInput
{
    public:
        Time getTime() const;
        float getMin() const;
        float getMax() const;

        /* and so on */
};

或者，工厂可以将UserInput 作为构造函数参数，因此它被这样调用

UserInput input;
// read input

auto Instance = Factory(input).create_type(3);

这样做您遵循另一个重要原则，Separation Of Concerns。

但是，我不能再告诉你什么是最适合你的情况，因为我不了解整个情况。你会发现;-)

Answer 2

正如其他人所建议的，如果需要，您的工厂可以采用输入参数来确定要创建哪个实例。例如，对于创建不同形状的简单形状工厂，您可以使用字符串/枚举来确定要创建的形状。但我强烈建议您的工厂按值向这些实例返回 Unique Ptr，这称为接收器，以便内存现在由请求者管理和拥有。

class Shape
{
public:
    virtual ~Shape() {}
    virtual void draw() = 0;
};

class Circle : public Shape
{
public:
    virtual void draw() override { // Implementation }
};

class Square : public Shape
{
public:
    virtual void draw() override { // Implementation }
};


class ShapeFactory
{
public:
    std::unique_ptr<Shape> createShape(const std::string& shapeType)
    {
        if(shapeType == "Circle")
            return std::make_unique<Circle>();
        else if(shapeType == "Square")
            return std::make_unique<Square>();
        else
            return nullptr;
    }
};