Niezbędne obiekty wielokrotnej blokady?

Question

Biorąc pod uwagę następujące klasy:

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 
     } 
    } 
} 

Czy to prawidłowe użycie dwóch obiektów blokujące Zakładając, że MethodOne() i MethodTwo() mogą być wywoływane z różnych wątków jednocześnie zauważając, że listOne i listTwo nie są związane w żaden sposób. Jedyne operacje związane z zamkami są określone w komentarzach powyżej.

Answer 1

Tak, jest prawidłowa. Pozwala to uniknąć niepotrzebnego blokowania jednej listy tylko dlatego, że trwają prace nad drugą listą.

Answer 2

Nie ma tutaj potrzeby oddzielnych obiektów blokujących. Poniższy kod będzie działać tak samo dobrze, z mniej kodu, mniejszym obciążeniu i mniejsze szanse na pomyłkę przy użyciu nieprawidłowej Lock:

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 
     } 
    } 
}