MVC验证不起作用

时间:2015-07-22 19:46:04

标签: c# asp.net-mvc validation

我有一个视图,其中包含一个用于输入用户名的文本框,然后是两个复选框。将用于将所述用户名添加到由复选框标记的角色。文本框不应允许输入空/空字符串,如果用户名已存在,则警告此人。

查看

@model QIEducationWebApp.Models.UserAdmin

<h1 class="page-header">Add New User</h1>

<script src="@Url.Content("~/Scripts/jquery.validate.min.js")" type="text/javascript"></script>
<script src="@Url.Content("~/Scripts/jquery.validate.unobtrusive.min.js")" type="text/javascript"></script>

@using (Html.BeginForm())
{
    @Html.ValidationSummary(true)

    <table class="table">
        <tr>
            <th class="table-row">
                User Name:
            </th>
            <td class="table-row">
                @Html.TextBoxFor(model => model.UserName)
                @Html.ValidationMessageFor(model => model.UserName)
            </td>
        </tr>
        <tr>
            <th class="table-row">
                Role:
            </th>
            <td class="table-row">
                @Html.DropDownListFor(model => model.UserRole,
                    @ViewBag.Roles as SelectList, " -- Select Role -- ", new { @class="form-control" })
                @Html.ValidationMessageFor(model => model.UserRole)
            </td>
        </tr>
        <tr><td class="table-row-blank"></td></tr>
        <tr>
            <td class="table-row-button">
                <input class="button" type="submit" value="Create" />
                <input type="button" class="button" value="Cancel" 
                    onclick="location.href='@Url.Action("AllUsers")'" />
            </td>
        </tr>
    </table>
}

模型

[EdmScalarPropertyAttribute(EntityKeyProperty=false, IsNullable=false)]
[Required(ErrorMessage = "Required")]
[Remote("ExistUser", "Admin", HttpMethod = "POST",
    ErrorMessage = "User is assinged, Edit instead")]
[DataMemberAttribute()]
public global::System.String UserName
{
    get
    {
        return _UserName;
    }
    set
    {
        OnUserNameChanging(value);
        ReportPropertyChanging("UserName");
        _UserName = StructuralObject.SetValidValue(value, false);
        ReportPropertyChanged("UserName");
        OnUserNameChanged();
    }
}
private global::System.String _UserName;
partial void OnUserNameChanging(global::System.String value);
partial void OnUserNameChanged();

控制器验证方法

[HttpPost]
public JsonResult ExistUser(String UserName)
{
    bool exist = db.UserAdmins.Any(u => u.UserName == UserName);

    return Json(!exist);
}

控制器发布方法

[Authorize(Roles = "Director")]
[HttpPost]
public ActionResult AddNewUser(UserAdmin user)
{
    if (ModelState.IsValid)
    {
        db.UserAdmins.AddObject(user);
        db.SaveChanges();
        return RedirectToAction("AllUsers");
    }
    ViewBag.Roles = new SelectList(db.UserRoles, "UserRoleID", "UserRole1", user.UserRole);
    return View(user);
}

目前,它允许在提交时传入空字符串,而不是显示错误。我的自定义验证甚至没有启动,我的调试也受到了影响。

我已经在应用程序的其他部分使用了它,但仍然有效。

P.S。如果你们需要更多代码,请告诉我,我会把它拿出来。

编辑::与原始帖子完全不同的代码。

Volkan Paksoy指出我缺少ModelState.IsValid并查看返回。

对于客户端验证我错过了我的

@section Scripts {
    @Scripts.Render("~/bundles/jqueryval")
}

2 个答案:

答案 0 :(得分:1)

首先,正如@Chris Bohatka建议的那样,你必须使用你的模型作为参数(在这种情况下是用户类)

然后你必须像这样检查ModelState的状态:

if (!ModelState.IsValid)
{
    // There is an error on the page, load the same view to display the errors
}

// Model is fine, carry on with the other stuff...

我调试并在ModelState错误集合中看到“必须输入用户名”,因此您需要做的是返回相同的视图,以便用户可以看到错误并进行修复。

答案 1 :(得分:1)

也许您没有启用客户端验证,在这种情况下,即使字段不正确,它也会发布帖子。此外,在您的post控制器方法中,您没有检查ModelState的有效性,这可能是无效的,这意味着您没有陷阱的验证问题。将其添加到控制器方法:

if (ModelState.IsValid) { do work };

并确保在web.config中有这些行以启用客户端(JavaScript)验证,在这种情况下,在字段值有效之前不会发布帖子:

<add key="ClientValidationEnabled" value="true" />
<add key="UnobtrusiveJavaScriptEnabled" value="true" />