注入存储库上的 IDisposable

我有以下 ADO .Net 存储库

public class Repository : IRepository, IDisposable
{
   private readonly IUnitOfWork UnitOfWork;
   private SqlConnection Connection;

   public Repository(IUnitOfWork unitOfWork, connectionString)
   {
      UnitOfWork = unitOfWork;
      Connection = new SqlConnection(connectionString);
      Connection.Open();
   }

   public MyObject FindBy(string userName)
   {
      //...Ado .Net command.ExecuteReader, etc.
   }
}

该存储库通过 IoC 容器注入到域服务中，并按如下方式使用：

public class UserDomainService : IUserDomainService
{
   private readonly IRepository Repository;

   public UserDomainService(IRepository repository)
   {
      Repository = repository;
   }

   public User CreateNewUser(User user)
   {
      using(Repository)
      {
         var user = Repository.FindBy(user.UserName);
         if(user != null)
            throw new Exception("User name already exists!");

         Repository.Add(user);
         Repository.Commit();
      }
   }
}

我的想法是，我总是将 Repository 对象放在 using 语句中，因此当它完成时，连接将关闭并被释放，但我认为这是一个问题，因为域服务类仍然存在，并且如果有第二次调用它，它将失败，因为存储库已被销毁。

现在我可以完全控制所有代码，并且我只想设计粗粒度服务调用，但整个事情感觉不太对劲。

我这样做是为了避免域服务知道存储库中的 OpenConnection 和 CloseConnection 方法。

这种设计本质上是不好的还是有更好的方法来做到这一点？

经过深思熟虑：当请求到达时，所有依赖关系树都会在 WCF 级别生成，当然您可以看到连接在那一刻打开，因为它发生在存储库的构造函数中，所以我相信它并没有那么糟糕，因为它是仅在该特定通话期间打开。我的这个假设是正确的还是我在过程中这么早就打开数据库连接做了一些非常糟糕的事情？

注入一个创建您需要的实例的工厂，而不是实例本身。

Take an IRepositoryFactory这样你就可以创建一个IRepository并在每次使用时将其丢弃。这样，域服务或工厂都不需要是一次性的。另外，重要的是，您仍然可以通过注入实现来保持代码抽象，而不是对其进行硬编码。

public class UserDomainService : IUserDomainService
{
   private readonly IRepositoryFactory RepositoryFactory;

   public UserDomainService(IRepositoryFactory factory)
   {
      RepositoryFactory = factory;
   }

   public User CreateNewUser(User user)
   {
      using (IRepository repository = RepositoryFactory.Create())
      {
         var user = repository.FindBy(user.UserName);
         if(user != null)
            throw new Exception("User name already exists!");

         repository.Add(user);
         repository.Commit();
      }
   }
}

您并不总是需要注入您需要的类型。在阅读 Castle Windsor（其思维方式是注册-解析-发布）时，您会发现如果您想在应用程序生命周期中的不确定时间解决问题，建议使用类型工厂。

您知道您需要一个存储库，但不知道when。与其请求存储库，不如请求一些东西creates他们。因此，抽象级别得以维持，并且您没有泄漏任何实现。