在 ASP.NET MVC 3 中使用视图模型

Question

我对查看模型还比较陌生，并且在使用它们时遇到了一些问题。在一种情况下，我想知道最佳实践是什么......

我将视图所需的所有信息放入视图模型中。这是一个例子——请原谅任何错误，这是我凭空想象出来的。

public ActionResult Edit(int id)
{
    var project = ProjectService.GetProject(id);

    if (project == null)
        // Something about not found, possibly a redirect to 404.

    var model = new ProjectEdit();
    model.MapFrom(project); // Extension method using AutoMapper.

    return View(model);
}

如果屏幕只允许编辑一两个字段，那么当视图模型返回时，它会丢失相当多的数据（理应如此）。

[HttpPost]
public ActionResult Edit(int id, ProjectEdit model)
{
    var project = ProjectService.GetProject(id);

    if (project == null)
        // Something about not found, possibly a redirect to 404.

    try
    {
        if (!ModelState.IsValid)
            return View(model) // Won't work, view model is incomplete.

        model.MapTo(project); // Extension method using AutoMapper.
        ProjectService.UpdateProject(project);
        // Add a message for the user to temp data.

        return RedirectToAction("details", new { project.Id });
    }
    catch (Exception exception)
    {
        // Add a message for the user to temp data.

        return View(model) // Won't work, view model is incomplete.
    }
}

我的临时解决方案是从头开始重新创建视图模型，从域模型重新填充它，将表单数据重新应用到它，然后正常进行。但这使得视图模型参数有些毫无意义。

[HttpPost]
public ActionResult Edit(int id, ProjectEdit model)
{
    var project = ProjectService.GetProject(id);

    if (project == null)
        // Something about not found, possibly a redirect to 404.

    // Recreate the view model from scratch.
    model = new ProjectEdit();
    model.MapFrom(project); // Extension method using AutoMapper.

    try
    {
        TryUpdateModel(model); // Reapply the form data.

        if (!ModelState.IsValid)
            return View(model) // View model is complete this time.

        model.MapTo(project); // Extension method using AutoMapper.
        ProjectService.UpdateProject(project);
        // Add a message for the user to temp data.

        return RedirectToAction("details", new { project.Id });
    }
    catch (Exception exception)
    {
        // Add a message for the user to temp data.

        return View(model) // View model is complete this time.
    }
}

有更优雅的方式吗？

编辑

两个答案都是正确的，所以如果可以的话我会奖励他们两个。然而，经过反复试验，我发现他的解决方案是最精简的，所以我向 MJ 点头。

我仍然可以使用助手，吉米。如果我将需要显示的内容添加到视图包（或视图数据）中，就像这样...

ViewBag.Project= project;

然后我可以执行以下操作...

@Html.LabelFor(model => ((Project)ViewData["Project"]).Name)
@Html.DisplayFor(model => ((Project)ViewData["Project"]).Name)

有点黑客，它需要用 < 装饰域模型在某些情况下，code>System.ComponentModel.DisplayNameAttribute，但我已经这样做了。

我很想打电话...

@Html.LabelFor(model => ViewBag.Project.Name)

但是动态会导致表达式出现问题。

Answer 1

经过一些反复试验（又名编码，然后讨厌它）学习，我目前首选的方法是：

我仅使用视图模型来绑定输入字段。因此，在您的情况下，如果您的视图仅编辑两个字段，那么您的视图模型将只有两个属性。对于填充视图（下拉列表、标签等）所需的数据，我使用动态 ViewBag。

我相信显示视图（即填充视图需要显示的任何内容）和捕获发布的表单值（绑定、验证等）是两个独立的问题。我发现将填充视图所需的数据与从视图发回的数据混合在一起会变得很混乱，并且常常会准确地创建您的情况。我不喜欢部分填充的对象被传递。

我不确定 Automapper（用于将域对象映射到动态 ViewBag）如何发挥作用，因为我还没有使用过它。我相信它有一个可以工作的 DynamicMap 方法？将发布的强类型 ViewModel 自动映射到 Domain 对象上应该不会有任何问题。

Answer 2

如果我理解正确，您的视图模型可能看起来与您的域实体非常相似。您提到视图模型可能会返回大部分为空，因为只有某些字段是可编辑的。

假设您的视图中只有少数字段可用于编辑（或显示），那么这些是您应该在视图模型中提供的唯一字段。我通常为每个视图创建一个视图模型，并让控制器或服务处理用户的输入，并在执行一些验证后将其与域实体映射回来。

这里有一个关于视图模型最佳实践的主题，您可能会觉得有用。

编辑：您还可以在“编辑/POST”操作中接受与“编辑/GET”操作所提供的不同的视图模型。我相信只要模型绑定器能够弄清楚，这就应该有效。