C# 중첩는 동기화

Question

나는 몇 가지 데이터 멤버와 클래스가 있습니다. 나는 Foo 클래스도 안전하게 쓰레드로 만들고 싶습니다. 아래 코드가이 작업을 수행하는 가장 좋은 방법입니까?

public Foo(Foo f) 
{ 
    lock(f) 
    { 
     foreach (int i in f.intLst) 
     { 
      this.intLst.Add(i) 
     } 
     //More complicated 
    } 
} 

public Foo Merge(Foo f1, Foo f2) 
{ 
    Foo fReturn = new Foo(f1);//Using the constructor above 

    lock(f2) 
    { 
     foreach (int i in f2.intLst) 
     { 
      fReturn.intLst.Add(i); 
     } 

     //More complicated merge code 
    } 

    return fReturn; 
} 

간단합니까? 대신 모니터를 사용해야합니까? 나는 락이 모니터 클래스를 구현한다는 것을 알고 있지만 아마도 여전히 이유가있다. 또한 위 샘플에서 교착 상태 처리에 대해 걱정할 필요가 있습니다.

f1에 복사 생성자 중에 f2가 변경되면 f1과 f2를 모두 lock()해야 할 필요가 있습니다.

Answer 1

예, 간단합니다.

코드가 좋습니다. 액세스 할 때까지 f2를 잠글 필요가 없으며 f1을 복사 한 후에 f1을 잠그지 않아도됩니다. 그래서 동시에 둘 다 잠글 필요가 없습니다.

예를 들어 항목 자체는 변경할 수없는 단순한 정수이므로 잠글 필요가 없지만 변경할 수있는 항목이있는 경우 개별 항목을 잠글 필요가 있습니다.

한 번에 하나의 잠금 만 있기 때문에이 코드로 교착 상태가 걱정되지 않아도됩니다. 물론 다른 코드가 잠금을 유지하면서 호출하면 교착 상태의 가능성에 대해 걱정할 필요가 있습니다.

Answer 2

lock()은 다른 lock() 블록에서만 작동합니다. 따라서 lock (f)는 lock (f)을 가지고있는 한 f로 작업중인 다른 코드를 차단하므로 lock (f)은 (f)의 모든 용도에 적용되어야합니다.

대상 Foo가 아직 구성되지 않았으므로 걱정할 필요가 없습니다. 다른 사람은 아직 참조 할 수 없습니다.