Требования для 404

Ниже приведены мои требования для решения 404 и ниже Я показываю, как я это реализую:

• Я хочу обрабатывать совпадающие маршруты с плохими действиями
• Я хочу обрабатывать совпадающие маршруты с плохими контроллерами
• Я хочу обрабатывать несовпадающие маршруты (произвольные URL-адреса, которые может ' я не понимаю) - я не хочу, чтобы они поднимались до Global.asax или IIS, потому что тогда я не смогу правильно перенаправить обратно в свое приложение MVC
• Мне нужен способ обработки таким же образом, как указано выше , пользовательские 404-е - например, когда идентификатор отправляется для объекта, который не существует (возможно, удален)
• Я хочу, чтобы все мои 404-е возвращали представление MVC (а не статическую страницу), на которое я могу перекачать больше данных позже, если это необходимо ( хороший дизайн 404 ) и они должны возвращать код состояния HTTP 404

Решение

Я думаю, вам следует сохранить Application_Error в Global.asax для более высоких таких как необработанные исключения и ведение журнала (например, ответ Шэя Якоби показывает), но не обработку 404. Вот почему я предлагаю убрать 404 из файла Global.asax.

Шаг 1: Найдите общее место для логики ошибок 404

Это хорошая идея для удобства сопровождения. Используйте ErrorController , чтобы можно было легко адаптировать будущие улучшения вашей хорошо продуманной страницы 404 . Кроме того, убедитесь, что ваш ответ содержит код 404 !

public class ErrorController : MyController
{
    #region Http404

    public ActionResult Http404(string url)
    {
        Response.StatusCode = (int)HttpStatusCode.NotFound;
        var model = new NotFoundViewModel();
        // If the url is relative ('NotFound' route) then replace with Requested path
        model.RequestedUrl = Request.Url.OriginalString.Contains(url) & Request.Url.OriginalString != url ?
            Request.Url.OriginalString : url;
        // Dont get the user stuck in a 'retry loop' by
        // allowing the Referrer to be the same as the Request
        model.ReferrerUrl = Request.UrlReferrer != null &&
            Request.UrlReferrer.OriginalString != model.RequestedUrl ?
            Request.UrlReferrer.OriginalString : null;

        // TODO: insert ILogger here

        return View("NotFound", model);
    }
    public class NotFoundViewModel
    {
        public string RequestedUrl { get; set; }
        public string ReferrerUrl { get; set; }
    }

    #endregion
}

Шаг 2: Используйте базовый класс Controller, чтобы вы могли легко вызывать свое настраиваемое действие 404 и подключать HandleUnknownAction

404 в ASP.NET MVC нужно перехватывать в нескольких местах. Первый - HandleUnknownAction .

Метод InvokeHttp404 создает обычное место для перенаправления на ErrorController и наше новое действие Http404 . Подумайте DRY !

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
}

Шаг 3. Используйте внедрение зависимостей в фабрике контроллеров и подключите 404 HttpExceptions

Примерно так (это не обязательно должно быть StructureMap):

Пример MVC1.0:

public class StructureMapControllerFactory : DefaultControllerFactory
{
    protected override IController GetControllerInstance(Type controllerType)
    {
        try
        {
            if (controllerType == null)
                return base.GetControllerInstance(controllerType);
        }
        catch (HttpException ex)
        {
            if (ex.GetHttpCode() == (int)HttpStatusCode.NotFound)
            {
                IController errorController = ObjectFactory.GetInstance<ErrorController>();
                ((ErrorController)errorController).InvokeHttp404(RequestContext.HttpContext);

                return errorController;
            }
            else
                throw ex;
        }

        return ObjectFactory.GetInstance(controllerType) as Controller;
    }
}

Пример MVC2.0:

    protected override IController GetControllerInstance(RequestContext requestContext, Type controllerType)
    {
        try
        {
            if (controllerType == null)
                return base.GetControllerInstance(requestContext, controllerType);
        }
        catch (HttpException ex)
        {
            if (ex.GetHttpCode() == 404)
            {
                IController errorController = ObjectFactory.GetInstance<ErrorController>();
                ((ErrorController)errorController).InvokeHttp404(requestContext.HttpContext);

                return errorController;
            }
            else
                throw ex;
        }

        return ObjectFactory.GetInstance(controllerType) as Controller;
    }

Я думаю, что лучше выявлять ошибки ближе к тому месту, где они возникают. Вот почему я предпочитаю описанное выше обработчику Application_Error .

Это второе место, где можно поймать ошибку 404.

Шаг 4. Добавьте маршрут NotFound в Global.asax для URL-адресов, которые не удалось проанализировать в вашем приложении.

Этот маршрут должен указывать на наше действие Http404 . Обратите внимание, что параметр url будет относительным URL-адресом, потому что механизм маршрутизации отсекает здесь часть домена? Вот почему у нас есть вся эта логика условного URL-адреса на шаге 1.

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

Это третье и последнее место, где можно поймать 404 в приложении MVC, которое вы не вызываете сами.Если вы не поймаете здесь несовпадающие маршруты, тогда MVC передаст проблему ASP.NET (Global.asax), и вам это не нужно в этой ситуации.

Шаг 5: Наконец, вызовите 404, когда ваше приложение не может что-то найти

Например, когда в мой контроллер ссуд (получен из MyController ) отправляется неверный идентификатор:

    //
    // GET: /Detail/ID

    public ActionResult Detail(int ID)
    {
        Loan loan = this._svc.GetLoans().WithID(ID);
        if (loan == null)
            return this.InvokeHttp404(HttpContext);
        else
            return View(loan);
    }

Было бы неплохо, если бы все это можно было подключить в меньшем количестве мест с меньшим количеством кода, но я думаю, что это решение более удобное в обслуживании, более тестируемое и довольно прагматичное.

Спасибо за обратную связь. Я бы хотел получить больше.

ПРИМЕЧАНИЕ: это было значительно отредактировано по сравнению с моим исходным ответом, но цель / требования те же - вот почему я не добавил новый ответ

Мне очень нравится решение Коттсакса, и я думаю, что оно очень четко объяснено. Мое единственное дополнение - изменить шаг 2 следующим образом

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
}

В основном это останавливает URL, содержащие некорректные действия И контроллеры, от запуска процедуры исключения дважды. Например, для таких URL, как asdfsdf/dfgdfgd

Единственный способ заставить метод @cottsak работать с недопустимыми контроллерами - это изменить существующий запрос маршрута в CustomControllerFactory, например:

public class CustomControllerFactory : DefaultControllerFactory
{
    protected override IController GetControllerInstance(RequestContext requestContext, Type controllerType)
    {
        try
        {
            if (controllerType == null)
                return base.GetControllerInstance(requestContext, controllerType); 
            else
                return ObjectFactory.GetInstance(controllerType) as Controller;
        }
        catch (HttpException ex)
        {
            if (ex.GetHttpCode() == (int)HttpStatusCode.NotFound)
            {
                requestContext.RouteData.Values["controller"] = "Error";
                requestContext.RouteData.Values["action"] = "Http404";
                requestContext.RouteData.Values.Add("url", requestContext.HttpContext.Request.Url.OriginalString);

                return ObjectFactory.GetInstance<ErrorController>();
            }
            else
                throw ex;
        }
    }
}

Я должен упомянуть, что использую MVC 2.0.