c# - 如何正确处理向字典添加新项目？

Question

我有一个 ASP.NET 应用程序，我在其中使用静态类作为缓存。该静态类内部是内部字典，其中包含缓存的对象。当然，静态类中有 Add/Remove/Clear 之类的方法...看起来如下：

public static class CacheManager
{
    private static Dictionary<string, object> cacheItems = new Dictionary<string, object>();

    private static ReaderWriterLockSlim locker = new ReaderWriterLockSlim();

    public static Dictionary<string, object> CacheItems
    {
        get
        {
            return cacheItems;
        }
    }

    public static void AddCacheItem(string key, object data)
    {
        locker.EnterWriteLock();
        try
        {
            cacheItems.Add(key, data);
        }
        finally
        {
            locker.ExitWriteLock();
        }
    }

    ...
}

当 ASP.NET 应用程序运行时，这些项目被添加到缓存（字典）中。我只想问我是否应该在 Add 方法中检查是否已经以这种方式添加了密钥：

    public static void AddCacheItem(string key, object data)
    {
        locker.EnterWriteLock();
        try
        {
            if (!cacheItems.ContainsKey(key))
            {
                cacheItems.Add(key, data);
            }
        }
        finally
        {
            locker.ExitWriteLock();
        }
    }

还是将其保留在第一个代码片段中？

先感谢您。

Answer 1

您可以实现另一种方法 TryAdd

public static bool TryAddCacheItem(string key, object data)
{
    locker.EnterWriteLock();
    try
    {
        if (cacheItems.ContainsKey(key))
        {
            return false;
        }
        cacheItems.Add(key, data);
        return true;
    }
    finally
    {
        locker.ExitWriteLock();
    }
}

我也建议使用ConcurrentDictionary。这是 ConcurrentDictionary 的代码。

Answer 2

只需执行两次：

cacheItems.Add(key, data);

......你会知道答案的。基本上，异常将被 try/finally 块抛出并捕获。我个人会使用 if 语句，而不是将所有逻辑责任放在 try 块上，但这是一个偏好问题......