实体框架的读/写方法约定。控制器与 DbContext？

Question

我正在为我正在构建的网站编写大量基本的 CRUD 视图和控制器逻辑。到目前为止，我已经在控制器中编写了大部分代码，这包括通常的验证、输入清理和错误处理。我应该在我的控制器中编写我所有的 DB I/O 代码吗？或者我应该将一些移到我的 DbContext 吗？我问这个，因为我听说过控制器类与模型类之间应该发生多少冲突的观点？将 Db 上下文的实例传递出控制器是否合适？或者我应该在 DbContext 上使用扩展类吗？

例如：

public ActionResult Create(ThingViewModel vModel)
{
    try
    {
        if (ModelState.IsValid)
        {
            var nm = vModel.ToActualModel();

            nm.RelatedThing = nm.RelatedThingId == null ? 
                null : db.RelatedThings.Single(v => v.Id == nm.RelatedThingId);
            nm.UtcCreatedOn = DateTime.UtcNow;

            db.Thing.Add(nm);
            db.SaveChanges();

            var successMessage = "You have created a new Thing!";
            return RedirectToAction("Index", new { successMessage = successMessage });
        }
        else
        {
            ViewBag.EntityName = "Thing";
            ViewBag.ControllerName = "Thing";
            ViewBag.Title = "Admin | Thing - Create";

            return View("~/Views/Thing/Create.cshtml", vModel);
        }
    }
    catch(Exception e)
    {
        var errorMessage = "An error occured when creating a new thing!";
        return RedirectToAction("Index", new { errorMessage = errorMessage });
    }
}

这应该变成：

public ActionResult Create(ThingViewModel vModel)
{
    try
    {
        if (ModelState.IsValid)
        {
            db.CreateNewThing(vModel) // Defined elsewhere

            var successMessage = "You have created a new Thing!";
            return RedirectToAction("Index", new { successMessage = successMessage });
        }
        else
        {
            ViewBag.EntityName = "Thing";
            ViewBag.ControllerName = "Thing";
            ViewBag.Title = "Admin | Thing - Create";

            return View("~/Views/Thing/Create.cshtml", vModel);
        }
    }
    catch(Exception e)
    {
        var errorMessage = "An error occured when creating a new thing!";
        return RedirectToAction("Index", new { errorMessage = errorMessage });
    }
}

在 DbContext 内部：

 public bool CreateNewThing(ThingViewModel vModel)
 {
     //Thing Creation logic
     Things.Add(thing);
     SaveChanges();
 }

为了澄清，我想为每个实体编写一次创建/编辑/删除逻辑，并能够在其他控制器中使用它们。因此，如果我有Person 和Pet 实体以及PersonController 和PetController，则有时需要将包含List<PetViewModel> 的PersonViewModel 写入数据库：Person 需要与每个Pet 一起创建。但是，PetController 已经定义了public ActionResult Create(PetViewModel vm)，但我不能从PersonController 内部使用它来将新的Pet 写入数据库。所以，我想将Create(PetViewModel vm) 的数据库逻辑移到其他地方，我可以从其他控制器内部访问它。我要把它移到哪里？将我的DbContext 的引用从控制器传递给静态辅助方法public static bool CreateHelper(DbContext db, PetViewModel vm) 是否合适？

Answer 1

第一个是正确的。请记住，它是一个 MVC 模式 - 模型 - 视图 - 控制器。您的控制器正在做它应该做的事情，包括使用 dbcontext 从数据库中检索信息（这反过来又在做它应该做的事情）。它仍然遵循分离关注点的良好 MVC 实践，但不会以过度复杂化为代价。几年前，当我觉得“正确”比使用我的常识更重要时，我很难学会这一点。将 CreateNewThing 放在 db 上下文中不会给您带来真正的好处，而且在更大更复杂的应用程序中只会使水变得混乱。

Answer 2

我认为在 MVC 项目中创建不一定是控制器的辅助类是很好的，但它不应该与您的数据层混合。 db.CreateNewThing(vModel) 应该避免。 db 用于数据存储和检索，而不是用于转换模型或做类似的事情。

Answer 3

您可以使用的一种模式是command pattern。这样您就可以将业务逻辑与控制器分离。

另见：Command, CommandHandler and CommandInvoker

样板：

public interface ICommandResponse
{
    bool Success { get; }
    IReadOnlyCollection<string> Errors { get; }
}

public class CommandResponse : ICommandResponse
{
    public bool Success { get; set; }
    public IReadOnlyCollection<string> Errors { get; set; }
}

public interface ICommand<in TModel>
{
    ICommandResponse Handle(TModel model);
}

public abstract class CommandBase<TModel> : ICommand<TModel>
{
    public abstract ICommandResponse Handle(TModel model);

    protected ICommandResponse Success()
    {
        return new CommandResponse { Success = true };
    }

    protected ICommandResponse Fail(params string[] errors)
    {
        return new CommandResponse { Errors = new ReadOnlyCollection<string>(errors) };
    }
}

代码：

public class CreateThingCommand : CommandBase<ThingViewModel>
{
    private readonly DbContext _context;

    public CreateThingCommand(DbContext context)
    {
        _context = context;
    }

    public override ICommandResponse Handle(ThingViewModel viewModel)
    {
        var model = viewModel.ToActualModel();

        model.RelatedThing =
            model.RelatedThingId == null
            ? null
            : _context.RelatedThings.Single(v => v.Id == model.RelatedThingId);
        model.UtcCreatedOn = DateTime.UtcNow;

        _context.Thing.Add(model);
        _context.SaveChanges();

        return Success();
    }
}

// Inside controller
public ActionResult Create(ThingViewModel vModel)
{
    try
    {
        if (ModelState.IsValid)
        {
            var result = createThingCommand.Handle(vModel);

            if(result.Success)
            {
                var successMessage = "You have created a new Thing!";
                return RedirectToAction("Index", new { successMessage = successMessage });
            }

            // Handle error
            return RedirectToAction("Index", new { failMessage = "Something went wrong" });
        }

        ViewBag.EntityName = "Thing";
        ViewBag.ControllerName = "Thing";
        ViewBag.Title = "Admin | Thing - Create";

        return View("~/Views/Thing/Create.cshtml", vModel);
    }
    catch(Exception e)
    {
        var errorMessage = "An error occured when creating a new thing!";
        return RedirectToAction("Index", new { errorMessage = errorMessage });
    }
}