是否应await无返回结果需求的异步数据库保存方法？

结论：必须await AddTeamToDbAsync，不await的风险远大于所谓的“快速返回”收益 #

以下是具体风险分析：

• 无法感知保存失败：不await的话，你永远不知道数据库操作是否成功——比如网络中断、数据库权限不足、约束冲突（哪怕提前检查了名称，并发场景下仍可能出现）等问题都会被忽略，直接返回“创建成功”的结果，但实际数据根本没写入数据库，这种数据不一致的问题排查难度极大。
• 并发场景下的重复数据风险：你提前做的CheckIfTeamExistsAsync和后续的保存操作不是原子性的，不await会拉长这个“检查-保存”的时间窗口，高并发时多个请求可能同时通过名称检查，最后都执行保存，导致重复数据入库。
• 未观察到的异常会引发严重问题：如果AddTeamToDbAsync抛出异常，未await的情况下这个异常会成为未观察到的任务异常，旧版本.NET中可能直接导致进程崩溃；就算新版本不会崩溃，异常也会被静默吞掉，你完全无法及时发现问题。
• 资源泄漏隐患：异步数据库操作会持有数据库连接等资源，不await可能导致这些资源无法及时释放，长期运行后可能耗尽连接池，导致整个应用无法访问数据库。

你认为“不await可更快返回结果”的理解在响应时间层面是事实，但这种“快”是以牺牲系统可靠性、数据一致性为代价的，完全不可取。

修正后的代码 #

public async Task<OperationResult<TeamDto>> AddTeamAsync(TeamForCreationDto team)
{
    var teamToAdd = _mapper.Map<Team>(team);
    bool IsNameTaken = await CheckIfTeamExistsAsync(team.Name);

    if (IsNameTaken)
    {
        return new OperationResult<TeamDto>
        {
            IsSuccess = false,
            ErrorMessage = "Provided name is already taken.",
            HttpResponseCode = 409
        };
    }
    else
    {
        await _teamRepository.AddTeamToDbAsync(teamToAdd);
        var teamToReturn = _mapper.Map<TeamDto>(teamToAdd);
        return new OperationResult<TeamDto>
        {
            IsSuccess = true,
            Data = teamToReturn,
            HttpResponseCode = 201
        };
    }
}

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