我的动作控制器代码看起来很业余

Question

第一次发帖，

我一直在尝试 MVC abit...我有一个具有多个输入字段的视图，其中一些字段在发帖时可能为空白。

帖子控制器内的操作方法看起来像这样，

public ActionResult Filter(int? id, string firstName, string lastName, bool? isMember)

我一直在使用 DynamicQuery 扩展，该扩展一直在使用，以便在我的数据库上执行动态 Linq 查询，并且我已将其封装在传递给的 Search 对象中用于执行的数据访问层。

但是，我还有一个自定义的 ViewData 对象，该对象被传递回视图以显示输入值和查询结果。

这一切在代码中看起来有点令人讨厌，因为我必须设置搜索对象属性和 ViewDatas。

public ActionResult Filter(int? id, string firstName, string lastName, bool? isMember)  { 
var search = new Search { 
Id = id, 
FirstName = firstName, 
LastName = lastName, 
Member =  isMember 
}; 

var memberViewData = new MemberViewData { 
Id = id, 
FirstName = firstName, 
LastName = lastName, 
Member =  isMember
}; 

memberViewData.Results = _dataRepository.GetMember(search); 

return View("Search", memberViewData); 

}

我是否想得太多了，真的应该将值传递到数据访问层并填充控制器中的 ViewData，还是有我可以使用的更优雅的模式或实践？

抱歉，如果这看起来很垃圾，没有分配人来交流想法和时间来深入研究框架。

Answer 1

使用modelbinder绑定数据

Answer 2

根据您的代码片段，MemberViewData 类除了具有 Search 类的属性外，还具有 Results 属性。 因此，第一步是使 MemberViewData 从 Search 派生并定义一个构造函数，该构造函数接受 Search 实例作为参数并从中分配其基本属性。 接下来我将更改操作方法，如下所示：

public ActionResult Filter(Search search)  
{ 
    return View("Search", new MemberViewData(search) 
    {
        Results = _dataRepository.GetMember(search)
    }); 
}

Answer 3

正如 Tadeusz 提到的，ModelBinder 可以帮助您构建 MemberViewData，这将只留下要获取的结果。

您还可以决定创建一个表示服务，该服务了解如何构建此视图数据对象并简单地委托给它。 不过，我更喜欢这里的模型绑定器方法。