我犯了一个错误,您能指出吗?
2009-09-08 10:01 Jeffrey Zhao 阅读(15520) 评论(113) 编辑 收藏 举报这是我最近在项目中犯的一个错误,您能指出吗?
这个项目在数据访问方面使用了传统的Repository模式。为此,我定义了一个Repository基类,可以让每个不同的Repository继承它:
public abstract class Repository<T> { ... } public class UserRepository : Repository<User> { ... } public class ArticleRepository : Repository<Article> { ... }
Repository类中提供了一些基础功能,可以让各个不同的Repository子类复用,这是十分常见的做法。其中一个功能便是资源的管理。在项目中,资源的管理通过Resource Manager进行——这不是.NET框架中自带的ResourceManager类,而是自己定义的组件:
public abstract class Repository<T> { private IResourceManager ResourceManager { get { ... } } } public interface IResourceManager { void Set(object key, object value); T Get<T>(object key); }
被Resource Manager管理的资源会在一定时刻统一释放,而不同的IResourceManager对象的实现也有不同的“作用域”。例如,在Web项目中常见的做法是,对于每个数据库来说,每个请求使用一个连接。因此,这里使用的可能就是RequestResourceManager,它基于HttpReuqest提供资源存储,保证资源的作用域是当前请求。
数据库连接也是需要管理的资源之一,因此我在Repository中定义了一个GetConnection方法(假设只需要连接一个数据库):
public abstract class Repository<T> { private readonly static Guid ConnectionKey = Guid.NewGuid(); protected IDbConnection GetConnection() { var conn = this.ResourceManager.Get<IDbConnection>(ConnectionKey); if (conn == null) { conn = new SqlConnection("..."); this.ResourceManager.Set(ConnectionKey, conn); } return conn; } }
我在Repository类中定义了一个全局的ConnectionKey静态字段,并且在初始化时给它指定一个新的GUID。这个字段是readonly的,因此除非应用程序重启,ConnectionKey的值不会改变。而在GetConnection方法中,我把ConnectionKey作为标识,向Resource Manager中获取资源。从GetConnection方法的实现中可以看出,由于我们使用了RequestResourceManager,因此如果是同一个请求的话,就会共享相同的数据库连接对象。
于是,我们的代码里就可以这么写:
public void DoSomething() { var userRepository = new UserRepository(); // 使用userRepository var articleRepository = new ArticleRepository(); // 使用articleRepository }
在使用userRepository的时候,ResourceManager里还没有数据库连接,因此UserRepository会创建一个新的Connection对象。而在使用articleRepository的时候,它会发现ResouceManager中已经存在一个Connection对象了,于是便可取出继续使用。而这个连接会在请求结束时自动释放。
这就是我这个设计的目的。但是,在实现的过程中我犯了一个严重的错误,您发现了吗?(结论)