我正在开发一个大型 MVC3 Web 应用程序，并且对 ModelState.IsValid 方法感到烦恼。
ModelState 几乎在我的所有控制器中都被使用，以验证发布的数据。
这些视图都基于包含不同类的 ViewModel，这些类显然包含可以标记为[Required] 的属性。
我遇到的问题是有时不需要所需的属性，我不得不使用ModelState.Remove 方法，以便ModelState.IsValid 变为真。
我的问题是使用ModelState.Remove，这是正确的做事方式还是有更有效的方法。
【问题讨论】：
标签： asp.net-mvc asp.net-mvc-3 modelstate
如果您的财产并非总是需要，则不应使用[Required] 来装饰它。
进行验证的一个很好的替代方法是实现接口IValidatableObject。
例如，假设您想让字段State 仅在国家/地区为United States 时才需要。你可以这样做：
public class AddressModel : IValidatableObject
{
[Required]
public string Country { get; set; }
public string State { get; set; }
public IEnumerable<ValidationResult> Validate(ValidationContext validationContext)
{
if(Country == "United States" && String.IsNullOrEmpty(State))
{
yield return new ValidationResult("State is required for United States", new [] { nameof(State) });
}
}
}
注意：这种验证只适用于服务器端。
其他选择？
正如其他答案中提到的，如果视图和验证非常不同，有时创建 2 个或更多模型是个好主意。
【讨论】：
这是我的解决方案 - ModelState 上的 RemoveFor() 扩展方法，模仿 MVC HTML 助手：
public static void RemoveFor<TModel>(this ModelStateDictionary modelState,
Expression<Func<TModel, object>> expression)
{
string expressionText = ExpressionHelper.GetExpressionText(expression);
foreach (var ms in modelState.ToArray())
{
if (ms.Key.StartsWith(expressionText + ".") || ms.Key == expressionText)
{
modelState.Remove(ms);
}
}
}
这是它的使用方法：
if (model.CheckoutModel.ShipToBillingAddress == true)
{
// REUSE BILLING ADDRESS FOR SHIPPING ADDRESS
ShoppingCart.ShippingAddress = ShoppingCart.BillingAddress;
// REMOVE MODELSTATE ERRORS FOR SHIPPING ADDRESS
ModelState.RemoveFor<SinglePageStoreModel>(x => model.CheckoutModel.ShippingAddress);
}
因此，在回答您的问题时，我相信在某些用例中这是正确的方法，并且像这样的强类型帮助器使它看起来更好看 - 并且更容易证明您是担心很多魔术字符串。
【讨论】：
ms.Key.StartsWith(expressionText + "[")。
我完全支持史蒂夫·摩根先生
因此，如果您的 ViewModel 并不总是需要某些属性为 Required，那么您不应该将其装饰为必需。
我不知道你为什么想要这个问题，但我想在某些情况下，如果 PropertyTwo 有价值，你需要 PropertyOne 成为 Required。
在这种情况下，您可能需要使用 CustomValidationAttribute 来检查这两个属性。
我正在使用这样的东西：
[AttributeUsage(AttributeTargets.Class, AllowMultiple = true, Inherited = true)]
public class PropertyNeededAttribute : ValidationAttribute
{
private const string defaultErrorMessage = "'{0}' needs '{1}' to be valid.";
public PropertyNeededAttribute(string originalProperty, string neededProperty)
: base(defaultErrorMessage)
{
NeededProperty = neededProperty;
OriginalProperty = originalProperty;
}
public string NeededProperty { get; private set; }
public string OriginalProperty { get; private set; }
public override object TypeId
{
get { return new object(); }
}
public override string FormatErrorMessage(string name)
{
return String.Format(CultureInfo.CurrentUICulture, ErrorMessageString,
OriginalProperty, NeededProperty);
}
public override bool IsValid(object value)
{
object neededValue = Statics.GetPropertyValue(value, NeededProperty);
object originalValue = Statics.GetPropertyValue(value, OriginalProperty);
if (originalValue != null && neededValue == null)
return false;
return true;
}
}
注意：Statics.GetPropertyValue(...) 什么都不做，只是从属性中获取值进行比较。
希望这有帮助:)
【讨论】：
从根本上说，您的问题是，虽然您的课程用 [Required] 装饰，但并非总是如此。如果您在不正确的上下文中进行操作，那么您确实应该使用未将属性定义为 [必需] 的类。
您应该真正使用针对其特定用途正确定义的 ViewModel，这可能意味着复制某些类。 ViewModel 与 UI 的实现相关联，虽然它可能使用您的领域模型中的类，但这并不总是正确的做法。
否则，选项是不使用 ModelState.IsValid，或者继续使用 ModelState.Remove。
但从逻辑上讲，您的 ViewModel 是“可验证的”是有意义的，而不必忽略某些验证错误。
【讨论】：
AddressModel，如果 countryCd 是爱尔兰，则可能不需要Zip。我今天仍在使用这种方法，如果国家不需要，我会从 modelstate 中删除 zip 错误。这是一个无法使用单独视图模型的示例。
如果您在两种不同的上下文中使用具有 [Required] 属性的同一视图模型，一种是需要该属性，另一种是不需要该属性，那么您需要手动将 ModelState 更改为你在做。
另一种方法是使用不同的视图模型。可能有一个基类，其中包含所有属性，但有问题的必需属性除外。然后从中派生两个视图模型，一个具有需要的属性，一个具有不需要的属性（我知道这是重复的）。您可以决定将它们完全分开而不使用继承。
【讨论】：