泛型MemoryCache类

Question

我希望有一个缓存类来缓存不同的类型。我希望每种类型都在不同的MemoryCache中缓存，但要以一种通用的方式缓存。

我做得对吗？

internal static class RecordsCache
{
    private static Dictionary<string, ObjectCache> cacheStore;
    static private CacheItemPolicy policy = null;

    static RecordsCache()
    {
        cacheStore = new Dictionary<string, ObjectCache>();

        ObjectCache activitiesCache = new MemoryCache(typeof(Activity).ToString());
        ObjectCache lettersCache = new MemoryCache(typeof(Letter).ToString());
        ObjectCache contactssCache = new MemoryCache(typeof(Contact).ToString());

        cacheStore.Add(typeof(Activity).ToString(), activitiesCache);
        cacheStore.Add(typeof(Letter).ToString(), lettersCache );
        cacheStore.Add(typeof(Contact).ToString(), contactssCache );

        policy = new CacheItemPolicy();
        policy.Priority = CacheItemPriority.Default;
        policy.AbsoluteExpiration = DateTimeOffset.Now.AddHours(12);
    }

    public static void Set<T>(string userID, int year, List<T> records)
    {
        var cache = cacheStore[typeof(T).ToString()];
        string key = userID + "-" + year.ToString();
        cache.Set(key, records, policy);
    }

    public static bool TryGet<T>(string userID, int year, out List<T> records)
    {
        var cache = cacheStore[typeof(T).ToString()];
        string key = userID + "-" + year.ToString();

        records = cache[key] as List<T>;
        return records != null;
    }

    public static void Remove<T>(string userID, int year)
    {
        var cache = cacheStore[typeof(T).ToString()];
        string key = userID + "-" + year.ToString();
        cache.Remove(key);
    }
}

Answer 1

1. 而不是拥有一个缓存私有静态Dictionary cacheStore；您可以有一个键类型:私有静态Dictionary cacheStore；这意味着您不需要到处调用typeof(T).ToString()。
2. 您复制了在所有三个方法中构建密钥的代码--应该将其提取到私有的BuildKey(string userId, int year)方法中。这意味着，如果您需要更改键的构建方式，您只需要触摸一个方法，而不是所有的方法。
3. 我将提供一个Register方法，它接受要为其创建新缓存的类型，而不是硬编码它。
4. 缓存策略也是不可配置的--这意味着除非您想要更改代码，否则您将继续使用它。
5. 您真的应该认真考虑静态缓存是否是正确的方法。它将在单元测试中给您带来麻烦，也会使多个类不可能有不同的缓存，例如，不同的策略。

Answer 2

在我看来，每件事都有很好的编码。

这里只是一些个人偏好：

1. 我将提供另外一个Set<T>方法，它接受一个函数作为输入，而不是List<T>：公共静态空集(string userId，int，Func> retrieveData) {}
2. 我希望有更多一种方法将Get和Set组合在一起，看起来是: public List TryGetAndSet(string userId，int retrieveData，Func> retrieveData) { //如果缓存项存在返回cacheItem //如果缓存项不存在，则通过执行retrieveData//如果检索到的结果，设置为缓存并返回结果}来检索数据。
3. 多1项，以检查某些缓存项是否存在。