一个线程不安全的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();
		}

	}
}

运行结果：

17237886
7754385
8187137
28050664
17237886

上面的运行结果可能在不同的运行环境上有所有同，但一般这五行输出不会完全相同。从这个输出结果可以看出，通过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