synchronized关键字在运行时导致了空指针错误?

donet8

我写了一个很简单的Singleton,用synchronized关键字来做inst实例引用的访问保护:

1. 
   
2. public class Revert {
   
3.         public static void main(String[] args) {
   
4.                 Revert.Singleton.GetInstance();
   
5.         }
   
6.         static class Singleton{
   
7.                 private static Singleton inst=null;
   
8.                 public static Singleton GetInstance(){
   
9.                         if(inst==null){
   
10.                                 synchronized(inst){//JVM memory barrier
    
11.                                         inst=new Singleton();
    
12.                                 }
    
13.                         }
    
14.                         return inst;
    
15.                 }
    
16.                 Singleton(){System.out.println("ctor");}
    
17.         }
    
18. }
    

复制代码

问题是，我一致性就报空指针错误。如果我调试的话，发现调试器会在synchronized那句话那里停留两次，但是根本就不执行inst=new Singleton();这句话。

我的代码有什么问题? 还是我理解的就不对?
我用Eclipse+jdk1.6

isaacxu

也许是讲的不清楚，还是烦请阅读《Effective Java》2nd里，“Item 3: Enforce the singleton property with a private constructor or an enum type”的相关内容，原文：“As of release 1.5, there is a third approach to implementing singletons. Simply make an enum type with one element:”
Joshua Bloch给出的代码：

1. // Enum singleton - the preferred approach
   
2. public enum Elvis {
   
3. INSTANCE;
   
4. public void leaveTheBuilding() { ... }
   
5. }

复制代码

写了一个Enum Singleton的Junit测试用例，在Eclipse里选择File->New->Junit Test Case,可以生成相应的Junit测试文件，然后，Run as Junit Test 就可以了。

1. /**
   
2. * File:EnumSingleton.java
   
3. * ---------------------------
   
4. * for test Enum Singleton
   
5. */
   
6. package org.cudemo;
   
7. 
   
8. /**
   
9. * @author isaacxu
   
10. *
    
11. */
    
12. public enum EnumSingleton {
    
13. 
    
14.         INSTANCE;
    
15.        
    
16.         private String name;
    
17. 
    
18.         // Private constructor can prevents instant from other classes
    
19.         private EnumSingleton() {
    
20.         }
    
21.     //set a obj name
    
22.         public void setName(String name) {
    
23.                 this.name = name;
    
24.         }
    
25.     //get a obj name
    
26.         public String getName() {
    
27.                 return name;
    
28.         }
    
29. 
    
30. }
    
31. 
    

复制代码

Junit 测试用例

1. /**
   
2. * File:EnumSingletonTest.java
   
3. * ------------------------------
   
4. * This a junit test case for test EnumSingleton
   
5. */
   
6. package org.cudemo;
   
7. 
   
8. import static org.junit.Assert.*;
   
9. 
   
10. import org.junit.Test;
    
11. 
    
12. public class EnumSingletonTest {
    
13. 
    
14.         @Test
    
15.         public void test() {
    
16.                 EnumSingleton firstObj = EnumSingleton.INSTANCE;
    
17.             firstObj.setName("Enum");
    
18.             EnumSingleton secondObj = EnumSingleton.INSTANCE;
    
19.             secondObj.setName("Singleton");
    
20.             assertTrue(firstObj == secondObj);
    
21.             assertEquals("Singleton", firstObj.getName());
    
22.             assertEquals("Singleton", secondObj.getName());
    
23.             
    
24.         }
    
25. 
    
26. }
    

复制代码

isaacxu

1. /**
   
2. * File:MyRevert.java
   
3. * ----------------------
   
4. *         Please read Item 3 in the 2nd Edition of <Effective Java>,  Bloch explains three ways of implementing
   
5. * a singleton in Java, the best is “Enum as Singleton”.Joshua Bloch is aka Josh Bloch,you can see his
   
6. * write code in OpenJdk source code.(e.g.Collections.java)
   
7. */
   
8. package org.cudemo;
   
9. 
   
10. /**
    
11. * @author isaacxu
    
12. *
    
13. */
    
14. public class MyRevert {
    
15. 
    
16.         /**
    
17.          * @param args
    
18.          */
    
19.         public static void main(String[] args) {
    
20.                
    
21.                 MyRevert.Singleton.getInstance();
    
22.                 MyRevert.MyNewSingleton.gotIt();
    
23. 
    
24.         }
    
25.         /**
    
26.          * before Java1.5
    
27.          * @author isaacxu
    
28.          *
    
29.          */
    
30.         public static class Singleton {
    
31.                 private static Singleton uniqInstance;
    
32. 
    
33.                 private Singleton() {
    
34.                        
    
35.                 }
    
36. 
    
37.                 public static synchronized Singleton getInstance() {
    
38.                         if (uniqInstance == null) {
    
39.                                 uniqInstance = new Singleton();
    
40.                         }
    
41.                         System.out.println("Hi,I am old java Singleton");
    
42.                         return uniqInstance;
    
43.                 }
    
44.                
    
45.         }
    
46.         /**
    
47.          * after Java1.6
    
48.          */
    
49.         public static enum MyNewSingleton {
    
50.                 INSTANCE;
    
51.                 public static void gotIt(){
    
52.                         System.out.println("Hi,I am the best Singleton!");
    
53.                 }
    
54.        
    
55.         }
    
56. 
    
57. }
    

复制代码

donet8

isaacxu 发表于 2012-08-06 17:58

你是说,java1.6以后声明一个INSTANCE就相当于声明了一个Singleton类型? 这么神奇? 我百度了半天Java 1.6 INSTANCE没找到什么有用的信息啊。