Question

我已经阅读了有关此主题的各种帖子，但无法找到我的问题的答案。

普遍的共识是我应该为1个工作单元（比如1个网页）创建一个上下文。我已经在我的Database.cs（见下文）中附上了＆＃39;使用＆＃39;因此对我来说 - 这意味着每次调用此类的方法时 - 都会创建一个上下文。因此，如果我要从HomeController.cs中的同一个Action调用Database.cs中的2个方法 - 这是否意味着创建了2个上下文？

在Database.cs中声明私有字段会不会更好：

private Entities db = new Entities()

并且让Database.cs类中的每个方法都可以访问它吗？哪种方法最好？

我当前的实现（我只会包含方法验证，但Database类中有很多方法）：

HomeController.cs

    [AllowAnonymous]
    public class HomeController : Controller
    {
        private IDatabase Database;

        public HomeController()
        {
            this.Database = new Database();
        }

        [HttpGet]
        public ActionResult Verify(string id)
        {
            if (Database.VerifyUser(id))
            {
                return View();
            }
            else
            {
                ViewBag.Error = "There was an error with the verification process";
                return View();
            }
        }
    }

Database.cs

public class Database : IDatabase
{
... some other methods ...
    public bool VerifyUser(string verificationHash)
    {
        using (Entities db = new Entities())
        {
            var userToVerify = db.VerifyUser(verificationHash);
            int count = userToVerify.Count();
            if (count == 1)
            {
                return true;
            }
            else
            {
                return false;
            }
        }
    }
}

db.VerifyUser（..） - 这是对存储过程的调用

Answer 1

是的，这意味着有两个DbContext实例。更好的方法是在Database类中使用一个DbContext实例，并在所有方法中使用此实例。

public class Database : IDatabase, IDisposeable
{
    private Entities db;
    public Database()
    {
        db = new Entities()
    }

... some other methods ...
    public bool VerifyUser(string verificationHash)
    {
        var userToVerify = db.VerifyUser(verificationHash);
        int count = userToVerify.Count();
        if (count == 1)
        {
            return true;
        }
        else
        {
            return false;
        }        
    }

    public void Dispose()
    {
        db.Dispose()
    }
}

然后当你从数据库实例完成后，你将它处理它并将处理DbContext

public class HomeController : Controller
{
    private IDatabase Database;

    public HomeController()
    {
        this.Database = new Database();
    }

    [HttpGet]
    public ActionResult Verify(string id)
    {
        using(this.Database)
        {
            if (Database.VerifyUser(id))
            {
                return View();
            }
            else
            {
                ViewBag.Error = "There was an error with the verification process";
                return View();
            }
        }
    }   
}

BTW：您可能更愿意在控制器级别处置资源。在这种情况下，您不需要在操作中添加using语句例如将此添加到您的控制器：

protected override void Dispose(bool disposing)
{
    this.Database.Dispose();
    base.Dispose(disposing);
}

Answer 2

在您的设计中是的DbContext在每个方法调用中创建和处理。

实际上，将所有数据库操作放到类中并反复创建DbContext并不是一个好的解决方案。您可能在将来遇到该课程时遇到问题。它可能有数百个方法及时，因此很难维护，并且所有实体在语义上都没有相互关联，因此可能会引起混淆。我认为将实体类型分成类是更好的解决方案。例如，您有一个用户，项目，部门。如果我们将我的解决方案应用于这些实体，那么uml类图将是这样的。

所有存储库都引用DbContext。它被称为依赖注入。这意味着dbcontext实例化一次并通过必要的存储库传递其引用，因此不会重新创建上下文。还有一个通用的存储库，您可以放置标准过程。

所以你可以使用像这样的存储库。

[HttpGet]
public ActionResult Verify(string id){

   using(var context = new DbContext())
   {
      var userRepo = new UserRepository(context); 
      //Department repository can be used over the same context.
      var departmentRepo = new DepartmentRepository(context);
      if(userRepo.verifyUser(id)){
         return View();
      }
   }

}

MVC EF数据库存储库模式

2 个答案: