Необходим несколько объектов блокировки?

Question

Учитывая следующий класс:

class x 
{ 
    Object lockOne = new Object(); 
    Object lockTwo = new Object(); 

    List<Something> listOne = new List<Something>(); 
    List<Something> listTwo = new List<Something>(); 

    void MethodOne() 
    { 
     lock(lockOne) 
     { 
      // some operation on listOne 
     } 
    } 

    void MethodTwo() 
    { 
     lock(lockTwo) 
     { 
      // some operation on listTwo 
     } 
    } 
} 

Правильно ли использовать два запорных объектов, предполагая, что MethodOne() и MethodTwo() могут быть вызваны из различных потоков одновременно, отметив, что listOne и listTwo не связаны в любом случае. Единственными операциями, связанными с блокировками, являются операции, указанные в комментариях выше.

Answer 1

Да, это правильно. Он избегает ненужного блокирования одного списка только потому, что работает другой список.

Answer 2

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

class x 
{ 
    List<Something> listOne = new List<Something>(); 
    List<Something> listTwo = new List<Something>(); 

    void MethodOne() 
    { 
     lock (listOne) 
     { 
      // some operation on listOne 
     } 
    } 

    void MethodTwo() 
    { 
     lock (listTwo) 
     { 
      // some operation on listTwo 
     } 
    } 
}