无法修复 ASP.NET MVC 验证

Question

一段时间以来，我一直在尝试修复我的表单验证，但无法正常工作。验证工作正常，除非我输入有效的电子邮件地址。如果我这样做，那么它会出于某种原因跳过所有其他验证。

此外，如果我在控制器中拆分所有内容时是否正确，我将不胜感激。是否应该有 2 个操作（1 个用于 GET 仅加载空表单，1 个用于 POST 当用户提交时）？我在那里做错了吗？

编辑： 更正：如果我输入有效的电子邮件地址，表单会正常提交并忽略其他验证。如果我没有有效的电子邮件地址，验证会检查所有内容。

这是我的模型 class:

public class User
{
    public int ID { get; set; }

    [DisplayName("Name")]
    [Required(ErrorMessage = "Name is required")]
    public string Name { get; set; }

    [DisplayName("Email")]
    [DataType(DataType.EmailAddress)]
    [Required(ErrorMessage = "Email is required")]
    [RegularExpression(
        @"^(?("")("".+?(?<!\)""@)|(([0-9a-z]((\.(?!\.))|[-!#$%&'\*\+/=\?\^`\{\}\|~\w])*)(?<=[0-9a-z])@))" +
            @"(?(\[)(\[(\d{1,3}\.){3}\d{1,3}\])|(([0-9a-z][-\w]*[0-9a-z]*\.)+[a-z0-9][\-a-z0-9]{0,22}[a-z0-9]))$",
        ErrorMessage = "Invalid email")]
    public string Email { get; set; }

    [DisplayName("Password")]
    [DataType(DataType.Password)]
    [Required(ErrorMessage = "Password required")]
    [MinLength(6, ErrorMessage = "Min 6 characters")]
    public string Password { get; set; }
}

这是我的控制器：

    public ActionResult Register()
    {
        ViewBag.LoggedIn = false;

        return View();
    }

    [HttpPost]
    public ActionResult Register(User user)
    {
        ViewBag.LoggedIn = false;

        user.Password = PasswordHash.CreateHash(user.Password);

        using (var context = new FoundationContext())
        {
            // if user exists
            if (context.Users.FirstOrDefault(n => n.Email == user.Email) != null) return Login(true);

            context.Users.Add(user);
            context.SaveChanges();
        }

        return View("~/Views/Home.Index.cshtml");
    }

这是我的表格：

    @using (Html.BeginForm())
    {
        @Html.AntiForgeryToken()

        <div class="form-horizontal">
            @Html.ValidationSummary(true, "", new { @class = "text-danger" })
            <div class="form-group">
                @Html.LabelFor(model => model.Name, htmlAttributes: new { @class = "control-label col-md-2" })
                <div class="col-md-10">
                    @Html.EditorFor(model => model.Name, new { htmlAttributes = new { @class = "form-control" } })
                    @Html.ValidationMessageFor(model => model.Name, "", new { @class = "text-danger" })
                </div>
            </div>

            <div class="form-group">
                @Html.LabelFor(model => model.Email, htmlAttributes: new { @class = "control-label col-md-2" })
                <div class="col-md-10">
                    @Html.EditorFor(model => model.Email, new { htmlAttributes = new { @class = "form-control" } })
                    @Html.ValidationMessageFor(model => model.Email, "", new { @class = "text-danger" })
                </div>
            </div>

            <div class="form-group">
                @Html.LabelFor(model => model.Password, htmlAttributes: new { @class = "control-label col-md-2" })
                <div class="col-md-10">
                    @Html.EditorFor(model => model.Password, new { htmlAttributes = new { @class = "form-control" } })
                    @Html.ValidationMessageFor(model => model.Password, "", new { @class = "text-danger" })
                </div>
            </div>

            <div class="form-group">
                <div class="col-md-10 text-center">
                    <input type="submit" value="Register" class="btn btn-primary" />
                </div>
            </div>
        </div>
    }

当然 body 的底部包括：

@Scripts.Render("~/bundles/jquery")
@Scripts.Render("~/bundles/jqueryval")
@Scripts.Render("~/bundles/bootstrap")
@RenderSection("scripts", required: false)

Answer 1

使用Modelstate.IsValid。它会告诉您是否已将任何模型错误添加到 ModelState。

public ActionResult Register(User user)
{
    if (!ModelState.IsValid) 
    {
       return View();
    }

    //your code
}

Answer 2

首先，您的结构似乎是正确的，将操作拆分为具有差异化的 Get 和 Post 操作是正确的，也是一个很好的方法。

我无法测试，但我认为问题在于您没有在控制器中正确处理验证；您应该检查 ModelState 是否有效。

    [HttpPost]
    public ActionResult Register(User user)
    {
        if (ModelState.IsValid)
        {
            ViewBag.LoggedIn = false;

            user.Password = PasswordHash.CreateHash(user.Password);

            using (var context = new FoundationContext())
            {
                // if user exists
                if (context.Users.FirstOrDefault(n => n.Email == user.Email) != null) return Login(true);

                context.Users.Add(user);
                context.SaveChanges();
            }

            return View("~/Views/Home.Index.cshtml");
        }
        return View(user);
    }

如果模型中存在错误（基于您使用 DataAnnotations 设置的规则），则采用这种方式。

此外，我不确定您的 DataAnnotations 属性是否正确：有些与本文中使用的不同：

https://docs.microsoft.com/en-us/aspnet/core/tutorials/first-mvc-app/validation

希望对您有所帮助。

Answer 3

为了检查 posted 模型是否有效，您需要在 post 操作中检查 ModelState.IsValid（参见下面的示例）。

Also, would appreciate advice about if I'm doing it right with the split of everything in the controller. Should there be 2 actions (1 for GET to just load the empty form and 1 for POST when the user submits it)? Am I doing something wrong there?

方法是对的，但你的做法不对。

假设您有一个 Register.cshtml 视图（不是局部视图）：由于视图具有模型，因此您应该始终提供它。所以你的 get 方法应该是这样的：

public ActionResult Register()
{
    ViewBag.LoggedIn = false;
    var model =  new User();
    return View(model);
}

对于post方法有两种方法：

POST-REDIRECT-GET 模式 (PRG) - 有点复杂（但在我看来这是更正确的做法）。 Post 方法而不是 return 视图应该 return 在错误和成功的情况下重定向结果。为了处理无效的模型状态，您可以使用 here

你的方式

[HttpPost]
public ActionResult Register(User user)
{
  ViewBag.LoggedIn = false;
  if(!this.ModelState.IsValid)
  {
     return View(user);
     //this will render a view with preserved invalid model state so all the values with the corresponding error messages will be shown.
  }

  user.Password = PasswordHash.CreateHash(user.Password);

  using (var context = new FoundationContext())
  {
      // if user exists
      if (context.Users.FirstOrDefault(n => n.Email == user.Email) != null) return Login(true);

      context.Users.Add(user);
      context.SaveChanges();
  }

  return RedirectToAction("Index", "Home");
}

无法修复 ASP.NET MVC 验证

Can't Fix ASP.NET MVC Validation

c#

asp.net-mvc

unobtrusive-validation