ASP.NET Identity UserManager.UpdateAsync返回null问题求助

你遇到的UserManager.UpdateAsync返回null（或更新失效）的核心原因是你没有使用数据库中已被上下文跟踪的用户实体，而是新建了一个完全独立的ApplicationUser对象去执行更新操作。

具体问题点 #

当你调用manager.FindByNameAsync(model.UserName)时，已经从数据库获取了一个被DbContext跟踪的user对象，但你随后创建了全新的updateUser实例——这个对象并没有和当前DbContext建立关联，UserManager无法识别它对应数据库中的哪条记录，自然无法完成有效更新。

修正后的代码 #

直接修改从数据库查询到的user对象，再调用UpdateAsync即可解决问题：

[HttpPost]
[Route("api/ChangeProfile")]
[ResponseType(typeof(AccountModel))]
public async Task<IHttpActionResult> ChangeProfile([FromBody]AccountModel model)
{
    // 备注：后续建议通过依赖注入获取UserManager，而非每次新建实例
    var userStore = new UserStore<ApplicationUser>(new ApplicationDbContext());
    var manager = new UserManager<ApplicationUser>(userStore);
    
    var user = await manager.FindByNameAsync(model.UserName);
    if (user != null)
    {
        // 直接修改已被上下文跟踪的user对象属性
        user.Email = model.Email;
        user.FirstName = model.FirstName;
        user.LastName = model.LastName;
        // 注意：若业务允许修改UserName，需先验证新名称的唯一性
        
        IdentityResult result = await manager.UpdateAsync(user);
        
        if (result.Succeeded)
        {
            return Ok(result);
        }
        else
        {
            // 返回具体错误信息，方便前端调试
            return BadRequest(string.Join(", ", result.Errors));
        }
    }
    else
    {
        return NotFound();
    }
}

额外优化建议 #

1. 通过依赖注入管理UserManager
   每次在方法内新建UserStore和UserManager会导致DbContext生命周期混乱，建议在Startup类中配置依赖注入，然后在控制器构造函数中注入UserManager<ApplicationUser>：

   private readonly UserManager<ApplicationUser> _userManager;
   
   public YourController(UserManager<ApplicationUser> userManager)
   {
       _userManager = userManager;
   }
   

   这样能避免重复创建实例，同时保证DbContext的正确跟踪。

2. 严格校验更新结果
   不要直接返回Ok(result)，先检查result.Succeeded状态，失败时把result.Errors中的具体错误返回给前端，便于定位问题。

3. UserName修改的特殊处理
   如果业务允许修改UserName，必须先验证新名称在系统中的唯一性，否则UpdateAsync会返回重复名称的错误。

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