Disadvanage данного Singleton cache patterm

Question

Я использую данный шаблон singleton для кэширования объекта класса в ASP.NET. Может ли кто-нибудь выделить недостатки этого подхода?

public class CacheManager 
{ 
private static CacheManager _instance; 

protected CacheManager(string key) { id = key; } 
public static CacheManager getInstance(string key) 
{ 
    if (HttpContext.Current.Cache[key] == null) 
     HttpContext.Current.Cache.Insert(key, _instance = new CacheManger(key), null, System.Web.Caching.Cache.NoAbsoluteExpiration, TimeSpan.FromSeconds(120),CacheItemPriority.Default,new CacheItemRemovedCallback(ReportRemovedCallback)); 

    return (CacheManager)HttpContext.Current.Cache[key]; 
} 

private static void ReportRemovedCallback(string CacheManager, object value, CacheItemRemovedReason reason) 
{    
    _instance = null; 
} 

public string id { get; private set; }   
public string field1 { get; set; } 
public string field2 { get; set; } 
public string field3 { get; set; }   
} 

Answer 1

Я не вижу причин для того, чтобы сделать это синглом. Я бы просто написал оболочку для каждого свойства, которое получает/устанавливает значения. Таким образом, вы можете легко вводить его с помощью контейнера IoC и повышать тестируемость.

Как сейчас, ваш код не является потокобезопасным, что может вызвать проблемы.