如何避免在 MVC4 中使用 ViewBag 在控制器和视图之间传递查询值

Question

我读到它是generally a bad practice to use ViewBags，而且混合 ViewBag 和 ViewModel 也不好，但我使用一些 ViewBags 在我的控制器和视图之间传递 sort order 参数，尽管我可以在 @987654323 上创建属性@ 对于视图，我首先需要这些值才能创建模型，所以看不到如何解决这个问题。

控制器

public ActionResult Index(string sortOrder, string searchString, 
                          string currentFilter, int? page, 
                          bool? includeComplete)
{

 ViewBag.CurrentSort = sortOrder;
 ViewBag.NameSortParam = string.IsNullOrWhiteSpace(sortOrder) ? "Name desc" : "";
 ViewBag.DateSortParam = sortOrder == "Date" ? "Date desc" : "Date";

 if (Request.HttpMethod == "GET")
     searchString = currentFilter;
 else
     page = 1;

 ViewBag.CurrentFilter = searchString;
 bool showCompleted = (includeComplete == null || includeComplete == false)
                        ? false : true;
 ViewBag.IncludeCompleted = showCompleted;

 int pageNumber = (page ?? 1);

 var query = Session.QueryOver<ToDo>();

 if (!string.IsNullOrWhiteSpace(searchString))
     query = query.WhereRestrictionOn(td => td.TaskName)
                      .IsInsensitiveLike(string.Format("%{0}%", searchString));

 if (!showCompleted)
     query.And(td => td.IsComplete == false);

 switch (sortOrder)
 {
    case "Name desc":
      query = query.OrderBy(td => td.TaskName).Desc;
      break;
        case "Date":
      query = query.OrderBy(td => td.DueDate).Asc;
      break;
    case "Date desc":
      query = query.OrderBy(td => td.DueDate).Desc;
      break;
    default:
      query = query.OrderBy(td => td.TaskName).Asc;
      break;
 }

 var result = query.Fetch(p=>p.Priority).Eager
           .Fetch(s=>s.Staff).Eager
           .List();

 var viewModel = AutoMapper.Mapper.Map<IEnumerable<ToDo>, 
                     IEnumerable<IndexToDoViewModel>>(result)
                     .ToPagedList(pageNumber, PageSize);
 return View(viewModel);

}

然后在我看来，我将使用 ViewBag 将值传递回控制器，例如我需要拥有的 sortOrder，然后才能将它应用到构成方法主体的 NHibernate 查询中，例如：

@Html.ActionLink("Name", "Index", new {sortOrder=ViewBag.NameSortParam, 
                                      includeComplete = ViewBag.IncludeCompleted})

我的视图模型

public class IndexToDoViewModel
{
    [DataType(DataType.Date)]
    [DisplayFormat(DataFormatString = "{0:dd MMM yyyy}")]
    [DisplayName("Date Due")]
    public DateTime? DueDate { get; set; }

    public Guid Id { get; set; }

    [DisplayName("Is Complete")]
    public bool IsComplete { get; set; }

    [DataType(DataType.MultilineText)]
    public String Notes { get; set; }

    public string Priority { get; set; }

    public string Staff {get; set;}

    [DisplayName("Name")]
    public String TaskName { get; set; }

    // These would potentially replace my ViewBag?
    public string CurrentSort { get; set; }
    public string CurrentFilter { get; set; }
    public string NameSortParam { get; set; }
    public string DateSortParam { get; set; }
    public bool IncludeCompleted { get; set; }

}

Answer 1

然后在我看来，我将使用 ViewBag 将值传递回 控制器

不，您将值传递给下一个控制器。当控制器的新实例将处理请求时，这些值将（可能）在浏览器往返后使用。

由于您没有将任何内容传递回当前控制器，因此无需使用ViewBag。如果您愿意，可以将值放入模型中。

---

即使您将数据从视图发送到控制器（这是非常不寻常的），使用模型仍然可以实现。示例：

MyModel viewModel = new MyModel();
ActionResult result = View(viewModel);

// here you can access anything that the view would put in the model

return result;

当然，您不能使用控制器中视图中的任何值来创建模型，因为您必须在视图之前创建模型。如果您需要视图中的任何数据来为其获取数据，您可以将其作为方法放入模型而不是控制器中。

Answer 2

我建议为您创建的每个视图创建一个 ViewModel，因为您很可能需要将更多信息传递给您的视图。

在这种情况下，您将拥有一个具有分页列表数据属性的 ViewModel，并且您可以为 SortOrder 添加另一个属性。

例如：

public class MyViewModel
{
    public IEnumerable<ToDo> TodoList { get; set; }
    public string SortOrder { get; set; }
}

查看 SO 问题：ViewModel Best Practices