MVC 路由和局部视图

Question

好的，我有 2 两个问题：

1 - 我有一个返回成员图像列表的视图。如果您单击图像，它需要在同一视图中显示成员信息，但现在我收到以下错误：

当前对控制器类型“HomeController”的操作“ViewMembers”的请求在以下操作方法之间不明确： ServingTeam.Controllers.HomeController 类型上的 System.Web.Mvc.ActionResult ViewMembers(Int32) ServingTeam.Controllers.HomeController 类型上的 System.Web.Mvc.ActionResult ViewMembers()

控制器：

public ActionResult ViewMembers()
{
    TempData["returnMember"] = false;

    List<Members> members = memberRepo.GetAllMembers();

    return View(members);
}

[ActionName("ViewMembers")]
public ActionResult ViewMembers(int slug) 
{
    Members member = new Members();

    member = memberRepo.GetMember(slug);

    if (member != null)
        return View(member);

    return View("ViewMembers");
}

2 - 如何在显示成员列表的视图中显示单个对象？我创建了一个局部视图来显示成员的信息，但它不允许我这样做。

任何帮助将不胜感激

Answer 1

您有两个具有相同路由的控制器操作（方法）。您需要重命名其中之一。您仍然可以从两者返回相同的视图。我建议将第二个重命名为ViewMember。

对于问题的第二部分：

如果您坚持对两者使用相同的视图：显示成员列表和显示“选定”成员信息，则将 SelectedMember 属性添加到您的模型或将其添加到 ViewBag 并将其传递给您的部分看法。因此，您的操作可能如下所示：

public ActionResult ViewMember(int slug) 
{
    List<Members> members = memberRepo.GetAllMembers();
    Member member = memberRepo.GetMember(slug);

    ViewBag["SelectedMember"] = member;

    return View("ViewMembers", members);
}

确保在渲染局部视图之前检查ViewBag["SelectedMember"] 是否不是null，或者在局部视图中处理null 的情况。

Answer 2

我建议不要使用 TempData 或 ViewBag，因为两者都不是强类型、直观（对于您或其他查看您的代码的人）、难以维护等。首先创建您的 ViewModel ：

public class ViewMemberViewModel
{
  public ViewMemberViewModel()
  {
    // I like having my classes in valid states 
    // before the constructor finishes
    this.Members = new List<Member>();
  }      


  public List<Member> Members { get; set; }
}

接下来，使用Nullable<int> 创建单个操作。为什么可以为空？因为使用magic numbers (type #3) 通常不是描述没有价值的最佳方式。没有值，已经在框架中创建（null）。这与Ehsan Sajjad's anwswer 非常相似。

public ActionResult ViewMembers(int? slug) 
{
  var model = new ViewMemberViewModel();

  if (slug.HasValue)
  {
    var member = memberRepo.GetMember(slug);
    model.Members.Add(member);
  }
  else
  {
    var members = memberRepo.GetAllMembers();
    model.Members.AddRange(members);
  }

  return View(members);
}

风景不错：

@model ViewMemberViewModel

@foreach(var member in Model.Members)
{
  // The Controller already uses logic
  // to return 1 or more members.
}

@* don't forget if there aren't any *@
@if (Model.Members.Count == 0)
{
  // No members to display
}

Answer 3

你为什么要做两个不同的动作。一个动作就可以为您完成这项工作：

[ActionName("ViewMembers")]
public ActionResult ViewMembers(int slug=0) 
{
    if(slug > 0)
    {
    Members member = null;

    member = memberRepo.GetMember(slug);

    if (member != null)
        return View(member);

    return View("ViewMembers");
    }
    else
    {
     TempData["returnMember"] = false;

     List<Members> members = memberRepo.GetAllMembers();

     return View(members);
    }

}