Web API 2.0异常处理

Question

我正在使用WebAPI2.0创建我自己的项目。我的Api包含添加、获取和预订产品的功能。

我想处理异常，但有一些问题我有点困惑。

我有一个控制器：ProdctController与动作方法addProdct。

   [Route("~/CKService.svc/author/add")]
    [HttpPost]
    public IHttpActionResult AddProduct(ProductRequest request)
    {
        ProductManager.AddProduct(request.ProductId, request.ProductName, 
        request.Price);

        return Ok(new AddProductResponse()
        {
            Status = AddProductStatus.Success
        };)
    }

现在，如果产品已经存在，那么数据访问层将抛出DuplicateKeyException

我想处理这个问题，并像这样回复：

  return Ok(new AddProductResponse()
    {
        Status = AddProductStatus.AlreadyExists
    };)

1. 返回HTTP状态代码200是否合理
2. 如果不向控制器添加catch DuplicateKeyException，我怎么做，因为我认为这不是正确的方法

谢谢

Answer 1

HTTP 200是不共振的-它指示成功的请求。使用4xx（错误）代码代替。409（冲突）适合您已经存在的对象的情况。

请参阅：由于现有匹配资源而导致资源创建POST失败时的HTTP响应代码

捕捉异常是完全可以的。未经处理的异常将导致500错误（内部服务器错误），这无论如何都没有意义。相反，您应该捕获异常并返回如下HTTP错误（异常处理ASP.NET）：

 throw new HttpResponseException(
        Request.CreateErrorResponse(HttpStatusCode.Conflict, message))

Answer 2

这里有一个建议。（我要强调的是，这只是您可以选择的众多选项之一）

重构manager以处理并返回添加产品的状态，然后让操作根据状态返回相应的响应代码。

[Route("~/CKService.svc/author/add")]
[HttpPost]
public IHttpActionResult AddProduct(ProductRequest request) {
    var status = ProductManager.AddProduct(request.ProductId, request.ProductName, request.Price);
    var response = new AddProductResponse() {
        Status = status
    };
    if(status == AddProductStatus.Success) {
        return Ok(response);
    }
    return BadRequest(response);
}

对于未按预期完成的请求，避免返回200 OK。这将误导提出请求的客户。