同步值，而不是对象[重复]

Question

我想在 Java 中做这样的事情

  public void giveMoney(String userId, int money) {
    synchronized (userId) {

        Profile p = fetchProfileFromDB(userId);
        p.setMoney(p.getMoney() + userId);
        saveProfileToDB(p);

    }
   }

当然，在字符串上同步是不正确的。这样做的正确方法是什么？

Answer 1

您可以为字符串使用代理对象。

Object userIdMutex = new Object();

synchronized (userIdMutex) {
    Profile p = getProfile(userId);
    p.setMoney(p.getMoney() + p);
    saveProfile(p);
}

在您访问userId 时使用此互斥锁。

Answer 2

如果用户 ID 集有限，您可以在 String 的实习版本上进行同步。

如果您需要对实习进行更多控制，请使用 String.intern()（有一些缺点）或 Guava Interners 之类的东西。

Answer 3

这个呢：

String userId = ...;
Object userIdLock = new Object();
synchronized (userIdLock) {
    Profile p = getProfile(userId);
    p.setMoney(p.getMoney() + p);
    saveProfile(p);
}

这很简单，最重要的是显而易见。

Answer 4

我想有几个选择。

最简单的方法是您可以将userId 映射到线程安全映射中的锁定对象。其他人提到了实习，但我认为这不是一个可行的选择。

但是，更常见的选项是在 p（配置文件）上进行同步。如果getProfile() 是线程安全的，这是合适的，而且我怀疑它可能是。

Answer 5

原则上，您可以在 Java 中的任何对象上进行同步。在 String 对象上同步本身并不是“不正确的”；这取决于你到底在做什么。

但是如果userId 是方法中的局部变量，那么这将不起作用。每个执行该方法的线程都有自己的变量副本（大概是指每个线程的不同String 对象）；线程之间的同步当然只有当你让多个线程在同一个对象上同步时才有效。

您必须在对象的成员变量上创建您正在同步的对象，该对象包含您拥有synchronized 块的方法。如果多个线程在同一个对象上调用该方法，您将实现互斥。

class Something {
    private Object lock = new Object();

    public void someMethod() {
        synchronized (lock) {
            // ...
        }
    }
}

您还可以使用 java.util.concurrent.locks 包中的显式锁，如果您需要，它可以为您提供更多控制：

class Something {
    private Lock lock = new ReentrantLock();

    public void someMethod() {
        lock.lock();
        try {
            // ...
        } finally {
            lock.unlock();
        }
    }
}

特别是如果你想要一个独占锁来写入，但你不希望线程在读取时必须相互等待，你可能想要使用ReadWriteLock。

Answer 6

根据您的示例，我假设您想要获得对配置文件类的锁定，对其进行更改，然后释放锁定。在我看来，同步并不是你所需要的。您需要一个类来管理这些记录，并允许您在需要对其进行更改时锁定和解锁记录，也就是源代码控制样式。

看看这个：Lock class Java 5

Answer 7

从理论上讲，由于可以对实习对象进行 GC，因此可以在不同时间对不同对象（具有相同值）进行同步。仍然可以保证互斥性，因为不可能在同时时间对不同的对象进行同步。

然而，如果我们在不同的对象上同步，happens-before 关系是有疑问的。我们必须检查实现以找出答案。而且由于它涉及到 GC，Java 内存模型没有解决这个问题，因此推理可能相当困难。

这是理论上的反对意见；实际上我认为这不会造成任何问题。

不过，对于您的问题，仍然可以找到简单、直接且理论上正确的解决方案。例如Simple Java name based locks?