ASP.NET MVC 5 该单例缓存实现方案是否线程安全？

一、当前实现的正确性 #

当前实现的缓存核心逻辑是可行的：优先从HttpRuntime.Cache读取产品列表，缓存未命中时从数据库加载数据并写入缓存，设置48小时过期时间。但存在两个关键问题：

• 可变集合篡改风险：List<BaseItem>是引用类型，直接返回缓存中的列表实例后，外部调用方若修改该列表，会直接破坏缓存内的数据一致性。
• 并发性能瓶颈：GetProduct方法每次调用都依赖加锁的GetProducts()获取列表，所有请求必须串行执行，高并发场景下会严重拖慢响应速度。

二、线程安全性 #

当前实现具备线程安全性：所有涉及缓存读取、写入以及产品查询的操作都持有同一个静态锁Lock，确保同一时刻只有一个线程能执行这些代码块，避免了缓存竞态条件（比如多个线程同时检测到缓存为空，重复触发数据库查询）。但锁的粒度过大，牺牲了并发性能。

三、优化建议 #

1. 缩小锁粒度，提升并发能力 #

使用双重检查锁定（Double-Check Locking），仅在缓存初始化阶段加锁，读取缓存时无需加锁：

private readonly static object Lock = new object();

private List<BaseItem> GetProducts()
{
    // 首次无锁读取缓存
    List<BaseItem> products = (List<BaseItem>)HttpRuntime.Cache.Get("Products");
    if (products == null)
    {
        lock (Lock)
        {
            // 加锁后再次检查缓存，防止多线程重复触发DB查询
            products = (List<BaseItem>)HttpRuntime.Cache.Get("Products");
            if (products == null)
            {
                products = DBManager.inst.GetProducts();
                // 将列表转为只读，避免外部修改缓存数据
                var readOnlyProducts = new ReadOnlyCollection<BaseItem>(products);
                HttpRuntime.Cache.Insert("Products", readOnlyProducts, null, DateTime.MaxValue, TimeSpan.FromHours(48));
                products = readOnlyProducts.ToList();
            }
        }
    }
    return products;
}

2. 移除GetProduct方法的锁 #

GetProduct仅做查询操作，在GetProducts返回只读列表的前提下，查询是线程安全的，无需加锁：

public BaseItem GetProduct(int ArticulID)
{
    return GetProducts().FirstOrDefault(x => x.Articul == ArticulID);
}

3. 可选：改用更现代的缓存组件 #

推荐使用System.Runtime.Caching.MemoryCache替代HttpRuntime.Cache，它提供了更灵活的缓存策略和原生线程安全的API，适配更多业务场景。

内容的提问来源于stack exchange，提问作者sexty