Как заставить MVC проверять IValidatableObject - PullRequest
31 голосов
/ 22 июня 2011

Кажется, что когда MVC проверяет Модель, которая сначала проходит через атрибуты DataAnnotation (например, обязательные или диапазон), и если какой-либо из них не удается, он пропускает запуск метода Validate в моей модели IValidatableObject.

Есть ли способ запустить MVC и запустить этот метод, даже если другая проверка не пройдена?

Ответы [ 2 ]

35 голосов
/ 22 июня 2011

Вы можете вручную вызвать Validate (), передав новый экземпляр ValidationContext, например:

[HttpPost]
public ActionResult Create(Model model) {
    if (!ModelState.IsValid) {
        var errors = model.Validate(new ValidationContext(model, null, null));
        foreach (var error in errors)                                 
            foreach (var memberName in error.MemberNames)
                ModelState.AddModelError(memberName, error.ErrorMessage);

        return View(post);
    }
}

Предостережение этого подхода состоит в том, что в случаях, когда нет ошибок уровня свойства (DataAnnotation), проверка будет выполняться дважды. Чтобы избежать этого, вы можете добавить свойство к вашей модели, скажем, логическое значение Validated, которое вы устанавливаете равным true в вашем методе Validate () после его запуска, а затем проверяете перед вызовом метода вручную в вашем контроллере.

Итак, в вашем контроллере:

if (!ModelState.IsValid) {
    if (!model.Validated) {
        var validationResults = model.Validate(new ValidationContext(model, null, null));
        foreach (var error in validationResults)
            foreach (var memberName in error.MemberNames)
                ModelState.AddModelError(memberName, error.ErrorMessage);
    }

    return View(post);
}

А в вашей модели:

public bool Validated { get; set; }

public IEnumerable<ValidationResult> Validate(ValidationContext validationContext) {
    // perform validation

    Validated = true;
}
26 голосов
/ 23 сентября 2011

Есть способ сделать это, не требуя шаблонного кода в верхней части каждого действия контроллера.

Вам нужно будет заменить связыватель модели по умолчанию на один из ваших:

protected void Application_Start()
{
    // ...
    ModelBinderProviders.BinderProviders.Clear();
    ModelBinderProviders.BinderProviders.Add(new CustomModelBinderProvider());
    // ...
}

Ваш поставщик связывателей моделей выглядит следующим образом:

public class CustomModelBinderProvider : IModelBinderProvider
{
    public IModelBinder GetBinder(Type modelType)
    {
        return new CustomModelBinder();
    }
}

Теперь создайте настраиваемый связыватель модели, который фактически вызывает проверку.Вот где тяжелая работа проделана:

public class CustomModelBinder : DefaultModelBinder
{
    protected override void OnModelUpdated(ControllerContext controllerContext, ModelBindingContext bindingContext)
    {
        base.OnModelUpdated(controllerContext, bindingContext);

        ForceModelValidation(bindingContext);
    }

    private static void ForceModelValidation(ModelBindingContext bindingContext)
    {
        var model = bindingContext.Model as IValidatableObject;
        if (model == null) return;

        var modelState = bindingContext.ModelState;

        var errors = model.Validate(new ValidationContext(model, null, null));
        foreach (var error in errors)
        {
            foreach (var memberName in error.MemberNames)
            {
                // Only add errors that haven't already been added.
                // (This can happen if the model's Validate(...) method is called more than once, which will happen when
                // there are no property-level validation failures.)
                var memberNameClone = memberName;
                var idx = modelState.Keys.IndexOf(k => k == memberNameClone);
                if (idx < 0) continue;
                if (modelState.Values.ToArray()[idx].Errors.Any()) continue;

                modelState.AddModelError(memberName, error.ErrorMessage);
            }
        }
    }
}

Вам также понадобится метод расширения IndexOf.Это дешевая реализация, но она будет работать:

public static int IndexOf<T>(this IEnumerable<T> source, Func<T, bool> predicate)
{
    if (source == null) throw new ArgumentNullException("source");
    if (predicate == null) throw new ArgumentNullException("predicate");

    var i = 0;
    foreach (var item in source)
    {
        if (predicate(item)) return i;
        i++;
    }

    return -1;
}
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...