使用Try/Catch伪同步处理异步API调用是否合理？附实现疑问

当前实现的问题 #

你的代码存在几个关键问题：

• 操作顺序错误：需求明确要求获取Items必须在Schedule写入数据库（POST完成）后执行，但你先执行了Items查询，再调用SaveChangesAsync，不符合要求。
• 空catch块吞异常：两个空catch会隐藏所有错误，导致调试困难，无法排查数据库操作失败的原因。
• finally块逻辑不合理：无论Schedule添加是否成功，都会执行Items查询，这不符合“POST完成后执行”的前提。

是否需要将每个操作设为Task？ #

不需要。OnPostAsync本身就是异步方法（返回Task<IActionResult>），你调用的数据库操作方法（SaveChangesAsync、ToArrayAsync）本身就返回Task类型，直接用await等待这些异步操作完成即可，无需额外将每个操作包装成独立的Task。

修正后的实现代码 #

public async Task<IActionResult> OnPostAsync()
{
    if (!ModelState.IsValid || _context.Schedules == null || Schedule == null)
        return Page();

    try
    {
        _context.Schedules.Add(Schedule);
        // 先完成Schedule的写入操作（POST到数据库）
        await _context.SaveChangesAsync();

        // 写入完成后，获取关联的Items
        Items = await _context.BoMItems.Include(m => m.Item)
            .Where(m => m.BOMId == Schedule.BOMId)
            .Where(m => m.IsRunnerItem == true)
            .ToArrayAsync();
    }
    catch (Exception ex)
    {
        // 这里添加异常处理逻辑，比如记录日志、返回错误提示
        // 示例：ModelState.AddModelError(string.Empty, "操作失败：" + ex.Message);
        return Page();
    }

    return RedirectToPage("./Index");
}

关键说明 #

1. 操作顺序调整：确保SaveChangesAsync（完成Schedule写入）执行完成后，再进行Items查询，严格符合需求的时序要求。
2. 异常处理优化：替换空catch块，添加实际的异常处理逻辑，便于问题排查和用户提示。
3. 异步操作正确使用：直接await数据库提供的异步方法即可，这些方法内部已经封装了Task逻辑，无需重复包装。

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