一个线程不安全的Singeton方式

一个线程不安全的Singeton模式
    设计模式中的单件（Singleton）模式如果按传统的方法设计，是线程不安全的。
如下：

package DesignPattern.singleton;class Singleton{private static Singleton sample;private Singleton(){}public static Singleton getInstance(){if (sample == null){Thread.yield();sample = new Singleton();}return sample;}}public class MyThread extends Thread{public void run(){Singleton singleton = Singleton.getInstance();System.out.println(singleton.hashCode());}public static void main(String args[]){Thread threads[] = new Thread[5];for (int i=0 ; i < threads.length ;i++){threads[i] = new MyThread();}for (int i=0 ; i < threads.length ;i++){threads[i].start();}}}

运行结果：

17237886775438581871372805066417237886

上面的运行结果可能在不同的运行环境上有所有同，但一般这五行输出不会完全相同。从这个输出结果可以看出，通过getInstance方法得到的对象实例是五个，而不是我们期望的一个。这是因为当一个线程执行了Thread.yield()后，就将CPU资源交给了另外一个线程。由于在线程之间切换时并未执行到创建Singleton对象实例的语句，因此，这几个线程都通过了if判断，所以，就会产生了建立五个对象实例的情况（可能创建的是四个或三个对象实例，这取决于有多少个线程在创建Singleton对象之前通过了if判断，每次运行时可能结果会不一样）。

要想使上面的单件模式变成线程安全的，只要为getInstance加上synchronized关键字即可。代码如下：

public static synchronized Singleton getInstance() {   }

当然，还有更简单的方法，就是在定义Singleton变量时就建立Singleton对象，代码如下：

private static final Singleton sample = new Singleton(); 

然后在getInstance方法中直接将sample返回即可。这种方式虽然简单，但不知在getInstance方法中创建Singleton对象灵活。读者可以根据具体的需求选择使用不同的方法来实现单件模式。
摘自：http://developer.51cto.com/art/200906/132354.htm

查看更多 下一篇