为实体框架查询使用单独的类文件（模型）而不是在控制器本身中编写答案

【问题标题】：Using a separate class file (model) for Entity Framework queries instead of writing in controller itself为实体框架查询使用单独的类文件（模型）而不是在控制器本身中编写
【发布时间】：2020-01-22 06:26:26
【问题描述】：

是否可以将 EF 查询写入模型文件夹内的单独类文件中，而不是将其写入控制器本身。

因为我在 msdn 中看到在控制器本身中编写所有 EF 查询。同时，我也曾在 msdn 中阅读过该控制器应该很短的信息。

使用模型我使用这种方法：

在控制器中：

        public JsonResult SaveStorageLocation(StorageLocations objStorageLocation)
        {
            int Result = 0;
            try
            {
                StorageLocationModel objStorageLocationModel = new StorageLocationModel();
                if (objStorageLocation.Id == Guid.Empty)
                {
                    Result = objStorageLocationModel.AddStorageLocation(objStorageLocation, SessionUserId);
                }
                else
                {
                    Result = objStorageLocationModel.UpdateStorageLocation(objStorageLocation, SessionUserId);
                }
            }
            catch (Exception ex)
            {
                Result = (int)MethodStatus.Error;
            }
            return Json(Result, JsonRequestBehavior.AllowGet);
        }

在模型类中：

public int AddStorageLocation(StorageLocations objStorageLocation, Guid CreatedBy)
        {
            MethodStatus Result = MethodStatus.None;
            int DuplicateRecordCount = db.StorageLocations.Where(x => x.Location.Trim().ToLower() == objStorageLocation.Location.Trim().ToLower()).Count();
            if (DuplicateRecordCount == 0)
            {
                objStorageLocation.Id = Guid.NewGuid();
                objStorageLocation.CreatedBy = CreatedBy;
                objStorageLocation.CreatedOn = DateTime.Now;
                objStorageLocation.ModifiedBy = CreatedBy;
                objStorageLocation.ModifiedOn = DateTime.Now;
                objStorageLocation.Status = (int)RecordStatus.Active;
                db.StorageLocations.Add(objStorageLocation);
                db.SaveChanges();
                Result = MethodStatus.Success;
            }
            else
            {
                Result = MethodStatus.MemberDuplicateFound;
            }
            return Convert.ToInt32(Result);
        }

        public int UpdateStorageLocation(StorageLocations objStorageLocationNewDetails, Guid ModifiedBy)
        {
            MethodStatus Result = MethodStatus.None;
            int DuplicateRecordCount = 
                db.StorageLocations.
                Where(x => x.Location == objStorageLocationNewDetails.Location && 
                x.Id != objStorageLocationNewDetails.Id).Count();
            if (DuplicateRecordCount == 0)
            {
                StorageLocations objStorageLocationExistingDetails = db.StorageLocations.Where(x => x.Id == objStorageLocationNewDetails.Id).FirstOrDefault();
                if (objStorageLocationExistingDetails != null)
                {
                    objStorageLocationExistingDetails.Location = objStorageLocationNewDetails.Location;
                    objStorageLocationExistingDetails.ModifiedBy = ModifiedBy;
                    objStorageLocationExistingDetails.ModifiedOn = DateTime.Now;
                    objStorageLocationExistingDetails.Status = (int)RecordStatus.Active;
                    db.SaveChanges();
                    Result = MethodStatus.Success;
                }
            }
            else
            {
                Result = MethodStatus.MemberDuplicateFound;
            }
            return Convert.ToInt32(Result);
        }

还是把所有的代码都写在控制器里面比较好？

【问题讨论】：

标签： c# entity-framework code-maintainability

【解决方案1】：

我希望您的问题很快就会结束，因为它将被视为基于意见。

除此之外，如果您的控制器中没有查询，则有很多优势。

控制器不应该规定如何访问数据，这是模型的工作。
如果您将模型（或服务或存储库，无论您想如何称呼它为应用程序）作为依赖项注入，模拟数据访问会容易得多。
稍后您可能会发现，如果将某些查询迁移到存储过程，则可以更好地处理它们，因为它们会处理大量数据。如果控制器不直接访问数据存储，则此更改将更容易进行。您可以在模型类中进行更改并保持相同的接口，或者编写一个新的实现然后注入。
可以更轻松地相互隔离地测试控制器和模型。

【讨论】：

【解决方案2】：

您希望您的代码始终是可测试的。

永远不要把逻辑放在你的模型中，把逻辑项放在模型文件夹中会使你的结构变脏并且更容易失去概览。

您应该使用实现接口的存储库类。在repository类中可以执行EF逻辑，捕获数据库异常等等。

您的控制器将被注入存储库接口。通过这种方式，您可以将控制器与 EF 逻辑分开测试，因为您可以模拟该接口。您的存储库也可以对其开放功能进行测试。

【讨论】：