一段单例模式的代码

class ConnectionPoolManager {

    private static final Object LOCK = new Object();

    private static ConnectionPoolManager connectionPoolManager = null;

    public static ConnectionPoolManager getInstance() {
        //这里有两次判断，我理解为减少不必要的同步开销
        if (connectionPoolManager == null) {
            synchronized (LOCK) {
                if (connectionPoolManager == null) {
                    connectionPoolManager = new ConnectionPoolManager();
                }
            }
        }
        return connectionPoolManager;
    }
    
    private ConnectionPoolManager(){
        super();
    }

}

感谢一下下面这个哥们的提醒。

评论

4 楼 w8y56f 2011-10-14  

我觉得第一个if判断是为了提高效率，如果ConnectionPoolManager 是null了才会进入条件，否则直接return了，可以越过同步代码块。第二个if条件判断确实是像上面的老兄说的那样，是为了安全，否则可能实际上并不止new了一个ConnectionPoolManager ，当然new了这么多个ConnectionPoolManager，但是只有一个不会被GC回收，就是被变量connectionPoolManager 引用的这个。


另外这个方法确实是比把synchronized直接加在方法上效率要高

3 楼 zhaixf2001 2009-09-09  

walsh 加一个私有的构造方法确实是必须的，防止通过new 的方式直接生成一个实例。

关于第一点，里面的if判断肯定是必须的，而同步lock外面的那次判断是在不加的情况下也能达到单例的效果，不过开销太大。

2 楼 walsh 2009-09-06  

还有，你这段代码，根本就不是单例模式。

你这样写，并不能保证在应用中只有一个实例，因为你用的是默认构造方法。

应该在你的类中加上一个private的构造方法，那样才能算得上是单例模式。

1 楼 walsh 2009-09-06  

引用

//这里有两次判断，我理解为减少不必要的同步开销  
        if (connectionPoolManager == null) {  
            synchronized (LOCK) {  
                if (connectionPoolManager == null) {  
                    connectionPoolManager = new ConnectionPoolManager();  
                }  
            }  
        } 

其实这里做两次判断并不仅仅是为了减少同步开销，是另有原因的，一个重要的原因就是在多线程中保证该实例唯一。

如果把synchronized代码块中的，那个if判断去掉，就会出现问题，就有可能在一个多线程应用中创建不止一个connectionPoolManager实例。


举个简单的例子：
如果线程Thread_1和Thread_2同时竞争lock锁，假如Thread_1获得了lock锁，那么Thread_2就要在synchronized (LOCK) 这行代码处等待，当Thread_1实例化对象之后，退出同步代码块，释放了lock锁，这个时候，Thread_2获得了lock锁，从而进入同步代码块，如果这个时候，没有那个if判断，对于thread_2来说，它还是认为connectionPoolManager为空（从第一个if可知），所以还会new一个connectionPoolManager对象。

这样就不是单实例了。