如何在验证摘要中显示错误

时间:2013-01-25 00:31:51

标签: c# asp.net .net asp.net-mvc validation

我是MVC的新手,我正在试图弄清楚为什么没有显示验证摘要。这是代码:

视图:

<% using(var form = Html.BeginForm("Create", "User"))
{%>
    <table>
        <thead>
            <th>Create User</th>
            <th><%= Html.ValidationSummary(false) %></th>
        </thead>
        <tbody>
            <tr>
                <td><%= Html.LabelFor(model => model.Creating.Username) %>:</td>
                <td><%= Html.TextBoxFor(model => model.Creating.Username) %></td>
            </tr>
            <tr>
                <td><%= Html.LabelFor(model => model.Creating.Firstname) %>:</td>
                <td><%= Html.TextBoxFor(model => model.Creating.Firstname)%></td>
            </tr>
            <tr>
                <td><%= Html.LabelFor(model => model.Creating.Lastname) %></td>
                <td><%= Html.TextBoxFor(model => model.Creating.Lastname) %></td>
            </tr>
            <tr>
                <td colspan="2">
                    <input type="submit" value="Create" />
                </td>
            </tr>
        </tbody>
    </table>

<%}%>

相关控制器方法:

[HttpPost]
public ActionResult Create(User creating)
{
    var response = _service.Save(creating);
    if (response.Success)
        return RedirectToAction("Index");
    response.Errors.CopyToModelState(this.ModelState);
    return RedirectToAction("Index");
}

业务逻辑方法:

public Response Save(User user)
{
    //Place Validation logic here
    //Check username is between 3-30 characters and make sure the username is unique
    //return response if username fails business rules

    bool isDataInvalid = false;
    List<ValidationError> errorList = new List<ValidationError>();
    if ((user.Username.Length < 3) || user.Username.Length > 30)
    {
        ValidationError invalidUsernameLengthError = new ValidationError();
        invalidUsernameLengthError.Property = "Creating.Username";
        invalidUsernameLengthError.ErrorMessage = "must be between 3 and 30 characters long";
        errorList.Add(invalidUsernameLengthError);
        isDataInvalid = true;
    }

    if (isDataInvalid)
    {
        return new Response()
        {
            Success = false,
            Errors = errorList
        };   
    }

    _repository.Save(user);

    return new Response()
    {
        Success = true
    };
}

帮助方法:

public static void CopyToModelState(this List<ValidationError> errors,  ModelStateDictionary modelState)
{
    foreach (var error in errors)
    {
        modelState.AddModelError(error.Property, error.ErrorMessage);
    }
}

逻辑按照预期行事,但没有任何显示。我已经检查了输出的HTML,并且验证没有被写入。我已经尝试将模型的属性分配给modelState并直接在相关字段上显示验证,但这也不起作用。有什么想法吗?


啊!执行RedirectToAction会导致新请求,因此错误数据将丢失。此外,我正在使用不同的控制器,所以我需要显式调用原始视图(Index.aspx)。我的index.aspx期望的模型实际上不是用户对象,它是一个不同的列表对象,所以我需要这样做:

        var users = _service.FindAll();
        return View("Index", new UserListModel() { Users = users });

而不是RedirectToAction。这显然是验证错误的标准模式 - 成功(没有错误)您使用RedirectToAction,但是对于错误,您需要返回正确的视图。


好的,谢谢mattytommo - 这真的很有用。我仍然遇到问题。我现在有这个用于控制器 - 类似于你的建议,但仍然没有显示错误消息。我尝试了数据注释但无法使其工作(我正在使用MVC2),并且一直在尝试我能想到的所有内容以修复现有代码。

    [HttpPost]
    public ActionResult Create(User creating)
    {
        var response = _service.Save(creating);
        if (response.Success)
            return RedirectToAction("Index");

        foreach (var error in response.Errors)
        {
            ModelState.AddModelError(error.Property, error.ErrorMessage);
        }

        return RedirectToAction("Index");
    }

还有什么想法吗?我很欣赏这个建议!

1 个答案:

答案 0 :(得分:3)

我认为您按值传递了ModelState变量,因此您在该函数中所做的更改实际上并未保存。尝试重现你的功能,但对于实际的ModelState对象,就像这样:

[HttpPost]
public ActionResult Create(User creating)
{
    var response = _service.Save(creating);
    if (response.Success)
        return RedirectToAction("Index");

    foreach (var error in response.Errors)
    {
        ModelState.AddModelError(error.Property, error.ErrorMessage);
    }

    return View(creating);
}

您还应该查看MVC数据注释,可以通过将这两个属性放在Username属性上来简单地替换您的验证:

[MinLength(3), MaxLength(30)]
public string UserName { get; set; }

或只使用StringLength的一个属性(感谢@SimonWhitehead):

[StringLength(30, MinimumLength = 3)]

数据注释:Here