近期看到到Struts1与Struts2的比較。说Struts1的控制器是单例的，线程不安全的；Struts2的多例的，不存在线程不安全的问题。之后又想到了之前自己用过的HttpHandler。。。

这些类。好像单例的线程安全问题确实是随处可见的。

可是仅仅是知道这个是不安全的，也没有认真分析过。接下来就细致分析下。

一，改动单例模式代码

首先我先写一段单例类的代码：

/*** @ClassName: Sigleton* @Description: 单例类* @author 水田* @date 2015年12月19日 上午10:12:55*/
public class Sigleton {private static Sigleton sigleton;private Sigleton() {// put the initMethod for this class};public static Sigleton getInstance() {// in this demo ,we use "Lazy-load Singleton"if (sigleton == null) {sigleton = new Sigleton();}return sigleton;}
}

这里我使用的是延迟载入，不管是使用延迟载入还是以下的饿汉式：

public class Sigleton {private static final Sigleton sigleton=new Sigleton();private Sigleton() {// put the initMethod for this class};public static Sigleton getInstance() {return sigleton;}
}

这两种情况使用哪一种。要依据实际情况来推断：究竟我是要在此类还没有使用之前进行初始化。还是要在用到它去拿它的时候才初始化。还要看你的实际应用场景。比方说，我这个类超级大，这时候，你部署好了之后。就把它New了。然后放在内存中，十年八年的没人用。这不是浪费么？（情况举的比較极端，就是这个意思吧。只是你要是硬要跟我说。内存啥的越来越不值钱。或者爷有的是钱买内存。我也没办法！

仅仅能送你句，怪不得你没有女朋友！

）

细致分析这两种方法，然后看看哪里存在线程不安全的因素。

先来瞅瞅第一种，lazy-load方式：

在调用getInstance的时候。先推断，是不是已经被New过了。假设没，那么我new,完了之后返回。想象下，多线程。当在一个线程内。运行到if (sigleton == null),还有一个线程内，好巧啊。也运行到这里。

然后两个线程同一时候推断发现还没这个东西，然后各自new一个。

破坏了我的单实例的原则。

相比另外一种直接new的方式。这样的方法显然是不安全的。可是，假设我要用到这样的lazy-load方式，就要对它进行改进了。

简单改进：

  public static synchronized Sigleton getInstance()

加个keyword。

可是这样的方法还是不好，产生问题的仅仅有sigleton = new Sigleton();如今我锁定了整个方法。有点儿多余了。再改下：

 public final Sigleton getInstance() {// in this demo ,we use "Lazy-load Singleton"if (sigleton == null) {synchronized (this) {sigleton = new Sigleton();}}return sigleton;}

然后改完了之后。我们再从逻辑上看下是不是有漏洞：

还是刚才的问题。俩线程，同一时候运行到if判空的时候，第一个线程由于调度原因，进入同步方法，运行了new操作，第二个线程判空完了之后，进不去，还等在同步方法外面。第一个线程出了同步方法，第二个线程进入了同步方法，又new了一个对象。。

。

。貌似确实有逻辑楼栋，再改下：

 public final Sigleton getInstance() {// in this demo ,we use "Lazy-load Singleton"if (sigleton == null) {synchronized (this) {if (sigleton == null){sigleton = new Sigleton();}}}return sigleton;}

当第二个线程进入同步方法之后，要不要新new一个对象，还要推断下。

二，After双重检查

三。怎样安全+单例使用

1。使用volatilekeyword

  private volatile static Sigleton sigleton;

public class Sigleton {private static class SigletonHolder {private static final Sigleton INSTANCE = new Sigleton();}private Sigleton() {};public static final Sigleton getInstance() {return SigletonHolder.INSTANCE;}
}