让我的存储库中的 2 个方法在同一个控制器 ActionResult 中运行是否正确？

Question

我正在使用 ASP.NET 4.0 MVC 和 C# 创建一个商店，并且对它相当陌生。

我开始创建显示特定类别中的产品的视图页面。

在特定类别页面上，我想要产品列表，并且还想要从数据库中获取类别名称及其相关描述。

目前，我这样做的方法是在我的存储库中使用两种方法：

1. 使用字符串检索特定类别的产品列表
2. 使用字符串检索特定类别

然后我在一个 ActionResult 中使用这两种方法，然后将它们传递到视图。

有没有一种方法可以通过对数据库的 1 个方法调用来检索产品列表和类别名称、描述等，或者我做得是否正确？

感谢您提前提供的任何帮助。

我的代码如下：

StoreRepository

public class StoreRepository : topsports.Models.IStoreRepository
{
    private topsportsEntities db = new topsportsEntities();

    public IQueryable<Product> FindProductsByCategory(string c)
    {
        var products = from p in db.Products
                       where p.Category.Name == c
                       select p;

        return products;
    }

    public Category FindCategory(string c)
    {
        return db.Categories.SingleOrDefault(cg => cg.Name == c);

    }
}

IStoreRepository

public interface IStoreRepository
{
    IQueryable<Product> FindProductsByCategory(string c);
    Category FindCategory(string c);

}

StoreController

 public class StoreController : Controller
{
    IStoreRepository storeRepository;

    public StoreController()
        : this(new StoreRepository())
    {
    }
    public StoreController(IStoreRepository repository)
    {
        storeRepository = repository;
    }

    public ActionResult Index(string c)
    {
        var category = storeRepository.FindCategory(c);

        var products = storeRepository.FindProductsByCategory(c).ToList();

        var viewModel = new StoreViewModel
        {
            Products = products,
            Category = category

        };

        return View(viewModel);
    }
}

StoreViewModel

public class StoreViewModel
{
    public List<Product> Products { get; set; }
    public Category Category { get; set; }

}

Category.aspx

<h2><%: Model.Category.Name %></h2>

<p><%: Model.Category.Description %></p>


<ul>
    <% foreach (var item in Model.Products) { %>
        <li>
            <%: item.Name %>, 
            <strong><%: item.Description %></strong>
        </li>
    <%} %>
</ul>

Answer 1

存储库的目的是将数据访问层与业务逻辑分离。当您选择通过类别实体检索产品时，您依赖于延迟加载，这是实体框架的实现细节。例如，当您稍后决定切换到不同的数据访问层（例如手动创建的查询）时，您可能不再拥有此功能。

第二个问题是，当您将大量功能放入单个存储库方法中时，该方法的职责就变得不清楚。正如 @Andrew Barber 所描述的，是的，你会得到很多小方法。然后可以将它们组合起来以产生有用的功能。当您选择创建返回更多结果的更大方法时，您会遇到另一个问题。当方法返回三个或四个数据集时，您会遇到这样的问题：当您只需要这些数据集中的一个或两个数据集时，您要么要创建一个新方法，其作用比原始方法少，要么您将运行四个查询，其中一两个就足够了。

存储库的小方法在组合成一个更大的整体时旨在产生有意义的结果。方法很多不一定有问题。你的代码看起来不错:)。

Answer 2

您似乎不需要单独获取 Category 对象，因为您可以从其中一项引用它。但是，我认为像您一样这样做可能会很好，因为例如，您的方法将涵盖类别目前没有任何项目的情况，无论出于何种原因。