.NET MVC：如何在辅助方法中重定向响应

Question

我有一些代码对于我的一些控制器操作是通用的，这些代码被提取到一个私有“帮助器”方法中，只是为了整合。该方法预计会为我“获取”一个对象，尽管它会执行一些健全性/安全性检查，并且可能会将用户重定向到其他操作。

private Thingy GetThingy(int id)
{
    var thingy = some_call_to_service_layer(id);

    if( null == thingy )
        Response.Redirect( anotherActionUrl );

    // ... many more checks, all more complex than a null check

    return thingy;
}

public ActionResult ActionOne(int id)
{
    var thingy = GetThingy(id);
    // do more stuff
    return View();
}

// ... n more actions

public ActionResult ActionM(int id)
{
    var thingy = GetThingy(id);
    // do more stuff
    return View();
}

这可以正常运行，但 Elmah 会通知我一个异常：

System.Web.HttpException: Cannot redirect after HTTP headers have been sent.

所以，我的问题是：有没有更正确的方法来做我想做的事情？本质上，我想要的只是使当前操作停止处理，而是返回一个 RedirectToRouteResult。

Answer 1

你不能在一个动作中调用 Response.Redirect() 而不破坏执行流程导致错误。相反，您可以返回 RedirectToRouteResult 对象或 RedirectResult 对象。在你的行动中

return Redirect("/path");
//or
return RedirectToAction("actionname");

在您的情况下，但是您想返回一个 Thingy 对象，您需要分离出逻辑。您可以执行以下操作（我假设您想要重定向到不同的操作，否则 Oenning 的代码将起作用）

public ActionResult Get(int id) {

  var thingy = GetThingy(id);

  var result = checkThingy(thingy);
  if (result != null) {
    return result;
  }

  //continue...
}

[NonAction]
private ActionResult CheckThingy(Thingy thingy) {

  //run check on thingy
  //return new RedirectResult("path");

  //run another check
  //new RedirectResult("different/path");

  return null;
}

更新您可以将此代码放在扩展方法或控制器基类中

public static class ThingyExtensions {

  public static ActionResult Check(this Thingy thingy) {
    //run checks here
  }

}

Answer 2

一种方法是定义一个异常过滤器来为您处理重定向。

首先，创建一个自定义异常来表示您的重定向：

public class RedirectException : Exception {

    private readonly string _url;

    public RedirectException(string url) {
        _url = url;
    }

    public string Url { get { return _url; }}

}

然后，定义您的异常过滤器：

public class RedirectExceptionAttribute : FilterAttribute, IExceptionFilter {
    public void OnException(ExceptionContext filterContext) {

        if (filterContext.ExceptionHandled) return;
        if (filterContext.Exception.GetType() != typeof(RedirectException)) return;

        filterContext.Result = new RedirectResult(((RedirectException)filterContext.Exception).Url);
        filterContext.ExceptionHandled = true;
    }
}

全局注册新的异常过滤器，使其适用于所有控制器操作（如果这是您想要的）：

// in FilterConfig.cs (MVC 4.0)
public static void RegisterGlobalFilters(GlobalFilterCollection filters) {
    filters.Add(new RedirectExceptionAttribute());
}

现在，无论你想重定向到哪里，只要抛出一个 RedirectException：

private Thingy GetThingy(int id)
{
    var thingy = some_call_to_service_layer(id);

    if( null == thingy )
        throw new RedirectException( anotherActionUrl );

    // ... many more checks, all more complex than a null check

    return thingy;
}

public ActionResult ActionOne(int id)
{
    var thingy = GetThingy(id);
    // do more stuff
    return View();
}

// ... n more actions

public ActionResult ActionM(int id)
{
    var thingy = GetThingy(id);
    // do more stuff
    return View();
}

当然，您需要确保您没有在 try/catch 中调用 GetThingy()，或者如果您这样做了，请确保您重新抛出异常。

Answer 3

尝试：

return RedirectToAction("youraction", "yourcontroller");

希望对你有帮助。

Answer 4

这个怎么样。

public ActionResult ActionOne(int id)
{
    var thingy = GetThingy(id);
    if (thingy == null)
        return RedirectToAction("action", "controller");
    // do more stuff
    return View();
}

private Thingy GetThingy(int id)
{
    var thingy = some_call_to_service_layer(id);

    if( null == thingy )
        return null;

    // ... many more checks, all more complex than a null check

    return thingy;
}

顺便说一句，恕我直言，GetThingy(int id) 方法应该放在其他地方。也许和some_call_to_service_layer(id)在同一个地方。

Answer 5

好吧，我想我明白了。显然，您可以任意触发 ActionResult 的执行，仅用于此类事情。所以辅助方法变成了：

private Thingy GetThingy(int id)
{
    var thingy = some_call_to_service_layer(id);

    if( null == thingy )
        RedirectToAction("myAction").ExecuteResult(ControllerContext);

    // ... many more checks, all more complex than a null check

    return thingy;
}

运行良好，并允许我将代码保留在我想要的位置。很甜。