是否可以使用Hashtable作为锁对象？

Question

我有下面的代码，其中“_ht”是一个代表缓存的哈希表，“_isLoaded”代表它是否加载。

我们的系统有许多进程访问“_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

你当然可以锁定Hashtable对象，就像你可以在lock语句中使用任何引用类型实例的.NET。然而，它通常被认为是一种劣质的方法，主要是因为当代码的其他部分有一个或多个锁对象可用时，他们很难跟踪代码如何使用锁定，他们也可能使用它来锁定再次，不可思议，但你会惊讶于人们写的代码）。

对于通常的锁定，单独的锁定对象是最好的。我会注意到在你的代码示例中，_ht应该是readonly，并且如果你添加一个单独的锁定对象（例如lockObj），它也应该是只读的。

也就是说，单例情景不应该以这种方式实现。相反，您应该使用CLR自己的静态初始化，或Lazy<T>类：

private static readonly Hashtable _ht = InitializeTable(); 

internal static Hashtable GetHT() { return _ht; } 

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

    LoadHt(table); 

    return table; 
} 

或者：

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这个类已经过时了。作为非泛型类，您应该认真考虑升级代码以使用现在十年的泛型类型。 Dictionary<TKey, TValue>类是最直接的替代品，但人们有时使用Hashtable作为一个简单集合，对此，数据结构更合适。

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实例。

Best practice虽然是定义一个私有对象来锁定，或者私有静态对象变量来保护所有实例的公共数据。

private Object thisLock = new Object(); 

编辑感谢@PeterDuniho您指出