C# 静态属性锁定

Question

只是寻找此代码的代码审查。 ASP.net 缓存不是一个选项。静态列表将在每天获得超过 10K 页面浏览量的网站上被大量访问，并且可能会发生并发读取尝试。在重建列表时重新启动应用程序时，我想知道是否有任何我可能忽略的问题？锁定正在实例化的列表是最佳实践吗？

public class MyClass
{
        private static List<Entry> _listCache = null; 
        protected static List<Entry> ListCache
        {
            get
            {

                if (_listCache == null)
                {
                    _listCache = new List<Entry>();
                    lock (_listCache)
                    {
                        //Add items to the list _listCache from XML file
                    }
                }
                return _listCache;
            }
        }
        //....Other methods that work with the list
}

Answer 1

10k 次查看 - 每 8 秒一次...不确定您是否需要担心太多...;-p

但是重新编写代码 - 这使事情变得过于复杂，并且您仍然可能最终将其初始化两次。我只是使用静态构造函数来做到这一点；它会更加坚固。如果您必须具有完全隔离的延迟加载（即使在类型上使用其他静态方法），则可以使用内部类来实现相同的效果：

public class MyClass
{
    static class InnerCache {
        internal static readonly IList<Entry> _listCache;
        static InnerCache() {
            List<Entry> tmp  = new List<Entry>();
            //Add items to the list _listCache from XML file
            _listCache = new ReadOnlyCollection<Entry>(tmp);
        }
    }
    protected static IList<Entry> ListCache {
        get {return InnerCache._listCache;}
    }
}

我也会担心有人变异的机会列表 - 可能想使用只读列表！

Answer 2

没有真正的理由这对你不起作用。但是，如果您想按照示例代码的方式执行此操作，则需要在检查 _listCache 是否为 null 之前锁定。所以你需要一个单独的监视器来锁定。像这样的东西：

public class MyClass
{
        private static object _listCacheMonitor = new object();
        private static List<Entry> _listCache = null; 
        protected static List<Entry> ListCache
        {
            get
            {
                lock (_listCacheMonitor) {    
                    if (_listCache == null)
                    {
                        _listCache = new List<Entry>();
                        //Add items to the list _listCache from XML file
                    }
                }
                return _listCache;
            }
        }
        //....Other methods that work with the list
}

Answer 3

静态构造函数可能是您最好的选择。静态构造函数在运行时会阻塞所有依赖它的线程，并且它只会运行一次。正如这里的代码所示，锁实际上并没有做任何事情，并且有很多方式可能会发生不好的事情，包括同时从 XML 初始化多个列表。事实上，一个线程可以创建一个新列表，然后锁定并加载一个不同列表，然后返回第三个列表，具体取决于线程切换发生的时间。

Answer 4

多个线程可以初始化_listCache。根据代码生成优化和运行时执行优化，这可能会导致多个线程锁定和更新不同的对象。此外，您不能将列表公开为属性，允许任何人添加/删除带有 oa 锁的对象。

您最好使用不可变列表，多个读者可以在只读模式下安全地解析该列表。或者，您可以使用读写锁，但在初始化控制和访问读写控制之间，事情会变得非常棘手。