帮助 Windsor 和存储库以及工作单元模式

Question

我有这些接口：

public interface IUnitOfWork
{
    IPersonRepository People { get; }
    IBookRepository Books { get; }
    int Commit();
}

public interface IBookRepository
{
    Book GetBookById(int id);
    IQueryable<Book> GetAllBooks();
}

public interface IPersonRepository
{
    Person GetPersonById(int id);
    IQueryable<Person> GetAllPeople();
}

我实现 IUnitOfWork 如下：

public class SqlUnitOfWork : IUnitOfWork
{
    private readonly DbContext dbContext;

    public SqlUnitOfWork()
    {
        dbContext = new DbContext("name=SQLContainer");
    }

    public IPersonRepository People
    {
        get { return IoC.Container.Resolve<IPersonRepository>(new { DbContext = dbContext }); }
    }

    public IBookRepository Books
    {
        get { return IoC.Container.Resolve<IBookRepository>(new { DbContext = dbContext }); }
    }

    public int Commit()
    {
        return dbContext.SaveChanges();
    }
}

IBookRepository 和 IPersonRepository 的实现使用采用 DbContext 的构造函数code> 作为参数，并且此 DbContext 在 SqlUnitOfWork（上面的代码）中创建，并且我使用 Resolve 方法的重载传递此参数。

我的问题是，这是正确的方法吗？这是一个好的做法吗？

谢谢！

Answer 1

使用 DI 容器作为服务定位器很难说是一个好的实践。除此之外，在解析接口时将DbContext传递给容器是一个泄漏抽象，因为这意味着您知道一些您不应该知道的具体实现。

相反，我会推荐构造函数注入，它会像这样：

public class SqlUnitOfWork : IUnitOfWork
{
    private readonly DbContext dbContext;
    private readonly IPersonRepository personRepository;
    private readonly IBookRepository bookRepository;

    public SqlUnitOfWork(DbContext dbContext,
         IPersonRepository personRepository, IBookRepository bookRepository)
    {
        if (dbContext == null)
            throw new ArgumentNullException("dbContext");
        if (personRepository == null)
            throw new ArgumentNullException("personRepository");
        if (bookRepository = null)
            throw new ArgumentNullException("bookRepository");

        this.dbContext = dbContext;
        this.personRepository = personRepository;
        this.bookRepository = bookRepository;
    }

    public IPersonRepository People
    {
        get { return this.personRepository; }
    }

    public IBookRepository Books
    {
        get { return this.bookRepository; }
    }

    public int Commit()
    {
        return this.dbContext.SaveChanges();
    }
}

即使没有显式共享DbContext，也可以通过容器进行配置。由于此问题的上下文表明 Castle Windsor 是正在使用的容器，因此默认生命周期已经是 Singleton，因此您不必显式设置它。使用 Castle Windsor，DbContext 将自动在 SqlUnitOfWork 类和两个存储库之间共享。

但是，您也可以显式配置要共享的上下文，如下所示：

container.Register(Component.For<DbContext>().LifeStyle.Singleton);

如果您要使用另一个 DI 容器，API 会有所不同，但概念是相同的。

额外信息：我不知道整体上下文是什么，但是如果要在 Web 应用程序中使用，并且 DbContext 是实体框架或 LINQ to SQL 上下文，正确的生命周期配置将是 PerWebRequest，因为这些上下文类都不是线程安全的：

container.Register(Component.For<DbContext>().LifeStyle.PerWebRequest);