Question

我在一个正在研究的项目中偶然发现了这个方法，并立即认为出了问题。我觉得这对任何事都不适合。

但我没有得到这个意图和含义。这可以重构吗？第二个同步是否有意义 - 并不是第三个同步冗余/不必要的？

我刚刚开始在java中进入高级并发/线程安全编程 - 详细解释为什么这有意义或没有意义，为什么非常感谢！

 @Override
    public synchronized void run() {
        synchronized (this.market) {
            synchronized (this.market.getWallet()) {
                this.handler.handleEvent(new WalletLecherEvent(this, market,
                        market.leechWallet()));
            }
        }
    }

提前致谢

编辑，以提供更多上下文：

public class WalletLeecherWorker extends Worker {
        private IAbstractMarketAPI market = null;

        private Thread thread = null;

        public WalletLeecherWorker(IEventHandler handler, IAbstractMarketAPI market) {
            super(handler);
            this.market = market;
        }

        public void startThread() {
            if (thread != null)
                return;

            thread = new Thread(this);
            thread.start();
        }

        public MARKETs getMarket() {
            return this.market.getName();
        }

        @Override
        public synchronized void run() {
            synchronized (this.market) {
                synchronized (this.market.getWallet()) {
                    this.handler.handleEvent(new WalletLecherEvent(this, market,
                            market.leechWallet()));
                }
            }
        }
    }

..和方法market.getWallet()：

@Override
public Wallet getWallet() {
    return this.wallet;
}

我认为的意图是阻止所有线程获取旧钱包，从而错误地同步/弃用数据 - 只要此线程运行（）;

Answer 1

代码获取以下对象的锁：

this - 防止多个线程在同一个run()实例上调用WalletLeecherWorker
this.market - 如果另一个线程已获得锁定，则阻止run()继续进行，考虑到market实例可能已共享，这是一个合理的假设
this.market.wallet - 与之前相同

除了正在同步的run()方法之外，其中没有一个显然是不必要的。除了以下同步块之外，它不提供任何保护。代码本身可能是因为使用了非常细粒度的锁定，这个特定代码需要锁定所有内容，而其他代码可能只锁定钱包或市场和钱包。

然而，这段代码可能会出错（甚至是死锁）。如果以错误的顺序锁定对象，则可能会出现死锁。你看，它也不是很易读。将Thread作为具有startThread()方法的类的实例变量也是可疑的。代码可能不会被破坏，但它肯定不是很漂亮。

嵌套＆＃39;同步＆＃39;：如何重构这种同步的Java方法 - 它可以像这样有意义吗？

1 个答案: