我使用给定的单例模式来缓存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; }
}
答案 0 :(得分:2)
我认为没有理由让你成为单身人士。我会为每个获取/设置值的属性编写一个包装器。这样,您可以使用IoC容器更轻松地注入它并提高可测试性。
现在看来,你的代码不是线程安全的,这也可能导致问题。