在两个String对象上同步

Question

我只在一个字符串对象上找到了同步的答案，而不是两个。

这不是一项真正的任务，而是一项任务。我有一个图书馆可以把钱从一个账户转到另一个账户。我无法访问帐户对象以锁定它。我只能用图书馆里的东西。传输（字符串从、字符串到），这不是线程安全的。我有一个帐户ID为字符串的方法。我需要在没有死锁的情况下锁定这两个字符串。

到目前为止，我所做的是：

>

使用创建了新字符串。intern方法（字符串fr=from.intern（）。但这是一种糟糕的做法，我不允许使用这种方法。但它奏效了。

从旧字符串创建新字符串（String fr=new String（from））。这也有效（我没有死锁），但我对这个解决方案有怀疑。

还有其他方法可以锁定两个字符串吗？

我尝试使用ConcurrentHashMap并将字符串放在那里，但没有成功。

可能有一种方法可以将字符串放入某些对象中，但是这些对象应该在哪里创建呢？我可以在transfer（）中创建它们，但对局部变量进行同步也不是一种好的做法。

我的方法是：

public void transfer(String from, String to, int amount) {
        String fr = new String(from);
        String too = new String(to);

        int fromHash = System.identityHashCode(fr);
        int toHash = System.identityHashCode(too);

        if (fromHash < toHash) {
            synchronizedTransfer(from, to, amount, fr, too);
        } else if (fromHash > toHash) {
            synchronizedTransfer(to, from, amount, too, fr);
        } else {
            synchronized (tieLock) {
                synchronizedTransfer(from, to, amount, fr, too);
            }
        }
    }

private void synchronizedTransfer(String from, String to, int amount, String fr, String too) {
        synchronized (fr) {
            synchronized (too) {
                SomeLibrary.transfer(from, to);
            }
        }
    }

Answer 1

可以使用嵌套的同步块在两个对象上同步。为了防止任何死锁，我将按字典顺序比较这两个字符串：

public void transfer(String from, String to, int amount) {
        int comparingResult = from.compareTo(to);
        
        if (comparingResult > 0) {
            synchronized (from) {
                synchronized (to) {
                    SomeLibrary.transfer(from, to);
                }
            }
        } else if (comparingResult < 0) {
            synchronized (to) {
                synchronized (from) {
                    SomeLibrary.transfer(from, to);
                }
            }
        } else {
            if (from == to) { // Do not use equals() here
                synchronized (from) {
                    SomeLibrary.transfer(from, to);
                }
            } else {
                throw new UnsupportedOperationException();
            }
           
        }
    }

else部分是可选的，这取决于您可以在同一个帐户之间转移资金这一事实，在我看来，这有点无用。