修改ApplicationUser属性遇空引用错误及无更新问题求助

看起来你在更新Identity用户属性时遇到了两个典型问题：空引用异常和修改不生效。我来帮你拆解问题原因，并给出更简洁可靠的实现方式。

第一个代码的空引用原因 #

你第一段代码里的Usermanager（注意规范大小写应该是UserManager）大概率是未初始化的控制器属性。如果你的控制器没有通过依赖注入注入UserManager<ApplicationUser>，直接调用Usermanager.UpdateAsync(model)就会抛出空引用异常——因为这个变量根本没有指向任何实例。

第二段代码修改不生效的可能原因 #

你调整后的代码手动实例化了UserStore和UserManager，虽然能避免空引用，但存在几个隐患：

• 手动创建UserManager不是ASP.NET Identity的推荐实践，容易因为多DbContext实例导致上下文跟踪冲突，修改无法同步到数据库；
• 你没有检查UpdateAsync的返回结果，就算更新失败（比如邮箱已经被其他用户占用），也直接跳转，看不到错误；
• 模型绑定的ApplicationUser user可能因为表单字段和属性名不匹配，导致传入的属性值为空，修改等于没做。

推荐的简洁实现方式 #

最佳实践是通过依赖注入获取UserManager，它已经封装了所有用户操作的逻辑，不需要手动操作DbContext。

第一步：在控制器中注入UserManager #

确保你的控制器构造函数接收UserManager<ApplicationUser>：

private readonly UserManager<ApplicationUser> _userManager;

// 构造函数注入UserManager
public AdminController(UserManager<ApplicationUser> userManager)
{
    _userManager = userManager;
}

第二步：修改EditUserData Action #

[HttpPost]
[ValidateAntiForgeryToken]
public async Task<ActionResult> EditUserData(string fullName, string website, string email)
{
    // 获取当前登录用户ID
    var userId = User.Identity.GetUserId();
    // 通过UserManager异步查找用户（避免阻塞线程）
    var user = await _userManager.FindByIdAsync(userId);

    if (user == null)
    {
        // 处理用户不存在的情况
        return NotFound("当前用户不存在");
    }

    // 更新用户属性
    user.Email = email;
    user.UserName = email; // 注意：UserName通常有唯一约束，要确保邮箱未被其他用户占用
    user.FullName = fullName;
    user.Website = website;

    // 执行更新并获取结果
    var result = await _userManager.UpdateAsync(user);

    if (result.Succeeded)
    {
        // 更新成功，跳转设置页面
        return RedirectToAction("Settings", "Admin");
    }
    else
    {
        // 更新失败，把错误信息添加到ModelState，返回页面显示给用户
        foreach (var error in result.Errors)
        {
            ModelState.AddModelError(string.Empty, error.Description);
        }
        // 返回设置页面视图，携带错误信息
        return View("Settings");
    }
}

额外注意事项 #

• 把UserName设为Email时，建议在Startup的Identity配置中开启唯一邮箱约束：options.User.RequireUniqueEmail = true，避免重复值导致更新失败；
• 永远不要忽略IdentityResult的返回值，它包含了所有更新失败的具体原因（比如字段长度超限、格式错误等）；
• 避免手动实例化UserStore和UserManager，依赖注入会帮你处理好DbContext的生命周期和资源释放，减少潜在bug。

这样不仅能解决你遇到的两个问题，代码也更简洁易维护。

内容的提问来源于stack exchange，提问作者Kristian Bendtsen