在养成坏习惯之前需要建议

Question

我有一个名为 AuctionsController 的控制器。其中我有名为 Index() 和 AuctionCategoryListing() 的操作：

//Used for displaying all auctions.
public ActionResult Index()
{
    AuctionRepository auctionRepo = new AuctionRepository();
    var auctions = auctionRepo.FindAllAuctions();
    return View(auctions);
}

//Used for displaying auctions for a single category.
public ActionResult AuctionCategoryListing(string categoryName)
{
    AuctionRepository auctionRepo = new AuctionRepository();
    var auctions = auctionRepo.FindAllAuctions()
                       .Where(c => c.Subcategory.Category.Name == categoryName);
    return View("Index", auctions);
}

如您所知，它们都调用相同的视图（此操作称为“调用视图”。它的正确名称是什么？）。

@model IEnumerable<Cumavi.Models.Auction>

@{
    ViewBag.Title = "Index";
}

<h2>Index</h2>

<p>
    @Html.ActionLink("Create New", "Create")
</p>
<table>
    <tr>
        <th></th>
        <th>
            IDSubcategory
        </th>
        <th>
            IDCity
        </th>
        <th>
            IDPerson
        </th>
        <th>
            Title
        </th>
        <th>
            TextBody
        </th>
        <th>
            ContactNumber
        </th>
        <th>
            AskingPrice
        </th>
        <th>
            AddressDirection
        </th>
        <th>
            LatestUpdateDate
        </th>
        <th>
            VisitCount
        </th>
    </tr>

@foreach (var item in Model) {
    <tr>
        <td>
            @Html.ActionLink("Edit", "Edit", new { id=item.ID }) |
            @Html.ActionLink("Details", "Details", new { id=item.ID }) |
            @Html.ActionLink("Delete", "Delete", new { id=item.ID })
        </td>
        <td>
            @item.IDSubcategory
        </td>
        <td>
            @item.IDCity
        </td>
        <td>
            @item.IDPerson
        </td>
        <td>
            @item.Title
        </td>
        <td>
            @item.TextBody
        </td>
        <td>
            @item.ContactNumber
        </td>
        <td>
            @String.Format("{0:F}", item.AskingPrice)
        </td>
        <td>
            @item.AddressDirection
        </td>
        <td>
            @String.Format("{0:g}", item.LatestUpdateDate)
        </td>
        <td>
            @item.VisitCount
        </td>
    </tr>
}

</table>

它们都继承自同一个模型。

我的问题是，我是否以正确、适当的方式做事？或者这只是我设法拼凑起来的一个黑客。在我养成坏习惯之前帮助我。

Answer 1

你必须在某个地方进行分支，所以这可能更像是一个偏好问题。

我处理它的方法是使用一个方法，并让它以类别名称作为参数。由于字符串可为空，因此如果未指定字符串将为空。我的一个操作方法可能看起来像这样：

public ActionResult Index(string categoryName)
{
    AuctionRepository auctionRepo = new AuctionRepository();
    var auctions = auctionRepo.FindAllAuctions();

    if(String.IsNullOrEmpty(categoryName) == false)
      auctions = auctions.Where(c => c.Subcategory.Category.Name == categoryName);

    return View(auctions);
}

Answer 2

出于性能考虑，我更希望存储库执行过滤和分页等操作，并且 DRY 概念

public ActionResult Index(string categoryName)
    {
        AuctionRepository auctionRepo = new AuctionRepository();

    //Let the Repo Handle things like filtering and pagination, avoiding performance issues

        var auctions = auctionRepo.FindAllAuctions(categoryName); 

        return View(auctions);
    }

DAL 应该负责此任务。

Answer 3

MVC 的一个特点是视图和控制器是独立的。您同样可以对同一事物使用共享视图或部分视图，如果有的话，我会说这是一件好事，因为您正在编写可重用的代码并利用它。

Answer 4

我会将其修改为：

public ActionResult Index(string categoryName)
{

    AuctionRepository auctionRepo = new AuctionRepository();
    var auctions=auctionRepo.FindAllAuctions();

    if (!string.IsNullOrEmpty(categoryName))
    {
        auctions = auctions.Where(c => c.Subcategory.Category.Name == categoryName);
    }

    return View(auctions);
}

您的路线可能如下所示：

    context.MapRoute(
        "auction_defalt",
        "Auction/{categoryName}",
        new { controller="Auction", action = "Index", categoryName = UrlParameter.Optional }

由于操作非常相似，因此我认为没有理由将它们分开。

Answer 5

与任何框架一样，ASP.NET MVC 给您很多搬起石头砸自己脚的机会。如果没有深思熟虑，控制器操作、视图模型和视图的重用很快就会成为维护的噩梦。更不用说如果没有类似的考虑，您的路线将变得很难连接在一起。

遵循约定优于配置的原则，您可以通过使用单独的操作但重用视图部分来解决问题。对我来说，AuctionsController 的索引操作应该负责列出系统中的所有拍卖。我不会将我的类别操作称为 AuctionCategoryListing，而是将其简单地称为 Category。按照惯例，这可以将路由布局为：

• site.com/auctions/ ，用于索引
• site.com/auctions/category/CATEGORYNAME ，用于类别。

该路线很容易被用户理解，并且您也很容易明确了解每个路线的作用。 （至此，奥马尔在他的回答中提供了一个很好的建议 让您的存储库处理分页、过滤等。）

至于每个操作应返回的内容，您有多种选择。我的偏好是返回单独的视图，每个视图都包含对公共部分的引用。这使您可以灵活地围绕部分创建不同的视图，但可以重复使用常见的部分。

进一步阅读可能会有所帮助：

• 每个视图一个 ViewModel
• ASP.NET MVC 术语让我困惑 - 为什么是“ViewModel”？
• 什么时候应该在 asp.net mvc 中使用部分视图？