同步方法

Question

如果我有一个同步的公共方法和一个私有方法：

public synchronized void doSomething() {
    doSomethingElse()
}

private void doSomethingElse() {
}

我需要同步私有方法吗？

Answer 1

这取决于：

• 如果 doSomethingElse 可以安全地并发调用，那么您就不需要 synchronized。
• 如果不是，那么答案取决于调用它的位置：
  • 如果仅从其他synchronized方法调用它，则不需要synchronized（但将其标记为这样不会有什么坏处）；
  • 如果可以从本身不同步的方法调用它，那么它必须是同步。

Answer 2

这取决于你在做什么。您是否需要确保对 doSomethingElse() 的串行访问？

如果是这样，并且唯一调用 doSomethingElse() 的是 doSomething()，那么不，您不需要同步。但如果其他方法可以调用 doSomethingElse()，那么是的，您也应该同步它。

Answer 3

否：如果调用 doSomethingElse() 的唯一方法是通过另一个同步方法。

可能是：如果您以其他方式通过未同步的方法调用doSomethingElse()，则需要保护它免受并发访问。

Answer 4

这就是 @GuardedBy 注释的目的。如果您期望在调用该方法时必须持有锁，请用该锁和锁的名称对其进行注释（在示例中为：

@GuardedBy("this") private void doSomethingElse() {…}

然后您可以使用 FindBugs 检查不变量是否为 true 您还可以

使用其他net.jcip.annotations 用于描述线程安全性或缺乏线程安全性，并让 FindBugs 也验证这些假设，这本书也需要一个插头。

Answer 5

从同步方法（或从 synchronized 块内）调用的任何方法都会在同步时运行。如果仅从同步方法调用私有方法，则无需单独同步该私有方法。

Answer 6

如果同步一个代码块，那么从该代码块内（在同一线程上）调用的任何内容仍然保留初始锁。因此，当从 doSomething 调用时，doSomethingElse 仍然是同步块的一部分。

如果您这样做：

public synchronized void doSomething() {
  new Thread() {
    public void run() {
      doSomethingElse();
    }
  }.start();
}
private void doSomethingElse() {
}

那么 doSomethingElse 将不会拥有由 doSomething 获取的锁。

还要避免同步方法，因为它会公开并发策略的实现细节。请参阅有关同步（this）/同步方法的问题：在 Java 中避免同步（this）？

如果 doSomethingElse 必须同步，无论它是否是从 doSomething 调用的，同步 doSomethingElse 也没有坏处，因为同步锁是可重入（即，如果一个线程已经拥有一个对象的锁，它可以再次获得该锁）。

Answer 7

就我个人而言，我不喜欢同步方法。我更喜欢与某种锁变量同步，如下所示：

private final Object lock = new Object();

public void doSomething() {
  synchronized(lock) {
    // Do some safely
    doSomethingElse();
  }
  // Do some work un-safely
}

private void doSomethingElse() {
  // Do some work safely because still in lock above
  // ...
}

Answer 8

尽管您所做的很好，但在我看来，同步应该以尽可能低的粒度完成。这建议同步实际的私有函数。目前，您假设类中没有其他函数会独立调用私有函数。未来可能不会出现这种情况。

Answer 9

尽管在私有方法未同步时代码可以正常工作，但从可维护性的角度来看，使私有方法同步似乎是谨慎的做法。

内在锁是可重入的，在私有方法中添加synchronized关键字没有什么坏处。

将代码放入私有方法中会导致其他方法调用它，因此使私有方法同步是有意义的，以防将来从另一个不需要同步的方法调用它。