asp.net-mvc - 将Using Blocks放入存储库中有什么问题？

Question

我在存储库的每个方法中都有使用 block 。如果我想交叉引用方法，似乎初始化另一个 Datacontext 是违反最佳实践的，我做错了什么？如果我在类中声明 Datacontext 而不是在方法中使用 block ，我不会失去处理能力吗？？

public IList<something> GetSomething()
{ 
   using (DB db=new DB())

   { ...GetListofSomethingElse(id)

   } 
}

public IList<somethingelse> GetListofSomethingElse(int id)
{ 
    using (DB db=new DB())
   {
     ... return IList 

   } 
}

Answer 1

实际上，我认为在语义上(或者我该怎么说)在存储库中创建和处置数据上下文是不正确的。

我的意思是:如果您在存储库的每个方法中打开与数据库的新连接，恕我直言，您就做错了。这太细粒度了。存储库类不知道它所使用的“上下文”。您的存储库不应负责打开/关闭连接或启动和提交事务。上下文为王，存储库不知道它所使用的上下文。因此，恕我直言，打开新的 DataContext 对象并关闭/处置它们是应用程序层或服务层的责任。 (这同样适用于交易)。

所以，我是这样做的:(注意，我不使用 Entity Framework ，但我使用 NHibernate。我假设 EF 中的 DataContext 类类似于 NHibernate 中的 ISession):

using( ISession s = theSessionFactory.OpenSession() )
{
    ICustomerRepository cr = RepositoryFactory.GetCustomerRepository(s);

    Customer c1 = cr.GetCustomer(1);
    Customer c2 = cr.GetCustomer(2);

    // do some other stuff
    s.StartTransaction();

    cr.Save (c1);
    cr.Save (c2);

    s.Commit();

}

(这当然不是现实世界的代码；而且它甚至无法编译，因为 ISession 没有 Commit 方法。;)相反，StartTransaction 返回一个具有某种 commit 方法的 ITransaction，但是我想你会明白我的意思的。 ;) )