代码重构时的一个想法

   最近，在对以前的代码进行重构时，发现了这么一个问题，对于一个单例，在一个方法中传入该类的名字，如何根据这个名字获取该类的实例?
一般，都是根据类名.getSingletonInstance()就可以获取单例对象。

  先看下面代码：

 

package boke.test;
/**
 * 单列
 * 
 * @since jdk1.5及其以上
 * @author 毛正吉
 * @version 1.0
 * @date 2010.06.29
 * 
 */
class Singleton {
	
	private static final Singleton singleton = new Singleton();
	
	/**
	 * 私有构造
	 */
	private Singleton() {
		
	}
	
	/**
	 * 获得单例实例
	 * @return
	 */
	public static Singleton getInstance() {
		return singleton;
	}
	
	/**
	 * 其他方法
	 */
	public void doSomething() {
		System.out.println("doSomething...");
	}
	

}
------------------------
package boke.test;

import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;

/**
 * 测试例子
 * @author 毛正吉
 *
 */
public class Client {

	/**
	 * @param args
	 * @throws ClassNotFoundException 
	 * @throws IllegalAccessException 
	 * @throws InstantiationException 
	 * @throws NoSuchMethodException 
	 * @throws SecurityException 
	 * @throws InvocationTargetException 
	 * @throws IllegalArgumentException 
	 */
	public static void main(String[] args) throws ClassNotFoundException, InstantiationException, IllegalAccessException, SecurityException, NoSuchMethodException, IllegalArgumentException, InvocationTargetException {
		Class cls = Class.forName("boke.test.Singleton");
		System.out.println(cls.getName());
		Method method = cls.getDeclaredMethod("doSomething", new Class[]{});
		method.invoke(Singleton.getInstance(), null);
		method.invoke(cls.newInstance(), null); // 该方法有问题
	}

}

很容易发现注释部分由问题，不能访问单例类，因为构造方法为private修饰。

现在，有这样一个方法，我们暂不考虑这样做有没有意义，能否通过方法中传入的类名字符串来获取该类的唯一实例呢？或者不能实现？

public Singleton getSingletonInstance(String className) {
	...
	return 单例对象
}

仅此想法，欢迎大家讨论哈。

评论

13 楼 littleJava 2010-06-30  

spring的单例是通过ThreadLocal来保证线程的同步，普通的singleton有可能不提供这个功能

楼主能介绍下 你们代码重构的标准和规范吗？

12 楼 moonranger 2010-06-30  

risemanjavaeye 写道

1.把所有可能传进来的数做成单例模式，即构造哭私有化，一个 pubic static getInstance(){return this.instance;}.
2.在工厂方法里Class.forname(classname).getInstance()

interface singletonable<T>{
T getInstance(T t)
}

pubic static getInstance(){return this.instance;}
static方法你如何引用this？看你下面那个接口，这个static是多余的吧？

莫非是单例都要实现这个接口？那就很有意思了，没有实例的前提下，你如何调用这个getInstance？

Class.forname(classname).getInstance()，这句话也有问题吧？Class.forName(className)得到的是一个Class对象，上面哪来的getInstance()方法？

11 楼 moonranger 2010-06-30  

越来越不倾向于使用这样的单例，没法mock，不利于Unit test。个人认为Spring等DI容器管理的单例比较合适。

10 楼 maozj 2010-06-30  

risemanjavaeye 写道

1.把所有可能传进来的数做成单例模式，即构造哭私有化，一个 pubic static getInstance(){return this.instance;}.
2.在工厂方法里Class.forname(classname).getInstance()

interface singletonable<T>{
T getInstance(T t)
}

----------------------
泛型 这个想法不错

9 楼 maozj 2010-06-30  

huzhenyu 写道

完全没必要这样用，非常规！

仅此想法 非常规好事

8 楼 huzhenyu 2010-06-30  

完全没必要这样用，非常规！

7 楼 uran 2010-06-29  

有必要这么做吗？或许你该更深入的了解单例模式

6 楼 risemanjavaeye 2010-06-29  

1.把所有可能传进来的数做成单例模式，即构造哭私有化，一个 pubic static getInstance(){return this.instance;}.
2.在工厂方法里Class.forname(classname).getInstance()

interface singletonable<T>{
T getInstance(T t)
}

5 楼 lydawen 2010-06-29  

抛出异常的爱 写道

little_shieh 写道

Spring楼主没有用过吗，

spring的singleton与java的singleton不是一样东西.

如果在配置中指定初始化方法为getInstance应该就一样了。
否则只是容器中单例，实际jvm中还可以存在其它实例

4 楼 抛出异常的爱 2010-06-29  

little_shieh 写道

Spring楼主没有用过吗，

spring的singleton与java的singleton不是一样东西.

3 楼 little_shieh 2010-06-29  

Spring楼主没有用过吗，

2 楼 exloong 2010-06-29  

看这样行不行，覆盖clone方法，用它返回一个单例，然后用

return Class.forName("类名").clone();

1 楼 deadcode 2010-06-29  

Method mGetInstance = cls.getDeclaredMethod("getInstance", new Class[]{});
mGetInstance.invoke(null, new Object[]{});

没编译