asp.net – Web Api 2 – 返回NotFound(); vs使用全局异常处理程

比较这两种处理控制器ItemsController中“错误”的方法

A.使用从System.Web.Http.Results创建对象的方法

// GET api/user/userID/item/itemID
[Route("{itemID:int}",Name="GetItem")]
[ResponseType(typeof(ItemDTO))]
public IHttpActionResult Get(int userID,int itemID)
{
    if (userID < 0 || itemID < 0) return BadRequest("Provided user id or item id is not valid");
    ItemDTO item = _repository.GetItem(itemID);

    if (item == null) return NotFound();

    if (item.UserID != userID) return BadRequest("Item userID does not match route userID");

    return Ok<ItemDTO>(item);
}

B.通过注册自定义全局异常处理程序来抛出可以捕获的异常

// ex) in WebApiConfig.cs
// config.Services.Replace(typeof(IExceptionHandler),new GlobalExceptionHandler());
public class GlobalExceptionHandler : ExceptionHandler
{
    public override void Handle(ExceptionHandlerContext context)
    {
        Exception exception = context.Exception;

        HttpException httpException = exception as HttpException;

        if (httpException != null)
        {
            context.Result = new SimpleErrorResult(context.Request,(HttpStatusCode)httpException.GetHttpCode(),httpException.Message);
            return;
        }

        if (exception is RootObjectNotFoundException)
        {
            context.Result = new SimpleErrorResult(context.Request,HttpStatusCode.NotFound,exception.Message);
            return;
        }

        if (exception is BadRouteParametersException || exception is RouteObjectPropertyMismatchException)
        {
            context.Result = new SimpleErrorResult(context.Request,HttpStatusCode.BadRequest,exception.Message);
            return;
        }

        if (exception is BusinessRuleViolationException)
        {
            context.Result = new SimpleErrorResult(context.Request,(HttpStatusCode)422,exception.Message);
            return;
        }

        context.Result = new SimpleErrorResult(context.Request,HttpStatusCode.InternalServerError,exception.Message);
    }
}

GET api/user/userID/item/itemID
[Route("{itemID:int}",int itemID)
{
    if (userID < 0 || itemID < 0)
        throw new BadRouteParametersException("Provided user or item ID is not valid");
    ItemDTO item = _repository.GetItem(itemID);

    if (item.UserID != userID)
        throw new RouteObjectPropertyMismatchException("Item userID does not match route userID");

    return Ok<ItemDTO>(item);
}

这两个看起来都是有效的选择.因为我能够返回System.Web.Http.Results对象,所以似乎解决方案A.是最好的.

但是考虑在我的_repository中我的GetItem方法是这样实现的

public ItemDTO GetItem(int itemId)
{
    ItemInfo itemInfo = ItemInfoProvider.GetItemInfo(itemId);

    if (itemInfo == null) throw new RootObjectNotFoundException("Item not found");

    ItemDTO item = _autoMapper.Map<ItemDTO>(itemInfo);
    return item;
}

在这里,我可以跳过在GetItem中调用null的autoMapper,并跳过在控制器中检查null.

问题

>哪种方式更有意义？
>我应该尝试A&组合B’
>我应该尝试让我的控制器保持稀薄还是应该进行这种类型的验证&因为我可以访问NotFound()和BadRequest()方法,所以处理逻辑会保留在那里吗？
>我应该在框架管道中的其他位置执行此类逻辑吗？

我意识到我的问题更具体系结构,而不是“我如何使用此功能”,但是我还没有找到太多关于如何以及何时使用这些不同功能的解释.