ASP.NET Core MVC视图模型最佳实践：控制器传入构造函数是否可行？

这种做法属于不良实践，一定要避免，核心问题出在以下几点：

• 严重违反MVC职责分离原则：视图模型的核心职责是为视图提供结构化的数据，它完全不需要知道控制器的存在。直接依赖控制器会让两个组件的职责边界模糊，变成强耦合。
• 维护成本飙升：后续修改控制器的内部属性、方法时，很可能会牵一发而动全身，影响到视图模型的逻辑，排查问题的范围会变得很大。
• 可测试性极差：测试视图模型时，你必须先实例化一个完整的控制器实例，而控制器往往依赖HTTP上下文、各种服务等，这会让测试代码变得臃肿且难以维护。

正确的替代方案 #

应该由控制器主动提取视图模型需要的属性，再传递给它，而不是把整个控制器塞进去。举两个常见的实现方式：

方式1：直接设置视图模型的属性

public class SomeController : Controller {
    public IActionResult Index() {
        var vm = new ViewModel {
            UserId = User.Identity.Name,
            PageTitle = "首页",
            // 把控制器中需要的属性逐个传递
        };

        return View("Index", vm);
    }
}

public class ViewModel
{
    public string UserId { get; set; }
    public string PageTitle { get; set; }
    // 对应视图需要的属性
}

方式2：通过构造函数参数传递（更适合不可变属性）

public class ViewModel
{
    public ViewModel(string userId, string pageTitle) {
        UserId = userId;
        PageTitle = pageTitle;
    }

    public string UserId { get; }
    public string PageTitle { get; }
}

// 控制器中实例化
public class SomeController : Controller {
    public IActionResult Index() {
        var vm = new ViewModel(User.Identity.Name, "首页");
        return View("Index", vm);
    }
}

如果控制器里的属性是来自外部服务，也应该在控制器中先获取到数据，再传递给视图模型——始终保持视图模型的纯粹性，只负责承载视图需要的数据。

内容的提问来源于stack exchange，提问作者Duncan