如果在模型的LINQ查询中找不到任何结果，则返回错误消息

Question

我构建了一个编辑页面来更新数据，如果传递了正确的ID但传入的ID无效时，它会正常工作我得到一个空引用异常。我知道这是由于LINQ查询没有从数据库中找到任何有效数据的事实，但我不知道如何处理这个，除非我在我的视图中添加一堆IF语句，以便每次引用模型。这是我目前拥有的控制器代码。

public ActionResult EditSection(Int16 id = -1) 
    { 
     Section section = db.Sections.Find(id); 
     SectionAddEditVM model = new SectionAddEditVM { Section = section }; 

     if (section != null) 
     { 
      if (section.Type == "Collection") 
      { 
       RedirectToAction("Collection", new { id = id }); 
      } 

      model.SelectedType = section.Type; 
      return View(model); 
     } 

     ModelState.AddModelError("Section ID", "Invalid Section ID"); 
     return View(model); 
    } 

查看：

@model SectionAddEditVM 

@{ 
    ViewBag.Title = "Edit " + Model.Section.Title + " Information"; 
} 

<h2> 
    Edit @Model.Section.Title Information 
</h2> 

@using (Html.BeginForm(null, null, FormMethod.Post, new { enctype = "multipart/form-data" })) 
{ 
    @Html.AntiForgeryToken(); 
    @Html.ValidationSummary(false) 

    <p> 
     @Html.HiddenFor(m => m.Section.ID) 
     <label for="Title">Seciton Title:</label> @Html.EditorFor(m => m.Section.Title) 
     <br /> 
     <label for="RouteName">Section Route:</label> @Html.EditorFor(m => m.Section.RouteName) 
     <br /> 
     <label for="Type">Section Type:</label> @Html.DropDownListFor(m => m.Section.Type, new SelectList(Model.Type, "Value", "Text")) 
     <br /> 
     @Html.HiddenFor(m => m.Section.LogoFileID) 
     <label for="LogoFile">Logo Image:</label> <input id="LogoFile" name="LogoFile" type="file" /> 
     <br /> 
     <label for="Synopsis">Synopsis:</label> @Html.EditorFor(m => m.Section.Synopsis) 
     <br /> 
     <input type="submit" value="Edit Information" /> 
    </p> 
} 

Answer 1

解决方案是添加ELSE子句并初始化一个新的空白模型。

public ActionResult EditSection(Int16 id = -1) 
    { 
     Section section = db.Sections.Find(id); 

     if (section != null) 
     { 
      if (section.Type == "Collection") 
      { 
       RedirectToAction("Collection", new { id = id }); 
      } 

      SectionAddEditVM model = new SectionAddEditVM { Section = section }; 
      model.SelectedType = section.Type; 
      return View(model); 
     } 
     else 
     { 
      section = new Section(); 
      SectionAddEditVM model = new SectionAddEditVM { Section = section }; 

      ModelState.AddModelError("Section ID", "Invalid Section ID"); 
      return View(model); 
     } 
    } 

Answer 2

在你的控制器你已经检查是否section是null。所以，如果它是IS null只是返回一个不同的观点，说：“找不到部分”或什么。另外（如这里由@Aron建议），你可以回到这个观点404状态代码，所以控制器将是这个样子：

// find section based on ID 
// ... actual code ... 
if (section != null) 
{ 
    // process data and assign to model 
    return View(model); 
} 
else 
{ 
    Response.StatusCode = (int) System.Net.HttpStatusCode.NotFound; 
    return View("NoSectionFound"); 
} 

注意，通过返回不同的看法，不必返回不同的页面。 URL仍然是相同的，因为它基于控制器而不是视图。通过渲染不同的视图，您不必在通常显示数据的视图中使代码复杂化。

顺便说一句，避免给“太多”的信息，如“无效的部分ID”，它可以引导潜在的攻击者到哪里“戳”下一步。

我也会重新排列代码，所以只有在找到section的时候才给model赋值，而对于“no section found”视图，你显然不需要传递任何模型。