我期待的行为是this one:首先是所有DataAnnotations,然后,只有在没有错误的情况下,验证方法.我怎么发现这并非总是如此.我的viewmodel(一个演示版)有三个文件,一个字符串,一个十进制和一个小数?所有这三个属性都只有required属性.对于字符串和小数?行为是预期的行为,但对于小数,当为空时,必需的验证失败(到目前为止很好),然后执行Validate方法.如果我检查属性,其值为零.
这里发生了什么?我错过了什么?
注意:我知道必须使用required属性来检查该值是否为null.所以我希望被告知不要在不可空类型中使用required属性(因为它不会触发),或者,某种程度上该属性理解POST值并注意该字段未被填充.在第一种情况下,属性不应该触发,并且应该触发Validate方法.在第二种情况下,属性应该触发,并且不应触发Validate方法.但我的结果是:属性触发器和Validate方法触发.
这是代码(没什么特别的):
控制器:
public ActionResult Index() { return View(HomeModel.LoadHome()); } [HttpPost] public ActionResult Index(Homeviewmodel viewmodel) { try { if (ModelState.IsValid) { HomeModel.ProcessHome(viewmodel); return RedirectToAction("Index","Result"); } } catch (ApplicationException ex) { ModelState.AddModelError(string.Empty,ex.Message); } catch (Exception ex) { ModelState.AddModelError(string.Empty,"Internal error."); } return View(viewmodel); }
模型:
public static Homeviewmodel LoadHome() { Homeviewmodel viewmodel = new Homeviewmodel(); viewmodel.String = string.Empty; return viewmodel; } public static void ProcessHome(Homeviewmodel viewmodel) { // Not relevant code }
视图模型:
public class Homeviewmodel : IValidatableObject { [required(ErrorMessage = "required {0}")] [Display(Name = "string")] public string String { get; set; } [required(ErrorMessage = "required {0}")] [Display(Name = "decimal")] public decimal Decimal { get; set; } [required(ErrorMessage = "required {0}")] [Display(Name = "decimal?")] public decimal? DecimalNullable { get; set; } public IEnumerable<ValidationResult> Validate(ValidationContext validationContext) { yield return new ValidationResult("Error from Validate method"); } }
视图:
@model MVCTest1.viewmodels.Homeviewmodel @{ Layout = "~/Views/Shared/_Layout.cshtml"; } @using (Html.BeginForm(null,null,FormMethod.Post)) { <div> @Html.ValidationSummary() </div> <label id="lblNombre" for="Nombre">Nombre:</label> @Html.TextBoxFor(m => m.Nombre) <label id="lblDecimal" for="Decimal">Decimal:</label> @Html.TextBoxFor(m => m.Decimal) <label id="lblDecimalNullable" for="DecimalNullable">Decimal?:</label> @Html.TextBoxFor(m => m.DecimalNullable) <button type="submit" id="aceptar">Aceptar</button> <button type="submit" id="superAceptar">SuperAceptar</button> @Html.HiddenFor(m => m.Accion) }
解决方法
双方同意和expected behavior among developers是IValidatableObject的方法Validate()仅在未触发验证属性时被调用.简而言之,预期的算法就是这个(取自上一个链接):
>验证属性级属性
>如果任何验证器无效,则中止验证返回失败
>验证对象级属性
>如果任何验证器无效,则中止验证返回失败
>如果在桌面框架上并且该对象实现了IValidatableObject,则调用其Validate方法并返回任何失败
但是,使用问题代码,即使在[必需]触发器之后也会调用Validate.这似乎是一个明显的MVC错误.据报道here.
三种可能的解决方法:
>有一个解决方法here,虽然它的使用有一些陈述的问题,除了打破MVC预期的行为.通过一些更改以避免在同一字段中显示多个错误,代码如下:
viewmodel .Validate(new ValidationContext(viewmodel,null)) .ToList() .ForEach(e => e.MemberNames.ToList().ForEach(m => { if (ModelState[m].Errors.Count == 0) ModelState.AddModelError(m,e.ErrorMessage); }));
>忘记IValidatableObject并仅使用属性.它干净,直接,更好地处理本地化,并且最重要的是它可以在所有型号中重复使用.只需为您要执行的每个验证实施ValidationAttribute.您可以验证所有模型或特定属性,这取决于您.除了默认可用的属性(DataType,Regex,required和所有这些东西)之外,还有几个具有最常用验证的库.实现“失踪者”的是FluentValidation.
>仅实现IValidatableObject接口丢弃data annotations.如果它是一个非常特殊的模型并且它不需要太多验证,这似乎是一个合理的选择.在大多数情况下,开发人员将执行所有常规和常见验证(即必需等),如果使用了属性,则会导致默认情况下已经实现的验证上的代码重复.也没有可重用性.
评论前回答:
首先,我从头开始创建了一个新项目,只提供了您提供的代码.它永远不会同时触发数据注释和Validate方法.
无论如何,知道这一点,
根据设计,MVC3将[required]属性添加到非可空值类型,如int,DateTime或yes,decimal.因此,即使您从该小数中删除了所需的属性,它也就像它在那里一样.
这是错误的(或不是),但它的设计方式是有争议的.
在你的例子中:
>如果[required]存在且没有给出值,则’DataAnnotation’触发.从我的观点来看,完全可以理解
>如果不存在[必需]但值不可为空,则触发’DataAnnotation’.有争议,但我倾向于同意它,因为如果属性不可为空,则必须输入值,否则不向用户显示或只使用可空的小数.
看来,这种行为可能会在Application_Start方法中被关闭:
DataAnnotationsModelValidatorProvider.AddImplicitrequiredAttributeForValueTypes = false;
我猜这个属性的名字是不言自明的.
无论如何,我不明白为什么你想让用户输入不需要的东西,不要让那个属性可以为空.如果它是null,那么检查它是你的工作,如果在验证之前你不想在控制器内做空.
public ActionResult Index(Homeviewmodel viewmodel) { // Complete values that the user may have // not filled (all not-required / nullables) if (viewmodel.Decimal == null) { viewmodel.Decimal = 0m; } // Now I can validate the model if (ModelState.IsValid) { HomeModel.ProcessHome(viewmodel); return RedirectToAction("Ok"); } }
您觉得这种方法有什么不对,或者不应该这样?