Задать свойство является правилом, действительным с FluentValidation

Question

У меня есть валидатор, который выглядит так

public class ImageValidator : AbstractValidator<Image>
{
    public ImageValidator()
    {
        RuleFor(e => e.Name).NotEmpty().Length(1, 255).WithMessage("Name must be between 1 and 255 chars");

        RuleFor(e => e.Data).NotNull().WithMessage("Image must have data").When(e => e.Id == 0);

        RuleFor(e => e.Height).GreaterThan(0).WithMessage("Image must have a height");

        RuleFor(e => e.Width).GreaterThan(0).WithMessage("Image must have a width");
    }
}

Теперь мои модульные тесты не проходят, потому что высота и ширина заполняются на основе значения в данных.

Данные содержат байтовый массив, который создает растровое изображение. Если данные не равны нулю (и Id равен 0, значит, это новое изображение), я могу создать растровое изображение и получить значения высоты и ширины. После обновления высота и ширина уже будут заполнены, и данные могут быть нулевыми, поскольку они уже будут храниться в базе данных.

В любом случае я могу заполнить значения высоты и ширины, если правило проверки для данных истинно в моем валидаторе?

До того, как у меня был чек, как

if (myObject.Data == null) throw new ApplicationException(...

Но я думаю, что это правило валидации, и оно должно быть в валидаторе, или мне просто нужно разделить правила?

Answer 1

Спасибо Джону Питерсу, который указал мне правильное направление, но вот что я сделал в конце, как и было предложено.

Сначала я создал новый валидатор, который проверяет, действителен ли байтовый массив. Этот валидатор принимает действие, которое принимает объект и растровое изображение в качестве входных данных. Это действие вызывается только в том случае, если валидатор действителен.

public class IsImageValidator<T> : PropertyValidator
{
    private Action<T, Bitmap> _action;

    public IsImageValidator(Action<T, Bitmap> action) : base("Not valid image data")
    {
        _action = action;
    }

    protected override bool IsValid(PropertyValidatorContext context)
    {
        var isNull = context.PropertyValue == null;

        if (isNull)
        {
            return false;
        }

        try
        {
            // we need to determine the height and width of the image
            using (var ms = new System.IO.MemoryStream((byte[])context.PropertyValue))
            {
                using (var bitmap = new System.Drawing.Bitmap(ms))
                {
                    // valid image, so call action
                    _action((T)context.Instance, bitmap);                        
                }
            }

        }
        catch (Exception e)
        {
            return false;
        }

        return true;
    }
}

Чтобы проще было назвать этот валидатор, я создал простое расширение, например

public static class SimpleValidationExtensions
{
    public static IRuleBuilderOptions<T, TProperty> IsImage<T, TProperty>(this IRuleBuilder<T, TProperty> ruleBuilder, Action<T, Bitmap> action)
    {
        return ruleBuilder.SetValidator(new IsImageValidator<T>(action));
    }
}

Теперь я могу вызывать метод validator в моем ImageValidator, поэтому мой ImageValidator теперь выглядит следующим образом. Как вы можете видеть, я теперь могу установить свойства высоты и ширины.

public ImageValidator()
{
    RuleFor(e => e.Name).NotEmpty().Length(1, 255).WithMessage("Name must be between 1 and 255 chars");

    RuleFor(e => e.Data).IsImage((e, bitmap) =>
    {
        e.Height = bitmap.Height;
        e.Width = bitmap.Width;
    }).WithMessage("Image data is not valid").When(e => e.Id == 0);

    RuleFor(e => e.Height).GreaterThan(0).WithMessage("Image must have a height");

    RuleFor(e => e.Width).GreaterThan(0).WithMessage("Image must have a width");
}

Answer 2

Текущий код выглядит следующим образом, обратите внимание, что свободный интерфейс.NotNull() влияет на результаты RuleFor.

  RuleFor(e => e.Data)
 .NotNull()
 .WithMessage("Image must have data")
 .When(e => e.Id == 0);

Предположим, вы могли бы сделать это:

var rf = RuleFor(e => e.Data)
.CheckNull(p=>{
   if(p){ 
       // this is the true branch when data is null don't set height/width}
       rf.WithMessage("Image must have data");
       .When(e=>e.id==0); 
   else{  
       //data is not null set the height and with. 
        rf(e => e.Height).GreaterThan(0).WithMessage("Image must have a height");
        rf(e => e.Width).GreaterThan(0).WithMessage("Image must have a width");
   });

Чтобы вы могли это сделать, новая подпрограмма CheckNull должна иметь тот же тип, что и метод.NotNull() сегодня.