安全地使用“HttpContext.Current.Cache”

Question

我在这样的网络服务方法中使用Cache：

var pblDataList = (List<blabla>)HttpContext.Current.Cache.Get("pblDataList");

if (pblDataList == null)
{
    var PBLData = dc.ExecuteQuery<blabla>(@"SELECT blabla");

    pblDataList = PBLData.ToList();

    HttpContext.Current.Cache.Add("pblDataList", pblDataList, null,
        DateTime.Now.Add(new TimeSpan(0, 0, 15)),
        Cache.NoSlidingExpiration, CacheItemPriority.Normal, null);
}

但我想知道，这段代码是线程安全的吗？ Web 服务方法被多个请求者调用。在缓存为空的同时，不止一个请求者可能会尝试检索数据并添加到Cache。

查询需要 5 到 8 秒。围绕此代码引入锁定语句会防止任何可能的冲突吗？ （我知道可以同时运行多个查询，但我想确保一次只运行一个查询。）

Answer 1

缓存对象是线程安全的但HttpContext.Current 将无法从后台线程中获得。这可能适用于您，也可能不适用于您，从您的代码 sn-p 是否或不是您实际上正在使用后台线程，但如果您现在使用或决定在将来的某个时间点使用，请记住这一点。

如果您可能需要从后台线程访问缓存，请改用HttpRuntime.Cache。

此外，虽然缓存上的单个操作是线程安全的，但顺序查找/存储操作显然不是原子的。您是否需要它们是原子的取决于您的特定应用程序。如果同一查询多次运行可能是一个严重的问题，即如果它产生的负载超过您的数据库能够处理的负载，或者如果请求返回的数据立即被覆盖在缓存，那么您可能希望在整个代码块周围加一个锁。

但是，在大多数情况下，您真的想先进行分析，然后看看这是否真的是一个问题。大多数 Web 应用程序/服务不关心缓存的这一方面，因为它们是无状态的，而且缓存是否被覆盖也没关系。

Answer 2

你是对的。检索和添加操作不被视为原子事务。如果您需要防止查询多次运行，则需要使用锁。

（通常这不是什么大问题，但在长时间运行的查询的情况下，它可能有助于减轻数据库的压力。）

Answer 3

我相信 Add 应该是线程安全的 - 即如果 Add 使用相同的键被调用两次，它不会出错，但显然查询可能执行两次。

然而，另一个问题是数据是线程安全的。不能保证每个List<blabla> 都是隔离的——这取决于缓存提供者。内存缓存提供程序直接存储对象，因此如果任何线程编辑数据（添加/删除/交换列表中的项目，或更改其中一项的属性），则存在冲突风险。但是，使用 serializing 提供程序应该没问题。当然，这就要求blabla 是可序列化的……