Java单例模式在多线程中输出两个相同的对象？

Question

public class Foo {

    private static  Foo foo;
    private Foo(){}
    public static Foo getInstance(){
        if (foo==null){
            try {
                Thread.sleep(1);
            } catch (InterruptedException e) {}
            foo = new Foo();
            System.out.println(foo+"----"+Thread.currentThread().getName());
        }
        return foo;
    }

    public static void main(String[] args) throws Exception {

        Thread thread1 = new Thread(new Runnable() {
            @Override
            public void run() { 
                Foo foo1 = Foo.getInstance();
            }
        },"thread1");

        Thread thread2 = new Thread(new Runnable() {
            @Override
            public void run() {
                Foo foo2 = Foo.getInstance();
            }
        },"thread2");
    thread1.start();
    thread2.start();
    }
}

我只想模拟这些代码在多线程环境中是不安全的，但它总是输出两个相同的对象，如:

Foo@24c21495----thread1
Foo@24c21495----thread2

或

Foo@3d4b7453----thread1
Foo@3d4b7453----thread2

...

为什么？

Answer 1

我认为这里有两个综合原因:

• 打印的第一件事就是结果。各种代码可能正在此处进行 JIT 编译，并可能初始化其他资源。我相信这是一个有效的固有同步点。
• 你的 sleep 时间很短，而且很均匀，这无济于事。

基本上，我相信两个线程都在创建新实例，然后都在显示结果 - 除非你“幸运”，否则两个线程都会看到第二个执行分配的线程的结果。

如果你只是添加:

System.out.println("Starting");

在 main 的最开始，您有时会看到不同的输出。如果你也改变你的 sleep :

Thread.sleep((long) (Math.random() * 100));

...那么您就更有可能看到它。基本上，您希望一个线程在另一个线程处于 sleep 状态时完成整个“ sleep 、初始化、打印”。

另一种选择是简单地向构造函数添加打印语句:即使两个线程在您当前的诊断位置显示相同的对象，您仍然会看到正在构造两个实例，这表明代码是不安全的。