时间戳

首页 > 解决方案 > FluentValidation 在错误的视图模型上使用验证器

c# - FluentValidation 在错误的视图模型上使用验证器

问题描述

我第一次使用 FluentValidation。我有一些基本的验证工作，但后来我意识到我需要做一些数据库检索来进行一些更复杂的验证。这需要进行依赖注入，以便我可以使用数据库服务，这导致我进入当前状态：卡住。我不能让它工作。

为简化起见，我假设我的应用程序正在处理体育联盟和球队，因为我认为这是一个比合同、发票、资金来源、供应商和分包商更简单的思维模型。:-)

所以，假设我有一个体育联盟的视图模型。在该视图模型中，有一组用于该联盟中的球队的视图模型。

我有一个编辑联赛的屏幕。同一屏幕允许更改有关该联盟中球队的一些信息。

联赛视图模型

联赛的视图模型包含球队的视图模型列表。

[FluentValidation.Attributes.Validator(typeof(LeagueValidator))]
public class LeagueViewModel
{
    public string LeagueName { get; set; }
    public DateTime SeasonBeginDate { get; set; }
    public DateTime SeasonEndDate { get; set; }

    public List<TeamViewModel> TeamViewModels { get; set; }
}

我为 LeagueViewModel 创建了一个验证器。不幸的是，当我编辑联赛并单击提交按钮时，我收到以下错误消息：

InvalidCastException：无法将“TeamViewModel”类型的对象转换为“LeagueViewModel”类型。在 FluentValidation.ValidationContext.ToGenericT

显然它正在尝试使用 LeagueValidator 验证 TeamViewModel。

我经历了许多变化，试图弄清楚如何让它发挥作用。这就是我目前所拥有的。

验证器

public class LeagueValidator : AbstractValidator<LeagueViewModel>
{
    private readonly ILeagueService _leagueService;

    public LeagueValidator(ILeagueService leagueService)
    {
        _leagueService = leagueService;

        RuleFor(x => x.SeasonEndDate)
            .NotNull()
            .GreaterThan(x => x.SeasonBeginDate)
            .WithMessage("Season End Date must be later than Season Begin Date.");
    }
}

（LeagueService 位在那里，因为在实际代码中它需要检查一些数据库值，它使用服务来检索这些值。）

请注意，LeagueValidator 对 TeamViewModel 列表中的任何字段都没有任何验证规则。

联盟验证器工厂

public class LeagueValidatorFactory : ValidatorFactoryBase
{
    private readonly Container _container;

    public LeagueValidatorFactory(Container container)
    {
        _container = container;
    }

    public override IValidator CreateInstance(Type validatorType)
    {
        return _container.GetInstance<LeagueValidator>();
    }
}

依赖注入器

我们将 SimpleInjector 用于 DI。作为现有设置的一部分，它正在调用一个方法来注册服务。在该方法中，我添加了对此的调用：

private static void RegisterValidators(Container container)
{
    DataAnnotationsModelValidatorProvider.AddImplicitRequiredAttributeForValueTypes = false;

    var leagueValidatorProvider =
        new FluentValidationModelValidatorProvider(new LeagueValidatorFactory(container));
    leagueValidatorProvider.AddImplicitRequiredValidator = false;
    ModelValidatorProviders.Providers.Add(leagueValidatorProvider);

    container.Register<LeagueValidator>();
}

问题

  1. 我怎样才能让它正常工作？
  2. 为什么要尝试使用 LeagueValidator 来验证 TeamViewModel？
  3. 我是否需要为每个视图模型有一个单独的验证器和验证器工厂？
  4. 即使是那些没有任何验证规则的？
  5. 我如何告诉它哪个验证器用于哪个视图模型？

我想我一定是误解了一些基本的东西。

编辑

史蒂文在下面的回答让我指出了正确的方向！在我做出他建议的更改后，我遇到了另一个错误。一旦我解决了这个问题，它就可以工作了！以下是我为使上述代码正常工作所做的更改。

联赛视图模型

我删除了这条线，因为它没有必要。

[FluentValidation.Attributes.Validator(typeof(LeagueValidator))]

LeagueValidatorFactory

我将它重命名为“ValidatorFactory”，因为不管我创建了多少验证器，事实证明只有一个验证器工厂。然后我将 CreateInstance 方法更改为：

    public override IValidator CreateInstance(Type validatorType)
    {
        if (_container.GetRegistration(validatorType) == null)
        {
            return null;
        }

        return (IValidator)_container.GetInstance(validatorType);
    }

这不再明确指定要获取的验证器的类型（这就是为什么只需要一个工厂的原因）。为了确定给定类型的验证器是否可用，它调用 GetRegistration，如果没有找到则返回 null。

这很重要！对于每个视图模型，它都会尝试找到一个验证器。如果没有此 null 检查，则会引发 InvalidCastException。

依赖注入器

按照 Steven 的建议，我将 container.Register 替换为：

container.Register(typeof(IValidator<>), new[] { typeof(SimpleInjectorInitializer).Assembly });

这避免了每次添加新验证器时都需要明确列出每个验证器。

现在一切正常！非常感谢您的帮助，史蒂文！

标签： c#validationdependency-injectionsimple-injectorfluentvalidation

解决方案

我不熟悉 FluentValidation，但LeagueValidatorFactory考虑到它提供了要验证的类型，您似乎从容器中请求了错误的类型。

因此，我希望您的验证工厂看起来像这样：

public class LeagueValidatorFactory : ValidatorFactoryBase
{
    private readonly Container _container;

    public LeagueValidatorFactory(Container container) =>
        _container = container;

    public override IValidator CreateInstance(Type validatorType) =>
        (IValidator)_container.GetInstance(validatorType);
}

我从 FluentValidator 源代码中可以看到，这是该类型validatorType的封闭泛型版本，它是正在验证的实际类型。这意味着，您必须通过其接口注册验证器。例如：IValidator<T>TIValidator<T>

container.Register<IValidator<LeagueViewModel>, LeagueValidator>();

这种配置即代码（或显式注册）模型，您可以使用一行代码显式注册每个验证器，如果您只有几个验证器，它可能会正常工作，但这通常会导致组合根必须经常更新.

因此，更好的模型是使用Auto-Registration，您可以在其中IValidator<T>使用反射注册所有实现。幸运的是，您不必自己实现它；Simple Injector 为您提供支持：

var validatorAssemblies = new[] { typeof(LeagueValidator).Assembly };
container.Register(typeof(IValidator<>), validatorAssemblies);

这确保您在添加新验证器（在该特定程序集中）时永远不必更改您的组合根。

有了这个设置，我看不出为什么你应该用FluentValidation.Attributes.ValidatorAttribute. 如果可以，请删除它，因为它只会导致您的视图模型和验证器之间的不紧密耦合。

推荐阅读