Простой кэш в памяти

Добрый день!

Понадобилось тут сделать совсем примитивный кэш, чтобы лишний раз в базу не лазить. При этом данные в базе статичные, и вопрос не столько в обновлении данных, сколько в том, когда их выбросить, чтобы просто не занимать память, ну и простота использования конечно важна. Сначала хотел использовать MemoryCache, но мне показалось он слишком замороченный, да ещё и без строгой типизации.

За примером реализации прошу под кат.

Реализация

Общая идея такая: кэш должен быть удобным и прозрачным в использовании. Я вдохновился интерфейсом класса ThreadLocal. Поэтому в конструкторе требуется фабричный метод для получения исходных значений, а потом они уже кэшируются. Опять же, в моём случае, я посчитал вполне логичным держать данные на слабых ссылках (WeakReference), таким образом, пока данные нужны, кто-то их использует, они будут доступны. Дальше, по мере выделения новой памяти, данные из кэша будут вытесняться.
Есть всего два метода: индексатор, который вернёт данные либо из источника, либо из кэша.
А также метод CleanCache, чтобы очищать словарь от мёртвых ссылок и таким образом устранить утечку памяти. В реальных условиях далеко не всегда это необходимо. Например, в моих условиях новые записи в кэше будут появляться всего несколько раз в день, так что даже за годы работы ссылки не сожрут сколько-то значительный объём памяти.

Цель публикации

В основном, мне интересно услышать мнение о моей реализации, как бы вы сделали, есть ли ошибки. Ну и если кому-то это поможет — берите, используйте :)

Исходник

public class WeakCache<TKey,TValue> where TValue : class
    {
        const int cacheCleanInterval = 60;
        private readonly Func<TKey, TValue> getter;
        private readonly Dictionary<TKey, WeakReference> data = new Dictionary<TKey, WeakReference>();
        private readonly ReaderWriterLockSlim rwLock = new ReaderWriterLockSlim();
        private DateTime lastCacheClean = DateTime.MinValue;
        
        public WeakCache(Func<TKey,TValue> getter)
        {
            this.getter = getter;
        }

        public TValue this[TKey key]
        {
            get
            {
                CleanCache();
                try
                {
                    rwLock.EnterUpgradeableReadLock();
                    WeakReference wr;
                    TValue val;
                    if (data.TryGetValue(key, out wr))
                    {
                        val = (TValue)wr.Target;
                        if (val != null)
                            return val;
                    }
                    try
                    {
                        rwLock.EnterWriteLock();
                        if (data.TryGetValue(key, out wr))
                        {
                            val = (TValue)wr.Target;
                            if (val != null)
                                return val;
                        }
                        data[key] = new WeakReference(val = getter(key));
                        return val;
                    }
                    finally
                    {
                        rwLock.ExitWriteLock();
                    }
                }
                finally
                {
                    rwLock.ExitUpgradeableReadLock();
                }
            }
        }

        void CleanCache()
        {
            if ((DateTime.Now - lastCacheClean).TotalSeconds > cacheCleanInterval)
            {
                try
                {
                    rwLock.EnterWriteLock();
                    if ((DateTime.Now - lastCacheClean).TotalSeconds > cacheCleanInterval)
                    {
                        lastCacheClean = DateTime.Now;
                        var refs = data.ToArray();
                        foreach (var weakReference in refs)
                        {
                            if (!weakReference.Value.IsAlive)
                                data.Remove(weakReference.Key);
                        }
                    }
                }
                finally 
                {
                    rwLock.ExitWriteLock();
                }
            }
        }
    }