3

在使用 Mediatr 模式时,我发现将有意义的错误返回给 API 控制器非常具有挑战性。让我们以OrdersController.CancelOrder方法为例(src)。

在此示例中,它们“仅”返回Ok()and BadRequest()。在这种情况下,他们将如何返回诸如“此 orderid 不存在”(404)或“此订单已发货”(400)(...)之类的错误。

我们可以引入一个新类,称为Result保存返回值(如果有的话)和潜在的错误消息。在这种情况下,您的所有命令、查询都应该返回Result<YourModel>. 我们也可以直接在控制器中添加代码。我无法下定决心,两种解决方案都有优点和缺点。

你怎么看?

谢谢 Seb

4

1 回答 1

4

这正是我倾向于使用 Mediatr 的方式。
返回一个包装类。

如果我们以 eShopOnContainers 示例CancelOrder为例,我会有命令,返回一个CancelOrderCommandResult

public class CancelOrderCommand : IRequest<CancelOrderCommandResult>
{ }

CancelOrderCommandResult可能是这样的:

public class CancelOrderCommandResult
{
    public CancelOrderCommandResult(IEnumerable<Error> errors)
    {
        Success = false;
        Errors = errors;
    }

    public CancelOrderCommandResult(bool success)
    {
        Success = success;
    }

    public bool Success {get; set;}

    public IEnumerable<Error> Errors {get; set;}
}

我省略了这个Error类,但它可能只是包含错误信息、错误代码等的 POCO...

然后我们的处理程序变成

public class CancelOrderCommandHandler : IRequestHandler<CancelOrderCommand, CancelOrderCommandResult>
{
    private readonly IOrderRepository _orderRepository;

    public CancelOrderCommandHandler(IOrderRepository orderRepository)
    {
        _orderRepository = orderRepository;
    }

    public async Task<bool> Handle(CancelOrderCommand command, CancellationToken cancellationToken)
    {
        var orderToUpdate = await _orderRepository.GetAsync(command.OrderNumber);

        if(orderToUpdate == null)
        {
            return false;
        }

        try 
        {
            orderToUpdate.SetCancelledStatus();
            await _orderRepository.UnitOfWork.SaveEntitiesAsync();

            //iff success, return true
            return new CancelOrderCommandResult(true);
        }
        catch (Exception ex)
        {
            var errors = MapErrorsFromException(ex);
            return new CancelOrderCommandResult(errors)
        }
    }
}

同样,MapErrorsFromException为简洁起见省略,但您甚至可以将其作为依赖项注入。

在您的控制器中,当您调用时,_mediator.Send您现在返回CancelOrderCommandResult- 如果.Success为真,则像以前一样返回 200。

否则,您会有一系列错误 - 您可以使用这些错误来决定返回什么 - 400、500 等......

于 2018-06-06T05:04:32.857 回答