当无效参数传递到 ASP.NET MVC 控制器时，如何返回 404 状态？

Question

如果将无效参数传递给我的控制器，我想返回 HTTP 状态 404。例如，如果我有一个看起来像这样的控制器：

public ActionResult GetAccount(int id)
{
   ...
}

那么如果遇到这样的 url，我想返回 404：

/GetAccount
/GetAccount/notanumber

即我想捕获 ArgumentException 这是抛出。

我知道我可以使用可为空的类型：

public ActionResult GetAccount(int? id)
{
  if(id == null) throw new HttpException(404, "Not found");
}

但这非常令人讨厌且重复。

我希望我可以在必要时将其添加到我的控制器中：

[HandleError(View="Error404", ExceptionType = typeof(ArgumentException))]
public class AccountsController : Controller
{
  public ActionResult GetAccount(int id)
  {
    ...
  }
}

但这似乎效果不佳。

我看到了这篇文章和这个答案几乎解决了我的问题：

在这个答案中，有一个摘要BaseController 是从中派生出所有其他控制器的：

public abstract class MyController : Controller
{
    #region Http404 handling

    protected override void HandleUnknownAction(string actionName)
    {
        // If controller is ErrorController dont 'nest' exceptions
        if (this.GetType() != typeof(ErrorController))
            this.InvokeHttp404(HttpContext);
    }

    public ActionResult InvokeHttp404(HttpContextBase httpContext)
    {
        IController errorController = ObjectFactory.GetInstance<ErrorController>();
        var errorRoute = new RouteData();
        errorRoute.Values.Add("controller", "Error");
        errorRoute.Values.Add("action", "Http404");
        errorRoute.Values.Add("url", httpContext.Request.Url.OriginalString);
        errorController.Execute(new RequestContext(
             httpContext, errorRoute));

        return new EmptyResult();
    }

    #endregion
}

这非常适合使用 404 处理未知操作，但不允许我将无效数据作为 <代码>404。

我可以安全地覆盖 Controller.OnException (ExceptionContext filterContext) 像这样：

protected override void OnException(ExceptionContext filterContext)
{
  if(filterContext.Exception.GetType() == typeof(ArgumentException))
  {
    filterContext.ExceptionHandled = true;
    this.InvokeHttp404(filterContext.HttpContext);
  }
  else
  {
    base.OnException(filterContext);
  }
}

从表面上看，它似乎有效，但是我这样做是否会带来任何问题？

这在语义上正确吗？

Answer 1

最好的办法？动作方法选择器属性！

为了真正避免可为空的方法参数，我建议您编写一个 Action方法选择器属性实际上仅在提供id时匹配您的操作方法。它不会说未提供参数，但它无法匹配给定请求的任何操作方法。

我会将此操作选择器称为 RequireRouteValuesAttribute 并以这种方式工作：

[RequireRouteValues("id")]
public ActionResult GetAccount(int id)
{
    ...
}

为什么这是解决您问题的最佳解决方案？

如果您查看代码，您希望在与名称匹配但参数绑定失败的操作上返回 404（因为未提供或任何其他原因）。可以说，您的操作需要特定的操作参数，否则将返回 404。

因此，当添加操作选择器属性时，会添加对操作的要求，因此它必须匹配名称（由 MVC 给出）并且还需要特定的操作参数。每当未提供 id 时，此操作就不匹配。如果有另一个匹配的操作不是这里的问题，因为该特定操作将被执行。主要的事情已经完成了。操作与无效路由请求不匹配，而是返回 404。

有一个应用程序代码！

检查我的博客文章实现了这种属性您可以开箱即用。它完全符合您的要求：如果提供的路线数据不包含所有必需的值，它将与您的操作方法不匹配。

Answer 2

免责声明：这并不能涵盖所有情况

对于示例中的网址，可以在单行中返回 404。只需为 id 参数添加路由约束即可。

routes.MapRoute(
    "Default", // Route name
    "{controller}/{action}/{id}", // URL with parameters
    new { controller = "Home", action = "Index" }, // Parameter defaults
    new { id = @"\d+" } // restrict id to be required and numeric
);

仅此而已。现在，任何没有 id 或 id 的匹配网址都不是数字，会自动触发未找到错误（对此有很多方法可以处理，在您的示例中是一种，另一种是通过使用自定义 HandleErrorAttribute 等）。您可以在操作中使用不可为 null 的 int 参数。

Answer 3

我设法通过在所有路由的末尾添加此路由来实现此工作：

routes.MapRoute("CatchAllErrors", "{*url}",
    new { controller = "Error", action = "NotFound" }
);

注意：首先我遵循以下操作： 如何正确处理 404在 ASP.NET MVC 中？