잠금 개체로 Hashtable을 사용할 수 있습니까?

Question

"_ht"는 캐시를 나타내는 Hashtable이고 "_isLoaded"는로드 된 코드를 나타냅니다.

Google 시스템에는 '_ht'개체에 액세스하는 많은 프로세스가 있으며로드되지 않은 경우 대기해야합니다.

"_ht"를 잠금 개체로 사용하는 것이 잘못 되었습니까? 이 시나리오에서 전용 객체 유형 클래스 멤버를 사용해야합니까?

이 클래스는 SINGLETON입니다.

private Hashtable _ht = new Hashtable(); 
private bool _isLoaded = false; 

internal Hashtable GetHT() 
     { 
      if (_isLoaded == false) 
      { 
       lock (_ht) 
       { 
        if (_isLoaded == false) 
        { 
         LoadHt(_ht); 
        } 
       } 
      } 

      return _ht; 
     } 

Answer 1

을 지적 @PeterDuniho. 그러나 일반적으로 하나의 잠금 객체가 코드의 다른 부분에서 사용할 수있을 때 코드가 잠금을 사용하는 방식을 추적하는 것이 어렵 기 때문에 열등한 접근 방식으로 간주됩니다 (잠금을 위해 잠금 객체를 사용할 수도 있습니다). 다시 말하지만, 사람들이 때때로 쓰는 코드에 놀랄 것입니다.)

일반적으로 잠금의 경우 별도의 잠금 개체가 바람직합니다. 코드 예에서 _ht은 readonly이어야하며 별도의 잠금 객체 (예 : lockObj)를 추가하면 읽기 전용이어야합니다.

그렇다고 싱글 톤 시나리오를 이런 방식으로 구현하면 안됩니다.

private static readonly Hashtable _ht = InitializeTable(); 

internal static Hashtable GetHT() { return _ht; } 

private static Hashtable InitializeTable() 
{ 
    Hashtable table = new Hashtable(); 

    LoadHt(table); 

    return table; 
} 

또는 : 대신, 당신은 CLR 자신의 정적 초기화, 또는 Lazy<T> 클래스를 사용해야 하나

private static readonly Lazy<Hashtable> _ht = new Lazy<Hashtable>(() => InitializeTable()); 

internal static Hashtable GetHT() { return _ht.Value; } 

private static Hashtable InitializeTable() 
{ 
    Hashtable table = new Hashtable(); 

    LoadHt(table); 

    return table; 
} 

당신이 액세스 할 수있는 유형의 다른 구성원이있을 때 후자가 유용합니다, 하지만 가능한 한 오랫동안 해시 테이블 초기화가 지연되도록해야합니다 (예 : 실제로 코드에 액세스 할 수없는 경우이를 초기화하지 않아도 됨).

(나는 시나리오를 싱글 톤으로 설명했기 때문에 모든 것을 static으로 변경했으며이 경우 코드 예제에서는 static 구성원 만 이해할 수 있음).

마지막으로 Hashtable 클래스의 기한이 오래되었습니다. 비 제네릭 클래스로서, 당신은 정말로 코드를 10 년 된 제네릭 타입을 사용하도록 업그레이드하는 것을 진지하게 고려해야합니다. Dictionary<TKey, TValue> 클래스가 가장 직접적인 대체 클래스이지만 사람들은 간략한 집합으로 Hashtable을 사용하기 때문에 HashSet<T> 데이터 구조가 더 적절할 수 있습니다.

Answer 2

여기에있는 첫 번째 스레드를 원한다면 시작하십시오. 괜찮습니다. 하지만 일반적으로 잠금 객체를 사용합니다.

private Hashtable _ht = new Hashtable(); 
private bool _isLoaded = false; 
private object lockObj = new object(); 

internal Hashtable GetHT() 
{ 
    if (_isLoaded == false) 
    { 
     lock (lockObj) 
     { 
      if (_isLoaded == false) 
      { 
       LoadHt(_ht); 
      } 
     } 
    } 

    return _ht; 
} 

Answer 3

개체 자체가 잠기기 (보호됨)로 시작하지 않습니다. lock 키워드에 사용 된 참조는 동일한 객체 참조를 사용하는 다른 (또는 동일한) 코드 섹션과 동시에 실행하면 안되는 코드 섹션을 표시하거나 태그하는 데 사용됩니다. 실제로 개체 자체에 영향을주지는 않습니다. 대답은 예입니다. 기존 HashTable 인스턴스를 lock 문에 사용할 수 있습니다.

Best practice 그러나 잠글 사설 개체를 정의하거나 모든 인스턴스에 공통된 데이터를 보호하기 위해 개인 정적 개체 변수를 정의하는 것입니다.

private Object thisLock = new Object(); 

편집 덕분에 당신은 확실히 당신이 lock 성명에서 .NET의 모든 참조 형 인스턴스를 사용할 수있는 것처럼의 Hashtable 개체를 잠글 수 있습니다