Синхронизация: несколько блокировок - создание объектов блокировки?

Question

Быстрый (я думаю) вопрос параллелизма: я проходил курс многопоточности на Udemy.com, и учитель говорил через приведенный ниже код. Хотя он объяснил это, я все еще не уверен, почему вы создавали бы объекты lock1 и lock2, а не блокировали по list1 и list2.

App.java:

public class App { 

    public static void main(String[] args) { 
     Worker worker = new Worker(); 
     worker.main(); 
    } 
} 

Worker.java:

import java.util.ArrayList; 
import java.util.List; 
import java.util.Random; 


public class Worker { 

    private Random random = new Random(); 

    private Object lock1 = new Object(); 
    private Object lock2 = new Object(); 

    private List<Integer> list1 = new ArrayList<Integer>(); 
    private List<Integer> list2 = new ArrayList<Integer>(); 

    public void stageOne() { 

     synchronized (lock1) { 
      try { 
       Thread.sleep(1); 
      } catch (InterruptedException e) { 
       // TODO Auto-generated catch block 
       e.printStackTrace(); 
      } 

      list1.add(random.nextInt(100)); 
     } 

    } 

    public void stageTwo() { 

     synchronized (lock2) { 
      try { 
       Thread.sleep(1); 
      } catch (InterruptedException e) { 
       // TODO Auto-generated catch block 
       e.printStackTrace(); 
      } 

      list2.add(random.nextInt(100)); 
     } 

    } 

    public void process() { 
     for(int i=0; i<1000; i++) { 
      stageOne(); 
      stageTwo(); 
     } 
    } 

    public void main() { 
     System.out.println("Starting ..."); 

     long start = System.currentTimeMillis(); 

     Thread t1 = new Thread(new Runnable() { 
      public void run() { 
       process(); 
      } 
     }); 

     Thread t2 = new Thread(new Runnable() { 
      public void run() { 
       process(); 
      } 
     }); 

     t1.start(); 
     t2.start(); 

     try { 
      t1.join(); 
      t2.join(); 
     } catch (InterruptedException e) { 
      // TODO Auto-generated catch block 
      e.printStackTrace(); 
     } 

     long end = System.currentTimeMillis(); 

     System.out.println("Time taken: " + (end - start)); 
     System.out.println("List1: " + list1.size() + "; List2: " + list2.size()); 
    } 
} 

Answer 1

Я не думаю, что мотивация для этого выражается в коде, вы дали, но является вообще передовой практики. Тем не менее, одна и та же самая лучшая практика требует, чтобы объекты блокировки были также final.

Если рассматриваемые списки были либо приняты снаружи, либо подвержены наружу с помощью метода, то преимущество отдельных объектов блокировки становится более очевидным: никогда не рекомендуется открывать блокировки для чужой код, поскольку инопланетный код может затем использовать их самостоятельно для блокировки, нарушая ваш собственный шаблон использования.

Если списки строго конфиденциальны, то их мониторы могут использоваться для внутренней блокировки; однако позднее изменение политики доступа в списках может непреднамеренно повлиять на политику блокировки. Поэтому, начиная с частных замков, также можно избежать любых ошибок в будущем.