Вложенные 'synchronize': как этот синхронизированный метод Java будет реорганизован - и может ли это иметь какой-то смысл?

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(). Код не может быть сломан, но это, конечно, не очень красиво.