2

我正在阅读MVC 音乐商店教程，遇到了一个小问题/查询。

我想通过它的 id 找到一张专辑。

var album = db.Albums.Find(id)

但是，如果 id 不存在，这可以正常工作，那么当控制器将空模型传递给视图时，我会收到 NullReferenceException。

我可以想到两种方法来解决这个问题。

方法1：在控制器中检查null，如果为null，则显示不同的视图/重定向到不同的操作，例如索引页面或专用错误页面/404未找到页面。

    //
    // GET: /Store/Details/4
    public ActionResult Details(int? id)
    {
        var album = db.Albums.Find(id);

        if (album == null) return RedirectToAction("Index");
        else return View(album);
    }

方法 2：检查视图中的模型是否为空，如果是，则不显示模型特定项，而是显示错误消息。

@model MvcMusicStore.Models.Album
@{
    ViewBag.Title = "Details";
}
@if (Model == null)
{
    <h2>That album doesn't exist</h2>
} 
else 
{
    <h2>Details: @Model.Title</h2>
}

问题：有没有其他方法/最佳实践方法来处理这个问题？方法 1 与方法 2 相比有什么优势吗？

4

2 回答 2

4

您可以编写一个自定义操作过滤器，该过滤器将检查传递给视图的模型是否为空并呈现 404 页面。这样您就不需要在所有控制器操作中重复此逻辑：

public class CheckForEmptyModelAttribute: ActionFilterAttribute
{
    public override void OnResultExecuted(ResultExecutedContext filterContext)
    {
        var viewResult = filterContext.Result as ViewResultBase;
        if (viewResult != null && viewResult.Model == null)
        {
            var view404 = new ViewResult
            {
                ViewName = "~/Views/Shared/404.cshtml"
            };
            filterContext.Result = view404;
        }
    }
}

接着：

//
// GET: /Store/Details/4
[CheckForEmptyModel]
public ActionResult Details(int? id)
{
    var album = db.Albums.Find(id);
    return View(album);
}

替代方法包括通过从您将在其中检索模型的类派生来编写自定义路由Route，如果找不到，则简单地不匹配路由。

于 2013-06-23T16:36:05.003 回答
0 
//
// GET: /Store/Details/4
public ActionResult Details(int? id)
{
    var album = db.Albums.Find(id);
    return ViewIfNotNull(album);
}

// boxing
private ActionResult ViewIfNotNull(object model)
{
    if (album == null) return RedirectToAction("Index");
    else return View(album);
}

// OR generic
private ActionResult ViewIfNotNull<T>(T model)
{
    if (album == null) return RedirectToAction("Index");
    else return View(album);
}
于 2013-06-24T21:03:54.777 回答