在lockObject上同步和使用this作为锁有什么区别？

Question

我知道同步方法和同步块之间的区别，但我不确定同步块部分。

假设我有这段代码

class Test {
  private int x=0;
  private Object lockObject = new Object();

  public void incBlock() {
    synchronized(lockObject) {
      x++;
    }
    System.out.println("x="+x);
  }

  public void incThis() {  // same as synchronized method
    synchronized(this) {
      x++;
    }
    System.out.println("x="+x);
  }
}

在这种情况下使用 lockObject 和使用 this 作为锁有什么区别？这对我来说似乎是一样的。

当你决定使用同步块时，你如何决定哪个对象是锁？

Answer 1

就我个人而言，我几乎从不锁定“这个”。我通常锁定一个私人持有的引用，我知道没有其他代码会锁定该引用。如果您锁定“this”，那么了解您的对象的任何其他代码都可能会选择锁定它。虽然这种情况不太可能发生，但它肯定会发生——并且可能导致死锁，或者只是过度锁定。

你锁定的东西并没有什么特别神奇的地方——你可以有效地将它视为一个令牌。任何使用相同令牌锁定的人都会尝试获取相同的锁。除非您希望其他代码能够获取相同的锁，否则请使用私有变量。我也鼓励您将变量final - 我不记得我曾经曾经想要换锁的情况对象生命周期内的变量。

Answer 2

当我阅读《Java 并发实践》时，我也遇到了同样的问题，我想我应该对 Jon Skeet 和 spullara 提供的答案添加一些补充观点。

下面是一些示例代码，当 doStuff(ValueHolder) 方法执行时，它们甚至会阻塞“快速”的 setValue(int)/getValue() 方法。

public class ValueHolder {
    private int value = 0;

    public synchronized void setValue(int v) {
        // Or could use a sychronized(this) block...
        this.value = 0;
    }

    public synchronized int getValue() {
        return this.value;
    }
}

public class MaliciousClass {

    public void doStuff(ValueHolder holder) {
        synchronized(holder) {
            // Do something "expensive" so setter/getter calls are blocked
        }
    }
}

使用 this 进行同步的缺点是其他类可以在对您的类的引用上进行同步（当然不是通过 this）。在锁定对象引用时恶意或无意地使用 synchronized 关键字可能会导致您的类在并发使用下表现不佳，因为外部类可以有效地阻止您的 this-synchronized方法，并且您无法（在您的类中）执行任何操作来在运行时禁止这种情况。为了避免这种潜在的陷阱，您可以在私有最终对象上进行同步，或者使用java.util.concurrent.locks中的Lock接口。

对于这个简单的示例，您可以交替使用 AtomicInteger 而不是同步 setter/getter。

Answer 3

Effective Java Second Edition 的第 67 项是避免过度同步，因此我会在私有锁对象上同步。

Answer 4

Java 中的每个对象都可以充当监视器。选择一种取决于您想要的粒度。选择“this”的优点和缺点是其他类也可以在同一监视器上同步。不过，我的建议是避免直接使用 Synchronize 关键字，而是使用 java.util.concurrency 库中的结构，这些结构更高级别并且具有明确定义的语义。这本书中有很多来自著名专家的好建议：

Java Concurrency in Practice
http://amzn.com/0321349601

Answer 5

在这种情况下，选择哪个对象作为锁定并不重要。但您必须始终使用相同的对象进行锁定才能实现正确的同步。上面的代码不能确保正确的同步，因为您曾经使用“this”对象作为锁，然后使用“lockObject”作为锁。