ASP.NET MVC最佳实践：为模型加[NotMapped]字段还是创建视图模型？

我有WPF MVVM开发经验，现在转向ASP.NET MVC开发。我的User模型包含一个Message子模型，结构如下：

public class User
{
    public int Id { get; set; }

    [Required]
    [Display(Name = "Register Date")]
    public DateTime RegisterDate { get; set; }

    [Required]
    [StringLength(100)]
    public string Username { get; set; } = string.Empty;
    
    [Required]
    public Role Role { get; set; }

    public List<Message> Messages { get; set; } = new();
}

我需要在首页展示用户的最新消息。在WPF中我会创建视图模型来暴露这个属性供视图使用，现在在MVC里想到两种实现方式：

方式一：通过控制器映射到视图模型 #

public async Task<IActionResult> Index()
{
    var users = await _context.Users
        .Include(u => u.Messages)
        .ToListAsync();

    var vm = users.Select(u => new UserListViewModel
    {
        Id = u.Id,
        RegisterDate = u.RegisterDate,
        Username = u.Username,
        Role = u.Role,
        LatestMessage = u.Messages?.OrderByDescending(m => m.Timestamp)
            .FirstOrDefault()?.Content ?? string.Empty
    });

    return View(vm);
}

方式二：在实体模型中添加[NotMapped]属性 #

public class User
{
    public int Id { get; set; }

    [Required]
    [Display(Name = "Register Date")]
    public DateTime RegisterDate { get; set; }

    [Required]
    [StringLength(100)]
    public string Username { get; set; } = string.Empty;
    
    [Required]
    public Role Role { get; set; }

    public List<Message> Messages { get; set; } = new();

    [NotMapped]
    public string LatestMessage => 
        Messages?.OrderByDescending(m => m.Timestamp)
        .FirstOrDefault()?.Content ?? string.Empty;
}

想请教在ASP.NET MVC中哪种方式是更佳实践，或者有没有更好的解决方案？

优先选择视图模型方案（方式一） #

这是ASP.NET MVC中的标准实践，也完全契合你熟悉的关注点分离原则：

• 实体模型职责单一：User作为领域实体，只承载核心业务数据和领域逻辑，视图相关的计算属性不会混入其中，避免实体模型臃肿，防止业务逻辑与视图需求耦合。
• 视图模型精准适配视图：UserListViewModel专门为首页展示需求定制，只包含视图需要的字段，避免传递冗余数据，后续可灵活添加其他视图专属逻辑，不会影响实体模型的其他使用场景。
• 性能优化空间更大：当前代码用Include加载了所有Messages，可进一步优化为直接在EF Core查询中仅获取最新一条消息，避免加载全部消息数据：

public async Task<IActionResult> Index()
{
    var usersWithLatestMessage = await _context.Users
        .Select(u => new UserListViewModel
        {
            Id = u.Id,
            RegisterDate = u.RegisterDate,
            Username = u.Username,
            Role = u.Role,
            LatestMessage = u.Messages
                .OrderByDescending(m => m.Timestamp)
                .Select(m => m.Content)
                .FirstOrDefault() ?? string.Empty
        })
        .ToListAsync();

    return View(usersWithLatestMessage);
}

这种写法无需Include，EF Core会自动生成高效SQL，只返回所需数据，性能远优于加载全部消息。

方式二的适用场景 #

方式二并非完全不可取，但仅适合非常简单、全局通用的视图需求：

• 如果LatestMessage在多个视图中都需要展示，且逻辑固定，不想重复编写映射代码，可考虑在实体中添加[NotMapped]属性。
• 但要注意：这种方式会让实体模型依赖视图需求，一旦视图需求变化（比如要展示最新消息的时间而非内容），就需要修改实体模型，违反单一职责原则；另外如果忘记Include加载Messages，该属性会返回空值，易引发潜在bug。

其他优化思路 #

如果项目规模较大，可引入AutoMapper简化实体到视图模型的映射，减少手动编写映射代码的工作量：

1. 定义UserListViewModel和映射规则
2. 在查询中直接投影到视图模型，或查询实体后用AutoMapper转换

不过对于简单场景，手动映射已足够，无需过度引入第三方库。

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