V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
Totato5749
V2EX  ›  问与答

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

  •  
  •   Totato5749 · 2015-12-10 11:38:11 +08:00 · 2706 次点击
    这是一个创建于 3254 天前的主题,其中的信息可能已经有所发展或是发生改变。
    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;
    }

    15 条回复    2015-12-10 14:44:13 +08:00
    rails3
        1
    rails3  
       2015-12-10 11:46:33 +08:00
    最好的静态内部类
    zhaohui318
        2
    zhaohui318  
       2015-12-10 11:50:08 +08:00
    经典案例啊,你这个性能要差很多
    zts1993
        3
    zts1993  
       2015-12-10 11:51:29 +08:00
    第一种 如果不为空 不用加锁,,你这个,自己没看出问题么。。。。
    xufang
        4
    xufang  
       2015-12-10 12:01:16 +08:00
    neo2015
        5
    neo2015  
       2015-12-10 12:05:45 +08:00
    public synchronized static Singleton getInstance(){
    if(mInstance == null){
    mInstance = new Singleton();
    }
    return mInstance;
    }

    private static class SingletonInstance{

    }
    sun2920989
        6
    sun2920989  
       2015-12-10 12:07:20 +08:00
    你的写法理论上说没问题 但是并发访问的时候不如第一种效率高 单例的东西 永远是需要生成的时候少 已经存在的时候多
    neo2015
        7
    neo2015  
       2015-12-10 12:08:11 +08:00
    public synchronized static Singleton getInstance(){
    if(mInstance == null){
    mInstance = SingletonInstanc.mInstance
    }
    return mInstance;
    }

    private static class SingletonInstance{
    private static Signleton mInstance = new Singleton()
    }
    SparkMan
        8
    SparkMan  
       2015-12-10 12:43:34 +08:00
    现在更流行用 enum 类型,比 double check 还牛逼
    pixstone
        9
    pixstone  
       2015-12-10 12:52:32 +08:00
    public static synchronized Singleton getInstance(){ 取方法的时候上锁。。。。这效率要多低啊。。。合理的上锁模式是 实例化的时候上锁 防止重复实例。
    baozijun
        10
    baozijun  
       2015-12-10 12:54:10 +08:00
    你这个每次获取对象时都是加锁的,效率非常低.上面的那个有对象直接返回,没有才加锁,然后判断 此时对象的状态 是否为空,空的话创建
    zouxcs
        11
    zouxcs  
       2015-12-10 13:12:22 +08:00
    静态内部类整一个
    Totato5749
        12
    Totato5749  
    OP
       2015-12-10 13:39:11 +08:00
    谢谢诸君,学到了很多。
    ganxiyun
        13
    ganxiyun  
       2015-12-10 14:19:28 +08:00
    我觉得楼主贴的 DCL 好像 java 不适用吧,难道现在可以这么用了?
    ganxiyun
        14
    ganxiyun  
       2015-12-10 14:20:34 +08:00
    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
        15
    unique  
       2015-12-10 14:44:13 +08:00
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   3431 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 25ms · UTC 11:27 · PVG 19:27 · LAX 03:27 · JFK 06:27
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.