EF Core中含数据库查询的计算属性实现及架构位置疑问

首先得说清楚为什么你原来的_context在EF Core里会null：EF Core的实体模型实例一般是由上下文加载或者创建的，但实体本身并不是依赖注入（DI）容器管理的服务，所以你没法直接在实体里注入上下文——这就是为什么原来的写法在EF Core里行不通的核心原因。

接下来咱们聊聊解决方案和架构上的最佳实践：

一、别在实体模型里放依赖数据库查询的计算属性 #

实体模型的核心职责是作为数据库表的映射载体，应该保持简洁，只包含和表结构对应的属性，不要掺杂需要查询数据库的业务逻辑。这样不仅能避免依赖上下文的问题，还能让实体的职责更清晰，后续维护也更方便。

二、合适的替代方案 #

1. 把计算逻辑移到视图模型（ViewModel）中 #

你已经在使用ViewModel了（比如代码里的EmployeeViewModel），这是个非常适合放计算属性的地方。因为ViewModel是为视图定制的，你可以在控制器里拿到上下文，查询所需的数据后，在ViewModel里完成计算。

举个针对你的Quotation的例子：

// 先定义对应的ViewModel
public class QuotationViewModel
{
    public int ID { get; set; }
    public int QuotationReference { get; set; }
    [Display(Name = "Version")]
    public int VersionNo { get; set; }
    public string VersionDisplay { get; set; }
}

// 在控制器里查询并计算
public IActionResult Details(int id)
{
    // 先获取当前报价记录
    var quotation = _context.Quotations.FirstOrDefault(q => q.ID == id);
    if (quotation == null)
    {
        return NotFound();
    }

    // 查询该引用下的最大版本号
    var maxVersion = _context.Quotations
        .Where(q => q.QuotationReference == quotation.QuotationReference)
        .Max(q => q.VersionNo);

    // 组装ViewModel
    var vm = new QuotationViewModel
    {
        ID = quotation.ID,
        QuotationReference = quotation.QuotationReference,
        VersionNo = quotation.VersionNo,
        VersionDisplay = $"{quotation.VersionNo}/{maxVersion}"
    };

    return View(vm);
}

这种方式简单直接，适合逻辑不复杂、不需要复用的场景，而且能保证每次都是实时计算最新数据。

2. 抽离到服务层（Service Layer） #

如果你的计算逻辑需要在多个地方复用，或者业务逻辑比较复杂，建议把这些逻辑抽到专门的服务类里。服务层可以通过DI注入上下文，专注处理业务逻辑，控制器只负责接收请求、调用服务、返回结果，符合单一职责原则。

比如创建一个QuotationService：

public class QuotationService
{
    private readonly AppDbContext _context;

    // 通过DI注入上下文
    public QuotationService(AppDbContext context)
    {
        _context = context;
    }

    public QuotationViewModel GetQuotationWithVersionDisplay(int quotationId)
    {
        var quotation = _context.Quotations.FirstOrDefault(q => q.ID == quotationId);
        if (quotation == null) return null;

        var maxVersion = _context.Quotations
            .Where(q => q.QuotationReference == quotation.QuotationReference)
            .Max(q => q.VersionNo);

        return new QuotationViewModel
        {
            ID = quotation.ID,
            QuotationReference = quotation.QuotationReference,
            VersionNo = quotation.VersionNo,
            VersionDisplay = $"{quotation.VersionNo}/{maxVersion}"
        };
    }
}

然后在控制器里注入这个服务来使用：

private readonly QuotationService _quotationService;

public QuotationsController(QuotationService quotationService)
{
    _quotationService = quotationService;
}

public IActionResult Details(int id)
{
    var vm = _quotationService.GetQuotationWithVersionDisplay(id);
    if (vm == null)
    {
        return NotFound();
    }
    return View(vm);
}

3. 数据库层面的计算列（可选） #

如果你的计算逻辑可以用SQL表达式实现（比如数值计算，或者简单的字符串拼接），可以考虑用EF Core的数据库计算列。这种方式下，计算逻辑由数据库负责，每次查询都会返回最新值。

比如在实体里标记：

public class Quotation
{
    public int ID { get; set; }
    public int QuotationReference { get; set; }
    [Display(Name = "Version")]
    public int VersionNo { get; set; }

    // 标记为数据库计算列
    [DatabaseGenerated(DatabaseGeneratedOption.Computed)]
    public string VersionDisplay { get; private set; }
}

然后需要在数据库迁移里设置计算列的SQL表达式（比如SQL Server用CONCAT(VersionNo, '/', (SELECT MAX(VersionNo) FROM Quotations WHERE QuotationReference = q.QuotationReference))）。不过这种方式局限性比较大，复杂逻辑很难用SQL实现，而且逻辑放在数据库里，后续维护不如代码直观。

三、关于你控制器代码的小建议 #

你在Create方法里手动实例化了EmployeeHolidaysController，这不是个好做法——控制器是由DI容器管理的，手动new容易导致依赖注入的问题，而且控制器的职责是处理HTTP请求，不应该被其他控制器调用。

建议把EmployeeHolidayMaintance()的逻辑抽成一个独立的服务（比如EmployeeHolidayService），然后在需要的控制器里注入这个服务来调用方法，这样代码更规范，也更容易维护。

备注：内容来源于stack exchange，提问作者Max Burridge