在ASP.NET MVC中检查Ajax请求和返回Json结果的好方法

Question

我想让我在控制器中的动作更加灵活.我的意思是共同的行动通常会返回:

...
return View("someView");

或者,例如,如果Ajax:

...
return Json(new {result="ok"});

我想要的是让我的行动更"多用".例如,我基于简单的非Ajax请求创建了我的UI层,然后我决定使它更加用户友好并添加了一些Ajax.这样我必须纠正一些动作以返回Json.

避免此类事情的最简单(也可能是最差)方法是在每个(或几乎每个)Action中编写以下代码:

if (Request.IsAjaxRequest) {
    return Json(new {result="ok"});
}
else { 
    return View("someView") 
}

但当然这种方法完全与DRY的原则相冲突.

所以我想找到实现"多用途"的良好做法.

一种方法是编写一些这样的辅助方法:

public ActionResult CheckForAjax(ActionResult result)
{
  return ActionResult(result, Json(new {result="ok"}));
}

public ActionResult CheckForAjax(ActionResult result, Json json)
{
  if (Request.IsAjaxRequest) {
     return json;
  }
  else {
     return result;
  }
}

这样我可以在Actions中调用助手:

return CheckForAjax(View(...));

要么

return CheckForAjax(View(...), Json(new {myCustomJson="hi"});

但我不知道这是好方法还是只是重新发明一些自行车:)也许最好使用动作过滤器？但我不知道如何将自定义Json传递给该过滤器......

谢谢你的任何建议

Answer 1

说实话,我认为你的原始解决方案很好,而你的第二个更多是违反DRY而不是第一个.您的第二个解决方案非常冗余,并提供了两种方法来完成一项工作,轻松处理.

这不仅是风格不佳,而且是可维护性问题.通过将两个函数用于一个目的,每次发生更改时都必须更新这两个函数.为什么你这样做也不是很清楚,这将使其他开发人员难以维护你的代码.

如果你问我,KISS(保持简单愚蠢)比DRY更重要(不要重复自己).如果你的代码很容易被理解,那么它就是很好的代码.