c# – 存储库模式 – 使其可测试,DI和IoC友好和IDisposable

public interface IRepository
{
   Idisposable CreateConnection();
   User GetUser();
   //other methods,doesnt matter
}

public class Repository
{
   private sqlConnection _connection;

   Idisposable CreateConnection()
   {
      _connection = new sqlConnection();
      _connection.open();
      return _connection;
   }

   User GetUser()
   {
      //using _connection gets User from Database
      //assumes _connection is not null and open
   }
   //other methods,doesnt matter 
}

using(_repository.CreateConnection())
{
    var user = _repository.GetUser();
    //do something with user
}

您的设计中存在一些问题点.首先,您的IRepository接口实现了多个级别的抽象.创建用户是一个比连接管理更高级别的概念.通过将这些行为放在一起,你打破了 Single Responsibility Principle,它决定了一个班级应该只有一个责任,一个改变的理由.你也违反了推动我们走向狭窄角色界面的 Interface Segregation Principle.

最重要的是,CreateConnection()和GetUser方法是时间耦合的. Temporal Coupling是代码气味,你已经看到这是一个问题,因为你可以忘记对CreateConnection的调用.

除此之外,您将开始在系统中的每个存储库中看到连接的创建,并且每个业务逻辑都需要创建连接或从外部获取现有连接.从长远来看,这变得无法维持.然而,连接管理是一个贯穿各领域的问题;你不希望业务逻辑关注这种低级别的问题.

您应该首先将IRepository分成两个不同的接口：

public interface IRepository
{
    User GetUser();
}

public interface IConnectionFactory
{
    Idisposable CreateConnection();
}

您可以在更高级别管理事务,而不是让业务逻辑管理连接本身.这可能是请求,但这可能过于粗糙.您需要的是在表示层代码和业务层代码之间的某处启动事务,而不必自己复制.换句话说,您希望能够透明地应用这种横切关注点,而无需反复写入.

这是我几年前开始使用应用程序设计的众多原因之一,其中业务操作是使用消息对象定义的,其相应的业务逻辑隐藏在通用接口之后.应用这些模式后,您将拥有一个非常明确的拦截点,您可以在其中启动与其相应连接的事务,并让整个业务操作在同一事务中运行.例如,您可以使用以下通用代码,这些代码可以应用于应用程序中的每个业务逻辑：

public class TransactionCommandHandlerDecorator<TCommand> : ICommandHandler<TCommand>
{
    private readonly ICommandHandler<TCommand> decorated;    
    public TransactionCommandHandlerDecorator(ICommandHandler<TCommand> decorated) {
        this.decorated = decorated;
    }

    public void Handle(TCommand command) {
        using (var scope = new TransactionScope()) {
            this.decorated.Handle(command);
            scope.Complete();
        }
    }   
}

此代码包装TransactionScope周围的所有内容.这允许您的存储库只是打开和关闭连接;这个包装器将确保使用相同的连接.这样,您可以将IConnectionFactory抽象注入到您的存储库中,并让存储库在其方法调用结束时直接关闭连接,而在.NET下将保持打开实际连接.