10

像这样实现单例模式的类的每个方法都需要使用 synchronize 关键字吗?

public class Singleton {

  private Singleton(){}

  public synchronized static Singleton getInstance()
    {   
        if(instance == null)
            instance = new Singleton ();

        return instance;
    }

  public void DoA(){
  }
}

由于 Singleton 不公开公共构造函数并且 getInstance() 方法是同步的,因此不需要同步方法 DoA 和 Singleton 类公开的任何其他公共方法。

这个推理正确吗?

4

4 回答 4

22

这就像任何其他课程一样。它可能需要也可能不需要进一步同步。

考虑以下示例:

public class Singleton {

  private Singleton() {}

  public synchronized static Singleton getInstance() { ... }

  private int counter = 0;

  public void addToCounter(int val) {
    counter += val;
  }
}

如果要从多个线程使用该类,addToCounter()则具有竞争条件。解决此问题的一种方法是addToCounter()同步:

  public synchronized void addToCounter(int val) {
    count += val;
  }

还有其他方法可以修复竞争条件,例如使用AtomicInteger

  private final AtomicInteger counter = new AtomicInteger(0);

  public void addToCounter(int val) {
    counter.addAndGet(val);
  }

在这里,我们在不使用synchronized.

于 2013-04-10T16:01:31.503 回答
10

好吧,Singleton 类的目的是它最多有一个实例,并且所有线程都可以访问同一个对象。

如果您不同步该getInstance方法,可能会发生以下情况

线程1进入getInstance()

线程2进入getInstance()

Thread1 计算instance == nulltrue

Thread2 评估instance == nulltrue

Thread1 赋值instance并返回

Thread2重新分配instance = new Singleton()并返回。

现在两个线程都有一个 Singleton 类的不同实例,这是这种模式应该阻止的。

同步可以防止两个线程同时访问同一个代码块。因此,当您实例化单例类时,在多线程环境中需要同步。

现在假设多个线程将尝试同时访问 Singletons 方法,这些方法也可能需要同步。特别是如果他们更改数据而不是只读取数据,这是真的。

于 2013-04-10T16:04:16.663 回答
1

使用单例的正确(实际上是最好的)方法

private static singleton getInstance() {
    if (minstance == null) {
        synchronized (singleton.class) {
            if (minstance == null) {
                minstance = new singleton();
            }
        }
    }
    return minstance;
}
于 2015-03-20T05:09:36.350 回答
0

延迟初始化和线程安全解决方案:

public class Singleton {

    public static class SingletonHolder {
        public static final Singleton HOLDER_INSTANCE = new Singleton();
    }

    public static Singleton getInstance() {
        return SingletonHolder.HOLDER_INSTANCE;
    }
}
于 2015-10-17T07:55:15.823 回答