当前位置: 代码迷 >> Java相关 >> 这样写抽象工厂对不对呀,该怎么解决
  详细解决方案

这样写抽象工厂对不对呀,该怎么解决

热度:10088   发布时间:2013-02-25 21:51:02.0
这样写抽象工厂对不对呀
小弟想完成下面这个面试题,想用工厂模式来完成,但是不知道写的对不对。
请大家帮忙给看看这样写行不呀。

如果要设计一个图形系统,请你设计基本的图形元件(Point,Line,Rectangle,Triangle)的简单实现

#####################################
抽象工厂类
Java code
public abstract class ShapFactory {    private static ShapFactory sf = null;    private static Object mutex = new Object();    public  static ShapFactory getInstance(){        if(sf==null){                        synchronized (mutex) {                 if(sf==null){                    try {                        return  new ShapFactoryImp();                    } catch (Exception e) {                        e.printStackTrace();                        return null;                    }                }            }        }        return sf;    }    public abstract Rectangle createrectangle();    public abstract Square createSquare();}


########################
抽象工厂实现类
public class ShapFactoryImp extends ShapFactory {

public Rectangle createrectangle() {
// TODO 自动生成方法存根
return new Rectangle();
}

public Square createSquare() {
// TODO 自动生成方法存根
return new Square();
}

}
[/code]



####################
各种形状的父抽象类
Java code
 abstract class Shap {    int lenght;    int weight;    int hight;    abstract void draw();}



########################
矩形类
Java code
public class Rectangle extends Shap {    @Override    void draw() {        System.out.println("I'm Rectangle");    }}


###############
方形类
Java code
public class Square extends Shap{    void draw() {        System.out.println("I'm Square");    }}


------解决方案--------------------------------------------------------
写在那里可以
不过我认为synchronized 关键字加在函数上,仁者见仁,智者见智
------解决方案--------------------------------------------------------
首先你这应该是 Factory Method 而不是 Abstract Factory, Abstract Factory 的定义是
“Provider an interface for creating families of related or dependent objects without specify their concrete classes”
  
对于 “单子” (singleton一般都翻译为单件), 当然是对具体的工厂来实现的
  
再者,单件并不是为了节省内存。使用单件可以保持对象创建的一致性。
------解决方案--------------------------------------------------------
另外,你这里提到了Rectangle、Square,并在工厂里定义了两个方法,分别返回 Rectangle、Square
而你在下面已经明确定义Rectangle、Square都集成了Shape抽象类,那么就不用返回具体类了,例如:
public abstract Shape createrectangle();
public abstract Shape createSquare();
更进一步说,如果以后还有其他的子类的话,比如平行四边形,梯形等等,那么你还得在工厂里加入新的方法
这样莫不如,传递一个字符串进去例如:
public abstract Shape createShape(String shape);
实现中可以这么写:
if(shapeName.equals("Rectangle")) return new Rectangle();
else if(shapeName.equals("Square")) return new Square();
else throw new Exception("创建Shape子类"+shapeName+"不成功");


------解决方案--------------------------------------------------------
很不错,,,,
不过,看看我写的怎么样?
很多人都关闭掉构造方法。都把构造方法public出来了。因此,外部对象依然可以创佳一个工厂对戏那个。因此,你使用的sigton并不算是完全使用。

public abstract class ShapFactory {
private static ShapFactory sf = null;
private static Object mutex = new Object();
public static ShapFactory getInstance(){
if(sf==null){

synchronized (mutex) { 
  相关解决方案