我有以下注册方法,我发誓我(手动)测试了一段时间并注意到如果用户名已经存在,结果只是 result.Succeeded 的错误值并将错误消息附加到ModelState(使用内置的 AddErrors(result) 辅助方法)。我很确定这种方法 (Register(...)
) 是 ASP.NET mvc 5 开箱即用的,但我想我改变了用户以包含用户名(而在框,电子邮件仅用作用户名)。
public async Task<ActionResult> Register(RegisterViewModel model)
{
if (ModelState.IsValid)
{
var user = new ApplicationUser { UserName = model.Username, Email = model.Email };
var result = await UserManager.CreateAsync(user, model.Password);
if (result.Succeeded)
{
await SignInManager.SignInAsync(user, isPersistent: false, rememberBrowser: false);
return RedirectToAction("Index", "Home");
}
AddErrors(result);
}
// If we got this far, something failed, redisplay form
return View(model);
}
相反,我目前收到的错误是 EntityValidationError 被抛出但未被捕获。
我知道我可以简单地捕捉到这个错误并继续我的一天,但我想确保如果不正确的行为是其他原因导致了这个问题。
更新:
在创建一个新的 MVC 项目后,我可以确认典型的行为(当注册了重复的用户名时)是 CreateAsync 应该返回一个 result.Succeeded 值为 false 的结果,并出现错误消息“用户名已被占用”应该附加到 ModelState。很明显我的代码或配置有问题,但我不知道从哪里开始探索。如果有帮助,我最近在我的代码的其他地方看到了 EntityValidationErrors,这些情况也不应该保证。请参阅:Unable to SaveChanges on a db update. Weird lazy loading behavior possibly?
最佳答案
我找到了自己的解决方案。正如我提到的,我改变了用户以包含用户名(以及使电子邮件可选)。此任务的一部分涉及创建自定义用户验证程序类。在自定义用户验证器的 ValidateAsync 方法中,我忘记检查用户名是否已经存在(并且不属于用户)。像这样:
async Task<IdentityResult> IIdentityValidator<TUser>.ValidateAsync(TUser item)
{
var errors = new List<string>();
// ...
// Piece of code I have now added
var owner = await _manager.FindByNameAsync(item.UserName);
if (owner != null && !EqualityComparer<string>.Default.Equals(owner.Id, item.Id))
{
errors.Add($"Username {item.UserName} is already taken");
}
// End of code I added
// ...
return errors.Any()
? IdentityResult.Failed(errors.ToArray())
: IdentityResult.Success;
}
我相信我学到的教训是 App 层验证之间的区别,其中验证发生在 UserManager 的 CreateAsync 方法中。在 App 层验证的情况下,错误将完全按照规定出现。如果省略了验证层,并且数据库面临相同的约束,那么在保存上下文时,它会抛出自己的错误。在这种情况下,一个稍微更神秘的 EntityValidationError。
关于c# - 为什么 UserManage.CreateAsync 在使用重复用户名时抛出 EntityValidationError 而不是返回失败结果?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/44620637/