java与模式:登记式单例模式
单例父类:
import java.util.HashMap;
public class RegSingleton
{
static private HashMap m_registry = new HashMap();
static
{
RegSingleton x = new RegSingleton();
m_registry.put( x.getClass().getName() , x);
}
/**
- 保护的默认构造子 / protected RegSingleton() {} /*
- 静态工厂方法,返还此类惟一的实例 */ static public RegSingleton getInstance(String name) { if (name == null) { name = "com.javapatterns.singleton.demos.RegSingleton"; } if (m_registry.get(name) == null) { try { m_registry.put( name, Class.forName(name).newInstance() ) ; } catch(Exception e) { System.out.println("Error happened."); } } return (RegSingleton) (m_registry.get(name) ); } public void remove(Player player){ players.remove(Player); }
}
登记式单例类的子类:
import java.util.HashMap;
public class RegSingletonChild extends RegSingleton
{
public RegSingletonChild() {}
/**
- 静态工厂方法 */ static public RegSingletonChild getInstance() { return (RegSingletonChild) RegSingleton.getInstance( "com.javapatterns.singleton.demos.RegSingletonChild" ); } }
这样登记式单例类的子类可以用自己的getInstance() 方法产生实例而不在父类中登记
我的问题是:
单例本身就是想要一个对象,为什么父类中还保存了个HashMap呢?
而且子类不能在父类中登记那就不登记呗 为什么说这个是登记式单例类的一个缺点呢?
1.这个单例模式写得有点奇怪,设计者初衷是想做一个工厂方法,结果没做好,
最好是在基类中用模板方法,再用工厂方法
2.父类中的HashMap保存不同子类的实例的
3.子类不能 在父类中登记,那这个HashMap就是没有意义的
不能保存 不同子类的单例实例,也就是没有为其他子类提供一个唯一注入入口。
4.[code="java"]static private HashMap m_registry = new HashMap();[/code]
这行代码也写得不好:
[quote]private static HashMap m_registry = new HashMap();[/quote]
先写访问属性(private public protected) 在加上 final 或者 static
最后跟上对象的抽象类型(不是具体类型,接口或者抽象类的类型)
对于static / private 属于隐藏 ,相对于重载/覆盖,为什么这么说呢:
1
Parent{
static void a() {}
}
Child extends Parent{
}
Child c = new Child();
c.a();//可以简单这么理解:先从Child找a,没有就找父类(即父类的定义子类继承)
2
Parent{
static void a() {}
}
Child extends Parent{
static void a() {}
}
Child c = new Child();
c.a()//执行Child的,因为Child的a隐藏了父类的a
3
Parent{
static void a() {}
}
Child extends Parent{
static void a() {}
}
Parent c = new Child();
c.a()//执行Parent的,因为静态绑定 声明期绑定关系
因此你的例子是【1】 子类继承了父类的类定义 且没有隐藏
隐藏/重载 是静态绑定 覆盖是动态绑定
你说的这种实现 可参考我的博文
[url]http://jinnianshilongnian.iteye.com/blog/1415463[/url]