通过ID获取单个TodoItem失败，疑为控制器代码问题求助

你的控制器代码确实存在逻辑错误，这正是无法通过ID获取单个待办项的原因：

• 颠倒的判断逻辑：当前代码中if (id != null)就返回NotFoundResult，意味着只要传入了有效的ID，就直接返回404错误，完全搞反了校验逻辑。
• 缺少查询结果校验：即使修正判断逻辑，查询到的待办项也可能为null，此时同样需要返回NotFound。

修正后的代码 #

[HttpGet("{id}", Name = "GetTodoItem")]
public async Task<ActionResult<Todo>> GetTodoItem(int id)
{
    // 单个资源查询的ID通常为必填项，直接用非空int即可，框架会自动拦截无效ID并返回400
    var todo = await _context.Todos.FindAsync(id);

    if (todo == null)
    {
        return NotFound();
    }

    return Ok(todo);
}

补充说明 #

如果你的业务场景允许ID为可选参数，才需要用int? id，此时要先判断id == null返回BadRequest或NotFound，再执行查询操作。但针对单个待办项的查询接口，让ID为必填项是更符合REST规范的设计。

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