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

正如其他人所建议的那样，如果需要，您的工厂可以使用输入参数来确定要创建的实例。例如，对于创建不同形状的简单Shape工厂，可以使用字符串/ Enum来确定要创建的形状。但我强烈建议您的工厂按值将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;
    }
};

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

2 个答案: