问题描述
无论操作是否很长，始终在我的操作中添加CancerationToken是不是一个好做法？
我当前正在将其添加到每个操作，我不知道它是对还是错。
[ApiController]
[Route("api/[controller]")]
public class DummiesController : ControllerBase
{
private readonly AppDbContext _dbContext;
public DummyController(AppDbContext dbContext)
{
_dbContext = dbContext;
}
[HttpGet("{id}")]
public async Task<ActionResult<Dummy>> GetAsync(int id, CancellationToken ct) // <<----- should I always do this?
{
var dummy = await _dbContext.Dummies.AsNoTracking().SingleOrDefaultAsync(e=>e.Id == id, ct);
if (dummy == null) return NotFound();
return dummy;
}
}
还需要添加CancellationToken ct = default(CancellationToken)吗？
推荐答案
是否应始终将CancerationToken添加到我的控制器操作？
不。您不应总是。
在ASP.NET核心MVC控制器中使用CancerationToken https://andrewlock.net/using-cancellationtokens-in-asp-net-core-mvc-controllers/
这是否正确将取决于您的应用程序。如果 请求修改状态，则您可能不想暂停执行 方法中途。另一方面，如果请求没有 副作用，那么你可能想要停止(可能是 代价高昂)尽快采取行动。
因此，如果您有如下所示的方法/操作(简化)；
await ProcessOrder();
await UpdateInventory();
您不希望在订单处理过程中取消订单，如果是这样，订单可以完成，但如果用户通过隧道并断开Internet连接，您将不会更新库存。
当操作无法包含在类似工作单元的模式(例如分布式系统)中，并且应尝试将取消次数降至最低时，这一点尤其重要。