ASP.NET MVC: проверка аннотирования данных достаточно?

Я соединил xVal с DataAnnotations и написал свой собственный фильтр действий, который проверяет любые параметры типа Entity для целей проверки. Поэтому, если какое-то поле отсутствует в обратной передаче, этот валидатор заполнит словарь ModelState, следовательно, модель будет недействительной.

Предварительные требования:

• Все мои объекты сущности / модели реализуют интерфейс IObjectValidator , который объявляет Validate () метод.
• мой класс атрибутов называется ValidateBusinessObjectAttribute
• xVal validation library

Код фильтра действий:

public void OnActionExecuting(ActionExecutingContext filterContext)
{
    IEnumerable<KeyValuePair<string, object>> parameters = filterContext.ActionParameters.Where<KeyValuePair<string, object>>(p => p.Value.GetType().Equals(this.ObjectType ?? p.Value.GetType()) && p.Value is IObjectValidator);
    foreach (KeyValuePair<string, object> param in parameters)
    {
        object value;
        if ((value = param.Value) != null)
        {
            IEnumerable<ErrorInfo> errors = ((IObjectValidator)value).Validate();
            if (errors.Any())
            {
                new RulesException(errors).AddModelStateErrors(filterContext.Controller.ViewData.ModelState, param.Key);
            }
        }
    }
}

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

[ValidateBusinessObject]
public ActionResult Register(User user, Company company, RegistrationData registrationData)
{
    if (!this.ModelState.IsValid)
    {
        return View();
    }
    ...
}

DataAnnotation явно недостаточно. Я также широко использую его для предварительной проверки моих вызовов модели предметной области, чтобы получать более качественные отчеты об ошибках и выходить из строя как можно раньше.

Однако вы можете настроить модель DataAnnotation самостоятельно, чтобы гарантировать, что свойства с [Required] ДОЛЖНЫ быть опубликованы. (код будет позже сегодня).

ОБНОВЛЕНИЕ Получите источник для DataAnnotations Model Binder и найдите эту строку в DataAnnotationsModelBinder.cs

// Only bind properties that are part of the request
if (bindingContext.ValueProvider.DoesAnyKeyHavePrefix(fullPropertyKey)) {

Измените ее на

// Only bind properties that are part of the request
bool contextHasKey = bindingContext.ValueProvider.DoesAnyKeyHavePrefix(fullPropertyKey);
bool isRequired = GetValidationAttributes(propertyDescriptor).OfType<RequiredAttribute>().Count() > 0;
if (contextHasKey || (!contextHasKey && isRequired)) {

Я написал свой собственный ValidationService для MVC 1.0, скопировав шаблоны из DataAnnotationsRuleProvider от xVal и DataAnnotationsModelBinder от Microsoft (и из комментариев Martijn). Интерфейс службы приведен ниже:

public interface IValidationService
{
    void Validate(object instance);

    IEnumerable<ErrorInfo> GetErrors(object instance);
}

public abstract class BaseValidationService : IValidationService
{
    public void Validate(object instance)
    {
        var errors = GetErrors(instance);

        if (errors.Any())
            throw new RulesException(errors);
    }

    public abstract IEnumerable<ErrorInfo> GetErrors(object instance);
}

Служба представляет собой средство выполнения проверки, которое просматривает дерево свойств экземпляра объекта, которое он получает, и фактически выполняет атрибуты проверки, которые он находит для каждого свойства, создавая список объектов ErrorInfo, когда атрибуты недопустимы. . (Я бы опубликовал весь исходный код, но он был написан для клиента, и я еще не знаю, уполномочен ли я это делать.)

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

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