Cleanly Umgang mit normalen Fehlern

Question

Ich benutze dieses Muster überall Daten aus der Datenbank zu packen und eine Ansicht anzuzeigen:

public ActionResult Index(int? id) 
    { 
     RequestViewModel model; 
     model = this.ClientRepository.GetRequest(id); 

     return View("~/Views/Requests/Index.aspx", model); 
    } 

Wenn das Repository null zurückgibt, was der Fall ist, wenn der Datensatz nicht existiert, dann scheißt meine Seite und gibt einen Fehler aus, weil das Modell null ist.

Ich möchte eine freundliche "die angeforderte Datensatz kann nicht gefunden werden" Nachricht statt der gelben Seite des Todes oder eine generische "ein Fehler aufgetreten" Seite.

Was ist das empfohlene Muster, um "normale" Fehler im Gegensatz zu unbehandelten Ausnahmen zu behandeln?

Danke,

Rick

Answer 1

Sie ein Action-Filter schreiben könnte:

public class NullModelCheckerAttribute : ActionFilterAttribute 
{ 
    public override void OnActionExecuted(ActionExecutedContext filterContext) 
    { 
     base.OnActionExecuted(filterContext); 
     var viewResult = filterContext.Result as ViewResultBase; 
     if (viewResult != null && viewResult.ViewData.Model == null) 
     { 
      // If the action selected a view to render and passed a null model 
      // render the NotFound.aspx view 
      var result = new ViewResult(); 
      result.ViewName = "~/Views/Errors/NotFound.aspx"; 
      filterContext.HttpContext.Response.StatusCode = 404; 
      filterContext.Result = result; 
     } 
    } 
} 

Und dann Ihre Basis-Controller dekorieren mit diesem Attribut (dass alle Steuerungen von ableiten):

[NullModelChecker] 
public class BaseController: Controller 
{ } 

Auf diese Weise bleibt Ihr aktueller Code unberührt.

- UPDATE:

In ASP.NET MVC 3 könnten Sie registrieren Ihre action filter globally, ohne auch nur mit ihm Ihre Basis-Controller dekorieren. Fügen Sie einfach die folgende zu Ihrem Application_Start in Global.asax:

GlobalFilters.Filters.Add(new NullModelCheckerAttribute()); 

Answer 2

ich mit ASP.NET MVC nicht vertraut bin. Ich bin jedoch vertraut mit Spring MVC.

Warum können Sie nicht einfach eine einfache Wenn-sonst-Bedingung setzen? Wie dieser:

public ActionResult Index(int? id) 
{ 
    RequestViewModel model; 
    model = this.ClientRepository.GetRequest(id); 

    if (model == null) { 
     return View("~/Views/Requests/FriendlyError.aspx"); 
    } 

    return View("~/Views/Requests/Index.aspx", model); 
}