ViewModel属性值符合长度时替换为数据库查询，应置于何处？

背景回顾 #

你的ViewModel结构如下：

public class ProductVM
{
       public string SerialNumber { get; set; }

       // 其他属性省略
}

表单允许用户输入100字符的SerialNumber或50字符的ProductId，当前在Controller的Action里，先检查ModelState.IsValid，再判断输入长度，如果是50位则视为ProductId，去数据库查询对应的SerialNumber。你认为这种方式存在两个核心问题：

1. ViewModel的验证逻辑应在ModelState.IsValid调用时完成，而非后续额外处理
2. Controller会因此变得臃肿，违反单一职责原则，增加流程理解难度

针对你的三个问题，逐一解答：

1. 是否应保留当前代码？ #

不建议保留。当前代码将数据转换、额外验证逻辑混杂在Controller中，违背了Controller只负责"接收请求、调度业务、返回响应"的职责定位。这类代码会随着业务迭代不断堆积，导致Controller越来越难维护，排查问题的成本也会大幅上升。

2. 是否应将检查/转换逻辑移出Controller？如果是，放在哪里？ #

必须移出，推荐以下几种方案（按耦合度从优到劣排序）：

方案一：自定义ModelBinder

ModelBinder是ASP.NET MVC中负责将请求数据绑定到ViewModel的核心组件，你可以在这里完成输入的识别、转换和验证，确保Action执行前，ViewModel已处于正确状态，ModelState.IsValid能直接反映最终验证结果。

示例实现：

public class ProductVMModelBinder : IModelBinder
{
    private readonly YourDbContext _db;

    // 通过构造函数注入数据库上下文或相关服务
    public ProductVMModelBinder(YourDbContext db)
    {
        _db = db;
    }

    public object BindModel(ControllerContext controllerContext, ModelBindingContext bindingContext)
    {
        var inputValue = bindingContext.ValueProvider.GetValue("SerialNumber")?.AttemptedValue;
        var vm = new ProductVM();

        if (string.IsNullOrEmpty(inputValue))
        {
            bindingContext.ModelState.AddModelError("SerialNumber", "请输入产品编号或序列号");
            return vm;
        }

        if (inputValue.Length == 50)
        {
            // 识别为ProductId，查询对应SerialNumber
            var serialNumber = _db.GetSerialNumber(inputValue);
            if (string.IsNullOrEmpty(serialNumber))
            {
                bindingContext.ModelState.AddModelError("", "未找到对应产品，请检查输入的ProductId");
                vm.SerialNumber = inputValue; // 保留用户输入以便页面回显
                return vm;
            }
            vm.SerialNumber = serialNumber;
        }
        else if (inputValue.Length == 100)
        {
            // 识别为SerialNumber，直接赋值
            vm.SerialNumber = inputValue;
        }
        else
        {
            bindingContext.ModelState.AddModelError("SerialNumber", "输入格式不正确，请输入50位ProductId或100位SerialNumber");
        }

        return vm;
    }
}

在ViewModel上绑定该ModelBinder：

[ModelBinder(typeof(ProductVMModelBinder))]
public class ProductVM
{
    public string SerialNumber { get; set; }
}

优化后的Controller会极度简洁：

[HttpPost]
public ActionResult GenerateReport(ProductVM vm)
{
    if (!ModelState.IsValid)
    {
        vm.Rehydrate();
        return View(vm);
    }

    var reportData = _productDomainService.GetReportData(vm.SerialNumber);
    var report = _infrastructure.GenerateReport(reportData);

    // 后续代码省略
}

方案二：封装转换服务

创建专门的服务类处理输入的识别、转换和验证，将逻辑从Controller中剥离，适合对ModelBinder不熟悉的场景。

示例实现：

public interface IProductIdentifierService
{
    (string SerialNumber, string ErrorMessage) ConvertAndValidate(string input);
}

public class ProductIdentifierService : IProductIdentifierService
{
    private readonly YourDbContext _db;

    public ProductIdentifierService(YourDbContext db)
    {
        _db = db;
    }

    public (string SerialNumber, string ErrorMessage) ConvertAndValidate(string input)
    {
        if (string.IsNullOrEmpty(input))
        {
            return (null, "请输入产品编号或序列号");
        }

        if (input.Length == 50)
        {
            var serialNumber = _db.GetSerialNumber(input);
            if (string.IsNullOrEmpty(serialNumber))
            {
                return (null, "未找到对应产品，请检查输入的ProductId");
            }
            return (serialNumber, null);
        }
        else if (input.Length == 100)
        {
            return (input, null);
        }
        else
        {
            return (null, "输入格式不正确，请输入50位ProductId或100位SerialNumber");
        }
    }
}

Controller中调用该服务：

[HttpPost]
public ActionResult GenerateReport(ProductVM vm)
{
    if (!ModelState.IsValid)
    {
        vm.Rehydrate();
        return View(vm);
    }

    var (serialNumber, errorMessage) = _productIdentifierService.ConvertAndValidate(vm.SerialNumber);
    if (!string.IsNullOrEmpty(errorMessage))
    {
        ModelState.AddModelError("", errorMessage);
        vm.Rehydrate();
        return View(vm);
    }

    var reportData = _productDomainService.GetReportData(serialNumber);
    var report = _infrastructure.GenerateReport(reportData);

    // 后续代码省略
}

3. 是否应新建Service Query，通过ProductId而非SerialNumber获取所需数据？ #

不推荐。这种方案会导致大量代码重复：你需要维护两套业务逻辑（通过SerialNumber取数据、通过ProductId取数据），后续业务规则变更时，需同时修改两处代码，大幅提升维护成本。更合理的方式是统一入口，将ProductId转SerialNumber的逻辑封装起来，让业务层只需要处理SerialNumber即可，保持业务逻辑的单一性。

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