建议在哪里执行验证：ViewModel、模型还是控制器？

Question

我有一个注册页面，想要对重复的用户名和电子邮件地址执行一些验证（除了 ViewModel 上的 StringLength 和必需注释之外）。目前，当注册表单回发时，我在控制器中执行此验证。我不确定这是否是正确的地方。

我无法想象 ViewModel 是正确的位置，因为它需要 ViewModel 引用我的 UserRepository。在模型类中进行这种验证有意义吗？

如果是这样，我如何在模型上实现这一点，以便在将信息发送到我的存储库之前检查信息是否有效？

更新

我的控制器操作的代码：

if (ModelState.IsValid)
        {
            if (!_userRepository.Exists(registerViewModel.Username))
            {
                if (!_userRepository.EmailExists(registerViewModel.Email))
                {
                    _userRepository.Add(
                        new User
                            {
                                Created = DateTime.Now, 
                                Email = registerViewModel.Email, 
                                Password = registerViewModel.Password, 
                                Username = registerViewModel.Username
                            });

                    _userRepository.SaveChanges();
                    TempData["registrationDetails"] = registerViewModel;

                    return RedirectToAction("Confirm");
                }
                else
                {
                    ModelState.AddModelError(string.Empty, "This email address is already in use.");
                }
            }
            else
            {
                ModelState.AddModelError(string.Empty, "This username is already taken.");
            }
        }

        return View(registerViewModel);
    }

更新2

域模型是否应该关心重复的用户名或电子邮件地址等约束，或者这是控制器层应该担心的事情？

更新3

似乎将验证逻辑放入控制器中最有意义，因为它可以在远程验证和提交时的模型验证中重用。检查重复项之类的事情通常应该在控制器中完成，或者在域模型中进行此类检查是否有意义？

谢谢，

Answer 1

我会在前端（可能是ajax）和后端上执行它 - 这取决于您的解决方案架构。

如果出现注册问题，我希望立即让用户知道。
在数据层/业务层和表示层的典型设置中，我将在业务逻辑中执行重复检查，并让控制器调用该代码段（除了在前端为用户进行 ajax 查找之外）。

顺便说一句：我通常更喜欢在 Windows 应用程序中只使用 MVVM（带有视图模型）。将 MVC 与 MVVM 结合起来会使事情变得不必要的复杂

Answer 2

我建议您在控制器中执行此操作。

主要原因是，无论应用程序的结构如何，您肯定需要使用 Ajax 来通知用户用户名是否已被占用。否则，这只是糟糕的可用性，无法证明您的代码结构合理。

为此，您希望有一个操作方法，可以通过ajax方式查看用户名是否存在。

总的来说，这确实意味着您最终可能会得到两套验证方法（UI 和模型），但这都是有充分理由的。

另外，为了制作一个良好的可用网站，您肯定会使用 Javascript 框架，例如 knockoutjs 或backbone。这就是真正的 MVVM，在这种情况下，将 ViewModel 作为模型层类（正如 Chris 提到的）并不是一个好主意。所以基本上你最终会得到两组验证。

Answer 3

正如 Rob 在评论中所说，这确实取决于您的应用程序的结构。我倾向于进行双重验证（对我的 ViewModel 进行验证以及对我的服务层进行验证），以确保访问服务的数据有效。这很有帮助，因为我的服务被多个客户端使用，但视图和 ViewModel 是特定于客户端的。前端验证具有即时反馈的优势，后端验证有助于保持数据干净。

Answer 4

我的建议是 - 对于验证，例如重复的电子邮件地址和用户名 - 将验证方法保留在控制器中。

或者在您的验证层中 - 将在 MVC3 的视图模型和数据层之间使用，

您将能够使用 Remote 属性将控制器中的方法作为操作添加到视图模型中的属性以获得即时结果

Answer 5

您应该在视图（客户端）和控制器（服务器端）中进行验证。如果您有 MVC 3，则可以使用新的 远程属性。它使用 jQuery 进行服务器端调用，您可以在其中检查用户或电子邮件地址是否存在。