編集アクションルートでAsp.Net MVC 5が未処理の例外をスローするのを防ぐにはどうすればいいですか？

Question

私はAsp.Net MVC 5で標準の編集アクションを持っています。私は~/food/editのようなIDなしでリクエストを取得したときに未処理の例外をスローしないようにしたいので、これを行いました。

public ActionResult Edit(int id = 0) 
    { 
     if (id == 0) 
     { 
      return new HttpStatusCodeResult(HttpStatusCode.BadRequest); 
     } 

     string result = _foodAppService.GetById(id); 
     FoodVm food = string.IsNullOrEmpty(result) 
      ? null 
      : JsonConvert.DeserializeObject<FoodVm>(result); 

     if (food == null) 
     { 
      return RedirectToAction("Index"); 
     } 

     return View(food); 
    } 

私の質問は次のとおりです。これをこの方法で処理するのが良い方法ですか、より適切な方法がありますか？

私は別の方法で尋ねる必要がある場合は、私に知らせてください、あなたの時間をありがとう。その優れた

public ActionResult Edit(int? id) 
{ 
    if (!id.HasValue) 
    { 
     return new HttpStatusCodeResult(HttpStatusCode.BadRequest); 
    } 
} 

Answer 1

が有効である可能性があります。メソッドOnExceptionのオーバーライドを使用すると、1つのメソッドでコントローラ内のすべての例外をデバッグして処理することができます。また、ポールSwetzのsaiedとして入力パラメータの検証を使用し

public class BaseController : Controller 
{ 
    protected override void OnException(ExceptionContext filterContext) 
    { 
     string redirectUrl; 
     var exception = filterContext.Exception; 

     if (exception is EntityException) 
     { 
      redirectUrl = "/Content/error.html"; 
     } 
     else 
     { 
      redirectUrl = "/Info/Index"; 
     } 

     //do whatever you wont 

     Response.Redirect(redirectUrl); 
    } 

： はちょうどこのようなすべてのコントローラに基本クラスを追加します。この方法はより一般的であり、すべての例外を傍受することができ、ユーザーにエラーを表示しません。

Answer 2

を行うのか、MVCで例外を処理するために、より総合的な方法が存在することができた場合はゼロに

Answer 3

まず、Try-Catchを使用するのがよい方法です。

public ActionResult Edit(int id) 
{ 
    try 
    { 
      if (id != 0 || id!=null) 
      { 
      string result = _foodAppService.GetById(id); 
      FoodVm food = string.IsNullOrEmpty(result) ? null:JsonConvert.DeserializeObject<FoodVm>(result); 

      if (food == null) 
      { 
       return RedirectToAction("Index"); 
      } 
      else 
      {  
      return View(food); 
      } 
      } 
     else 
      { 
       return new HttpStatusCodeResult(HttpStatusCode.BadRequest); 
      } 
    } 
    catch (exception ex) 
    { 
    return new HttpStatusCodeResult(HttpStatusCode.BadRequest); 
    } 
} 

Answer 4

@Franのアドバイスに従います。

[MissingParam(ParamName="id")] 
public ActionResult Edit(int id) 

私はメソッドのパラメータを台無しにする必要はありませんその方法は、いずれかの検証が前に起こる：私はこれをしなかったアクションでMissingParam

public class MissingParamAttribute : ActionFilterAttribute 
{ 
    public string ParamName { get; set; } 

    public override void OnActionExecuting(ActionExecutingContext filterContext) 
    { 
     if (filterContext.ActionParameters.ContainsKey(ParamName)) 
     { 
     if (filterContext.ActionParameters[ParamName] == null) 
     { 
      filterContext.ActionParameters[ParamName] = 0; 
     } 
     } 

    base.OnActionExecuting(filterContext); 
    } 
} 

と呼ばれるアクションフィルタ属性を構築しました。この実装は、オープン/クローズの原則に従います。私はその機能を拡張しましたが、質問のコードのように変更されませんでした。