ASP.NET MVC 3中的JsonResult：ModelState始终有效

Question

我正在尝试使用ModelState验证表单，但我的ModelState始终显示为有效。例如：当我尝试保存一个包含相同社会保障号码的2个人的formModel时，ModelState返回有效。我正在使用IValidatableObject来验证我的formmodel。有什么想法我可能做错了什么？我正在使用带有MVC 3的.Net 4.0。

public JsonResult LoadOccupantsDetailedInformation()
{
    //Load the personsFormModel with data
    return new JsonpResult(personsFormModel, JsonRequestBehavior.AllowGet);
}

[HttpPost]
public ActionResult SaveOccupantsDetailedInformation(
PersonsFormModel personsFormModel)
    {
//This line is always returning true even if I pass 2 persons with the same ssn
        if (ModelState.IsValid == false)
        {
            var errorList = ModelState.ToDictionary(
                      kvp => kvp.Key,
                      kvp => kvp.Value.Errors.Select(e => e.ErrorMessage).ToArray()
                );
            return Json(new { Errors = errorList });
        }
        //Save the data in personsFormModel to database
        return Json(new JsonResultViewModel { Success = true });
    }


public partial class PersonsFormModel : List<PersonFormModel>, IValidatableObject
{
    public IEnumerable<System.ComponentModel.DataAnnotations.ValidationResult> Validate(
    ValidationContext validationContext)
    {
        var validationResults
            = new List<System.ComponentModel.DataAnnotations.ValidationResult>();
        if (this.SSNs.Count() > this.SSNs.Distinct().Count())
        {
            validationResults.Add(new System.ComponentModel.DataAnnotations.ValidationResult(
            "All the persons in the household should have a unique SSN\\ITIN number",
                new[] { "SSN" }));
        }
        return validationResults;
    }
    private IEnumerable<string> SSNs
    {
        get
        {
            return this.Select(element => element.SSN);
        }
    }
}
    public class PrequalifyOccupantListPersonDetailedFormModel
{
    [Required(ErrorMessage = "SSN is required")]
    public string SSN { get; set; }
}

- Ajit Goel

---

我通过在SaveOccupantsDetailedInformation控制器操作中手动调用ValidateViewModel（personsFormModel，this）来解决了这个问题，在检查ModelState是否有效之前。我从stackoverflow.com的一个问题中得到了这个函数。 - Ajit Goel

Answer 1

我写了一个测试来验证你的验证，看起来很不错。

        [Test]
        public void Should_validate_person_form_model()
        {
            var input = new PersonsFormModel();
            input.Add(new PersonFormModel { SSN = "33" });
            input.Add(new PersonFormModel { SSN = "33" });
            _controller.ViewData.ModelState.Clear();

            var results = new List<ValidationResult>();

            bool isValid = Validator.TryValidateObject(input,
                new ValidationContext(input, null, null),
                results,true);

            Assert.IsTrue(!isValid, "Cannot have duplicates.");
        }

所以要么它没有被调用，要么您的数据是有效的。

为了测试前者，您可以在验证函数中设置断点，并确认它是否被触发。

为了测试后者，您可以提供所发布的确切数据和涉及的完整模型。

- dove

---

是的，我已经编写了一个测试用例，它可以正常工作，但当从视图调用保存控制器操作时，Validate方法没有被调用。为了解决这个问题，我们不得不更改PersonsFormModel，使其不再继承自List<PersonFormModel>，并从UI视图中将List<PersonFormModel>作为数组传递。我在保存FormModel时不再需要调用ValidateViewModel(personsFormModel, this)。 - Ajit Goel

@AjitGoel，您还需要其他帮助吗？还是已经解决了？ - dove

还要注意，如果模型由于数据注释所需的属性无效，则不会调用验证器，但它将返回无效。 - dove

而剩下的部分类是什么？ - dove

是的，Dove，我已经解决了这个问题。我已经在下面发布了正确的答案。 - Ajit Goel

Answer 2

这是我解决问题的方法：

***Controller:***
public JsonResult LoadOccupantsDetailedInformation()
{
    //Load the personsFormModel with data
    return new JsonpResult(personsFormModel, JsonRequestBehavior.AllowGet);
}

[HttpPost]
public ActionResult SaveOccupantsDetailedInformation(
PersonsFormModel personsFormModel)
    {
//This line is always returning true even if I pass 2 persons with the same ssn
        if (ModelState.IsValid == false)
        {
            var errorList = ModelState.ToDictionary(
                      kvp => kvp.Key,
                      kvp => kvp.Value.Errors.Select(e => e.ErrorMessage).ToArray()
                );
            return Json(new { Errors = errorList });
        }
        //Save the data in personsFormModel to database
        return Json(new JsonResultViewModel { Success = true });
    }

***Model:***
public class PersonsFormModel : IValidatableObject
{
    public PersonsFormModel()
    {
        this.Instance = new List<PersonFormModel>();
    }

    public List<PrequalifyOccupantListPersonFormModel> Instance { get; set; }

    public void Add(PersonFormModel personFormModel)
    {
        Instance.Add(personFormModel);
    }

    public IEnumerable<System.ComponentModel.DataAnnotations.ValidationResult> Validate(
    ValidationContext validationContext)
    {
        var validationResults
            = new List<System.ComponentModel.DataAnnotations.ValidationResult>();
        if (this.SSNs.Count() > this.SSNs.Distinct().Count())
        {
            validationResults.Add(new System.ComponentModel.DataAnnotations.ValidationResult(
            "All the persons in the household should have a unique SSN\\ITIN number",
                new[] { "SSN" }));
        }
        return validationResults;
    }
    private IEnumerable<string> SSNs
    {
        get
        {
            return Instance.Select(element => element.SSN);
        }
    }
}

public class PersonFormModel
{
    [Required(ErrorMessage = "SSN is required")]
    public string SSN { get; set; }
}

View: 

// Note, I am passing the "Instance" here. 

 var postData = JSON.stringify({ Instance: list });

            RealPage.DataManager.POST({
                url: "/Approval/SaveOccupantsDetailedInformation"
                , data: postData
                , success: function (data) {
                    if (data && data.Success) {

                    } 
                }
            }); 

- Ajit Goel