ASP.NET MVC Обработка исключений с AJAX/JSON

Question

У меня есть несколько методов в контроллере, которые выглядят как:

[HttpPost] 
public ActionResult AddEditCommentToInvoice(string invoiceNumber, string comments) 
{ 
    var response = new { success = true, msg = "Comment saved", statusMsg = "Comment saved" }; 

    try 
    { 
     var recordsModified = invoiceService.AddCommentsToInvoice(invoiceNumber, comments); 
     Log.Info(recordsModified ? "Updated Comment" : "Did not update Comment"); 

    } catch (Exception ex) { 
     Response.StatusCode = (int)HttpStatusCode.InternalServerError; 
     return Json(new { 
      success = false, 
      msg = "There is missing field data", 
      statusMsg = ex.Message 
     }, JsonRequestBehavior.AllowGet); 
    } 

    return Json(response, JsonRequestBehavior.AllowGet); 
} 

Хотя этот код работает, я не комфортно с этим подходом, потому что:

1. Try/Задвижки дорогие
2. код улавливает System.Exception
3. код некрасиво

Теперь я знаю, что могу использовать OnException или атрибут HandleError.
Я также провел некоторое исследование ELMAH, и это выглядит многообещающим.

Но я все еще хочу вернуть JSON через AJAX моему пользователю, чтобы указать, была ли операция успешной или нет.

Итак, мой вопрос: кто-нибудь использовал любой из трех методов (или, в частности, ELMAH), чтобы вернуть JSON через AJAX?

Answer 1

Я использую другой подход, подход, который может применяться на уровне контроллера или глобально через GlobalFilters. В моих MVC контроллеров, вы можете переопределить OnActionExecuted метод, и сделать это:

protected override void OnActionExecuted(ActionExecutedContext filterContext) 
    { 
     if (filterContext.Exception != null) 
     { 
     filterContext.Result = Json(new { success = false }); 
     return; 
     } 

     base.OnActionExecuted(filterContext); 
    } 

Это также может быть сделано как атрибут действия фильтра. Вам не понадобится обработка исключений в ваших контроллерах - если возникает исключение, то это обрабатывается в контексте результата.