最近刚开始看《剑指offer》这本书,其中有讲到单例模式,单例模式可以说是设计模式中最简单的一种了,实现起来也非常简单(曾经的我就是这么想的):
/**
* 单例模式下考虑多线程并发的情况
*
* @author likebamboo
*/
public class Singleton {
private static Singleton mInstance = null;
private Singleton() {
}
public static Singleton getInstance() {
if (mInstance == null) {
mInstance = new Singleton();
}
return mInstance;
}
}
但是事实上我错了,书中指出了这样写的一个问题:“…这样写在单线程的时候工作正常,但是在多线程的情况下就有问题了。设想如果两个线程同时运行到判断mInstance是否为null的if语句,并且mInstance 的确没有创建时,那么两个线程都会创建一个实例,这时候就不再满足单例模式的要求了。…”
是的,这样写确实不能保证多线程情况下只获得一个mInstance实例,于是,我们加上一个线程锁:
/**
* 单例模式下考虑多线程并发的情况
*
* @author likebamboo
*/
public class Singleton {
// 加上同步锁
private static Object lock = new Object();
private static Singleton mInstance = null;
private Singleton() {
}
public static Singleton getInstance() {
synchronized (lock) {
if (mInstance == null) {
mInstance = new Singleton();
}
}
return mInstance;
}
}
上述写法可以避免多线程同时访问时创建多个Singleton实例的情况,但是作者指出了上述写法的一个不完美的地方,那就是以后每次通过getInstance方法获取Singleton实例的时候不管mInstance是否为空,都会先加上一个同步锁,而加锁的过程是一个比较耗时间的过程,我们应该尽量避免,于是我们可以再在外层加一个判断mInstance是否为空:
/**
* 单例模式下考虑多线程并发的情况
*
* @author likebamboo
*/
public class Singleton {
// 加上同步锁
private static Object lock = new Object();
private static Singleton mInstance = null;
private Singleton() {
}
public static Singleton getInstance() {
if (mInstance == null) {
synchronized (lock) {
if (mInstance == null) {
mInstance = new Singleton();
}
}
}
return mInstance;
}
}
这样代码虽然复杂了,效率却比第二种写法要高。
一个看似简单的问题事实上却并不简单。争取每天有一些进步。