静态类和单例的抉择

先说说我要实现的要求：我要一个Map，这个Map是常量，但是它要进行初始化，填充数据。而且这个Map要可以重用，实现优雅点的

public class ApproveMap {

	private static Map<String, String> approveCodeMap = new HashMap<String, String>();

	/**
	 * @return
	 */
	public static Map<String, String> getApprove() {
		synchronized (approveCodeMap) {
			if (approveCodeMap.isEmpty())
				setApprove();
		}
		return approveCodeMap;
	}

	private static void setApprove() {
		/**
		 *
		 */
		approveCodeMap.put("101102", "11_0003_0001");
		approveCodeMap.put("101103", "11_0003_0004");
		approveCodeMap.put("101104", "11_0003_0005");
		approveCodeMap.put("101105", "11_0003_0002");
		approveCodeMap.put("101106", "11_0003_0003");
		approveCodeMap.put("101107", "11_0003_0006");

	}
}

以上是我写的一个方式，总觉得别扭，大家一块看看有什么更好的方法。

2010年6月更新：
这个东西看着确实不是东西，有静态初始化块了，没必要有getter了，实在是。。。

public class ApproveMap {

	public static final Map<String, String> approveCodeMap = new HashMap<String, String>();
	static {
		/**
		 *
		 */
		approveCodeMap.put("101102", "11_0003_0001");
		approveCodeMap.put("101103", "11_0003_0004");
		approveCodeMap.put("101104", "11_0003_0005");
		approveCodeMap.put("101105", "11_0003_0002");
		approveCodeMap.put("101106", "11_0003_0003");
		approveCodeMap.put("101107", "11_0003_0006");

	}
}

评论

20 楼 andot 2009-02-24  

icewubin 写道

andot 写道

icewubin 写道

andot 写道

静态初始化也要加锁，在多线程模式下，你不能保证在初始化之前不会被访问。

Java中的static代码块，不需要加锁的。

它是自动加锁的吗？

因为Java的类加载的特性，static代码块只会运行一次，所以不需要锁。

一般简单来说，当程序第一次调用到这个类的某个变量或方法（除了static final的变量），都算是首次加载这个类到内存中准备调用，但是调用之前，先初始化static的变量，然后执行static代码段（暂且认为static代码段在static变量定义之后），此时才算类加载完毕，再运行之前的调用请求。

当然后续再有其他调用请求，这个类加载的过程不会再执行一遍的。

如果是你说的这样的话，如果不加锁还是有可能会出问题的。

比如当第一次访问这个类的静态方法来获取HashMap中的值时，他会先执行static块，在执行static块的时候（没完成），如果另一个线程也同时调用这个类的静态方法来获取HashMap中的值时，就会有冲突了（比如还没有初始化完，得不到希望的值）。所以，就算在static块中初始化HashMap的元素，也还是加锁比较好。

19 楼 icewubin 2009-02-24  

chan.d 写道

前阵子碰到类似的问题，

这样处理的，

static{

    map = makeUnreachUrl(version);

}

    public static Map makeUnreachUrl(int version){
           .....
           return map;
}

因为初始化的时候需要根据version的值判断要加入map的对象,
所以是以上写法。

再说，在static块中直接写感觉有点不优雅的感觉。

Java本身就没动态语言优雅，将就吧，不然当你看到很多匿名内部类，更会感到恶心。

18 楼 icewubin 2009-02-24  

andot 写道

icewubin 写道

andot 写道

静态初始化也要加锁，在多线程模式下，你不能保证在初始化之前不会被访问。

Java中的static代码块，不需要加锁的。

它是自动加锁的吗？

因为Java的类加载的特性，static代码块只会运行一次，所以不需要锁。

一般简单来说，当程序第一次调用到这个类的某个变量或方法（除了static final的变量），都算是首次加载这个类到内存中准备调用，但是调用之前，先初始化static的变量，然后执行static代码段（暂且认为static代码段在static变量定义之后），此时才算类加载完毕，再运行之前的调用请求。

当然后续再有其他调用请求，这个类加载的过程不会再执行一遍的。

17 楼 chan.d 2009-02-24  

前阵子碰到类似的问题，

这样处理的，

static{

    map = makeUnreachUrl(version);

}

    public static Map makeUnreachUrl(int version){
           .....
           return map;
}

因为初始化的时候需要根据version的值判断要加入map的对象,
所以是以上写法。

再说，在static块中直接写感觉有点不优雅的感觉。

16 楼 andot 2009-02-24  

icewubin 写道

andot 写道

静态初始化也要加锁，在多线程模式下，你不能保证在初始化之前不会被访问。

Java中的static代码块，不需要加锁的。

它是自动加锁的吗？

15 楼 icewubin 2009-02-24  

andot 写道

静态初始化也要加锁，在多线程模式下，你不能保证在初始化之前不会被访问。

Java中的static代码块，不需要加锁的。

14 楼 whaosoft 2009-02-23  

没明白你要干嘛!!!!!!

13 楼 bachmozart 2009-02-23  

这个好像不需要抉择，static 块就完了，同步开销没必要时能省就省，我见过并发量大到一定程度时，竞争一个锁居然要花1秒

12 楼 yighter 2009-02-23  

类变量存在方法区，对象存在堆中，倾向于使用单例。

11 楼 case0079 2009-02-23  

这种需求下静态和单例有矛盾吗？
单例不过是不可以 NEW一个对象

10 楼 unsid 2009-02-23  

一个启动慢，一个第一次登陆慢

9 楼 yyjn12 2009-02-23  

在
# synchronized (approveCodeMap) { 
#             if (approveCodeMap.isEmpty()) 
#                 setApprove(); 
#         } 
的外边，再加一层  if(approveCodeMap.isEmpty())

双重检查锁，效率会比这个高很多。
这个不成了，每次取这个map都要排队了吗

8 楼 cdredfox 2009-02-23  

类似这样的需求，一至使用静态块

7 楼 forgood 2009-02-23  

同意楼上的观点，使用静态代码块，比较简洁，直接。

6 楼 5452 2009-02-20  

嗯，看了看，想了想，还是觉得大家说的静态代码块比较简洁，直接，我喜欢，谢谢

5 楼 taupo 2009-02-19  

andot 写道

静态初始化也要加锁，在多线程模式下，你不能保证在初始化之前不会被访问。

静态初始化需要枷锁吗？
静态变量是虚拟机加载类 的时候初始化的，如果第一次访问某个类的时候，该类没有被加载，虚拟机就会去加载该类
同时初始化静态变量

这和线程没有关系吧？

4 楼 andot 2009-02-19  

静态初始化也要加锁，在多线程模式下，你不能保证在初始化之前不会被访问。

3 楼 riching 2009-02-19  

dennis_zane 写道

直接初始化不就完了，非要搞个加锁

嗯，看网上说的单利模式的实现倾向于直接实现这种方式

2 楼 dennis_zane 2009-02-19  

直接初始化不就完了，非要搞个加锁

1 楼 dennis_zane 2009-02-19  

直接静态初始化不就完了，非要搞个加锁

  static{
     map.put....
  }