代码改变世界

我犯了一个错误,您能指出吗?

2009-09-08 10:01  Jeffrey Zhao  阅读(15519)  评论(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对象了,于是便可取出继续使用。而这个连接会在请求结束时自动释放。

这就是我这个设计的目的。但是,在实现的过程中我犯了一个严重的错误,您发现了吗?(结论