两种具有相同行为的方法

Gui*_*nge 2 c# asp.net-mvc-4

目标

使用相同的行为创建两个"不同"的方法.

问题

当有人访问我的应用程序时,我想显示一个项目列表 - 这个列表与提供的列表相同myapp.com/products/offers/.换句话说,我不想在两种方法之间重复相同的代码.所以我问:我该怎么办?

我现在在做什么

at HomeController,在Index其类型为的方法上ActionResult,有以下代码的片段:

public ActionResult Index()
{
    return RedirectToAction("Offers", "Products");
}
Run Code Online (Sandbox Code Playgroud)

同时,在ProductsControllerOffers方法:

public ActionResult Offers()
{
    var products = Products.Build.OffersList();
    var categories = Categories.Build.Main();

    ProductsViewModel viewModel = ProductsViewModel
    {
        Products = products,
        Categories = categories
    };

    return View(viewModel);
}
Run Code Online (Sandbox Code Playgroud)

现在有三件事需要考虑:

  1. 我的应用程序是将客户端重定向到另一个页面,生成第二个服务器请求浪费带宽;
  2. 应用程序的URL从更改myapp.com/myapp.com/Products/Offers/,我真的不想要这个;
  3. 如果我重复代码,那将是多余的 - 更多的ProductsController是,逻辑上的东西不应该存在HomeController.

而且,再一次:我该怎么办?

Sim*_*ead 6

将公共逻辑移动到"服务"或"助手"类:

class ProductListingHelper {
    public ProductsViewModel GetProductsViewModel() {
       var products = Products.Build.OffersList();
        var categories = Categories.Build.Main();

        return new ProductsViewModel() {
            Products = products,
            Categories = categories
        };
    }
}
Run Code Online (Sandbox Code Playgroud)

然后,在两个控制器中,执行以下操作:

 return View(new ProductListingHelper().GetProductsViewModel());
Run Code Online (Sandbox Code Playgroud)

请注意,正如Erik在评论中指出的那样,这将需要您创建两个视图.但是,您也可以通过将ProductListView作为其他两个视图呈现的部分视图来减少重复.

  • 另外我提到这需要两个视图,除非你做额外的工作(很容易但没有提到). (2认同)