工厂模式

[koalapheonix]

为什么大家写工厂模式的时候都是这样

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

class Circle : public Shape{
public:
       void Draw() { std::cout << "Drawing a Circle......Done" << std::endl; }
};

class Rectangle : public Shape {
public:
       void Draw() { std::cout << "Drawing a Rectangle... Done" << std::endl; }
};

Shape* ShapeFactory::instance( string name ) 
{
   if( name == "circle")
         return new Circle();
   else if( name == "rectangle" )
         return new Rectangle();
}

为什么不在ShapeFactory里定义一些常量

class ShapeFactory
{
///*********
public:
const static int CIRCLE = 1;
const static int RECTANGLE = 2;
}

Shape* ShapeFactory::instance( int name )
{
if( name == CIRCLE )
return new Circle();
else if( name == RECTANGLE )
return new Rectangle();
}

这样不更好吗？调用这个类的人就不可能写错

[vc1000]

[Quote=引用 10 楼 chain2012 的回复:]
LZ的问题可以说是风格问题。与设计无关
[/Quote]
同意

[zhaolinger2]

效果是一样的啊。
你的Factory怎么实现外面就怎么用呗。

[luciferisnotsatan]

用数字没有用字符串清楚。
如果用#define CIRCLE 1，那和直接写字符串差不多。

[koalapheonix]

不是呀，我调用的时候我可以写 ShapeFactory::instance(ShapeFactory::CIRCLE)这样不可能写错呀

[shenchenman]

什么叫工厂模式，楼主再去了解下吧。大工程里继承关系多的可能20多层，一个变量出问题，太乱了怎么找？都从工厂里出的零件，坏了，就找工厂

[war10811]

to 4l:
你吓唬谁啊
字符串有字符串的好处，常量有常量的好处

[koalapheonix]

说真的，继承20层，我通常会认为这个设计者是不是设计的有问题呀。

[xiaoboalex]

涉及到工厂的设计模式有三种：简单工厂，工厂方法和抽象工厂。
LZ的代码是简单工厂；简单工厂提供了字符串和具体产品对象之间的映射，将选择具体对象的代码封装起来。
所以不论是使用字符串还是常量，都和简单工厂本身无关，只是涉及到不同的编程习惯而已。

[luciferisnotsatan]

[Quote=引用 3 楼 koalapheonix 的回复:]

不是呀，我调用的时候我可以写 ShapeFactory::instance(ShapeFactory::CIRCLE)这样不可能写错呀
[/Quote]

所以说这么写就没什么区别，你也可以把ShapeFactory::CIRCLE定位字符串。

[controstr]

LZ所说的二种情况，第一种是判断字符串，第二种是使用整型

我认为使用整型比使用字符串好，何解？？
主要是运行效率问题，因为使用字符串比较比使用整型比较速度慢，所以整型比较从软件性能上考虑会比较好，但像楼主这种方式建议定义成enum，因为你定义const int aa = 1;这样的型式，往往变量值会有重复

[chainyu]

LZ的问题可以说是风格问题。与设计无关

[dooX8086]

如果在ShapeFactory添加常量成员这种方式，还不如将其独立成 ShapeType 类型
这样就可以使用ShapeFactory::instance(ShapeType::CIRCLE) .
工厂归工厂，类型归类型〜更加易于管理。

///////
#10 所说 风格问题与设计无关