时间戳

首页 > 解决方案 > CreateAsync 方法既不抛出错误也不更新数据库

c# - CreateAsync 方法既不抛出错误也不更新数据库

问题描述

我有以下简单的代码来使用该CreateAsync方法创建一个新用户。该代码不会引发任何错误，但也不会更新数据库。我IdentityResult result在行上添加了一个断点，它停止了，在if语句中添加了另一个断点，它不会停止。结果，我不知道如何调试这段代码并找到错误。有什么帮助吗？

public async Task<IdentityResult> Create(ApplicationUser user, string password)
{
    IdentityResult result = await _userManager.CreateAsync(user, password);

    if (!result.Succeeded)
    {
        throw new AppException("Failed");
    }

    return result;
}

从控制器调用此Create函数：

[AllowAnonymous]
[HttpPost]
[Route("api/ApplicationUser/Register")]
public IActionResult Register([FromBody]ApplicationUserDto userDto)
{
    //map dto to entity
    var user = _mapper.Map<ApplicationUser>(userDto);

    try
    {
        // save 
        _userService.Create(user, userDto.Password);
        return Ok();
    }
    catch (AppException ex)
    {
        // return error message if there was an exception
        return BadRequest(new { message = ex.Message });
    }   
}

该Register方法是从 React 接口调用的。

标签： c#asp.netasp.net-coreasync-awaitasp.net-identity

解决方案

你从来没有等待你的异步调用。因此，当您的操作结果返回时，它甚至可能还没有完成，因此可能永远不会运行。确保您等待任何异步调用。这意味着调用方法需要标记为异步（并使其返回一个任务或任务），一直到调用堆栈，直到它到达事件处理程序或框架代码。这是您的代码应如下所示：

[AllowAnonymous]
[HttpPost]
[Route("api/ApplicationUser/Register")]
public async Task<IActionResult> Register([FromBody]ApplicationUserDto userDto)
{
    //map dto to entity
    var user = _mapper.Map<ApplicationUser>(userDto);

    try
    {
        // save 
        await _userService.Create(user, userDto.Password);
        return Ok();
    }
    catch (AppException ex)
    {
        // return error message if there was an exception
        return BadRequest(new { message = ex.Message });
    }   
}

另请注意，将异常详细信息返回给客户端用户是个坏主意。异常可能包含攻击者可以用来利用您的系统的敏感信息。相反，只需返回一个通用的“抱歉，出现问题”消息并将异常详细信息记录到您的日志框架。如果您没有日志记录框架，请获取一个。

推荐阅读