将逻辑从 View 移动到 Controller 和 ViewModel

Question

我正在移动所有定义，如下所示：

 @(Model.Store.EmployeeType.Name == "Manager" ? Model.Store.HQ.Manager.DisplayName : Model.Store.Manager.DisplayName )

走出我的视野并进入viewModel：

public class ManagerViewModel 
{
    public string Manager {get;set;}
}

并在控制器中定义它们：

var viewModel = new ManagerViewModel();

viewModel.Manager = Model.Store.EmployeeType.Name == "Manager" ? Model.Store.HQ.Manager.DisplayName : Model.Store.Manager.DisplayName;

return View(viewModel);

现在，在我看来，我可以这样做：

@Model.Manager

我的问题是 - 这是否违反了瘦控制器的最佳实践？我有大约 30 个字段需要这种类型的处理，所以我的控制器会非常大 - 我正在为每个字段创建一个新属性。

Answer 1

不要太担心premature optimization。我认为您在这方面走在正确的道路上，不必担心太多。

如果您真的担心“瘦”控制器，可以从构造函数中填充 ViewModel。

public class ManagerViewModel 
{

    public ManagerViewModel(ManagerModel model){
        // initialize in here
        this.Manager = model.Store.EmployeeType.Name == "Manager" ? model.Store.HQ.Manager.DisplayName : model.Store.Manager.DisplayName;
    }

    public string Manager {get;set;}
}

var viewModel = new ManagerViewModel(model);
return View(viewModel);