关于 Java 单例模式的 Synchronized 的用法

[Totato5749]

1. public static Singleton getInstance() {
   
2. if (instance == null) {
   
3. synchronized (Singleton.class) {
   
4. if (instance == null) {
   
5. instance = new Singleton();
   
6. }
   
7. }
   
8. }
   
9. return instance;
   
10. }
    

我发现网上单例模式中 getInstance 方法基本上都是这么写的,我觉得这种写法虽然正确但是一点都不易读。 请教各位，像我下面这样写正确吗？

public static synchronized Singleton getInstance(){
if(mInstance == null){
mInstance = new Singleton();
}
return mInstance;
}

[rails3]

最好的静态内部类

[zhaohui318]

经典案例啊，你这个性能要差很多

[zts1993]

第一种 如果不为空 不用加锁，，你这个，自己没看出问题么。。。。

[xufang]

http://coolshell.cn/articles/265.html

[neo2015]

public synchronized static Singleton getInstance(){
if(mInstance == null){
mInstance = new Singleton();
}
return mInstance;
}

private static class SingletonInstance{

}

[sun2920989]

你的写法理论上说没问题 但是并发访问的时候不如第一种效率高 单例的东西 永远是需要生成的时候少 已经存在的时候多

[neo2015]

public synchronized static Singleton getInstance(){
if(mInstance == null){
mInstance = SingletonInstanc.mInstance
}
return mInstance;
}

private static class SingletonInstance{
private static Signleton mInstance = new Singleton()
}

[SparkMan]

现在更流行用 enum 类型，比 double check 还牛逼

[pixstone]

public static synchronized Singleton getInstance(){ 取方法的时候上锁。。。。这效率要多低啊。。。合理的上锁模式是 实例化的时候上锁 防止重复实例。

[baozijun]

你这个每次获取对象时都是加锁的,效率非常低.上面的那个有对象直接返回,没有才加锁,然后判断 此时对象的状态 是否为空,空的话创建

[zouxcs]

静态内部类整一个

[Totato5749]

谢谢诸君，学到了很多。

[ganxiyun]

我觉得楼主贴的 DCL 好像 java 不适用吧，难道现在可以这么用了？

[ganxiyun]

DCL 不是用

// Works with acquire/release semantics for volatile
// Broken under Java 1.4 and earlier semantics for volatile
class Foo {
private volatile Helper helper;
public Helper getHelper() {
Helper result = helper;
if (result == null) {
synchronized(this) {
result = helper;
if (result == null) {
helper = result = new Helper();
}
}
}
return result;
}

// other functions and members...
}

[unique]

如何正确地写出单例模式
https://github.com/biezhi/jb/blob/master/designpatterns/singleton.md