什么属于存储库，什么不属于存储库？

Question

在实现存储库模式的 ASP.NET MVC 应用程序中，我很好奇如果非数据相关方法仍然属于给定存储库的一般焦点，是否适合将它们放置在存储库中。 例如，假设一个 ProductsRepository 具有添加和删除 ProductImages 的方法，这些 ProductImages 在数据库和本地文件存储中也有部分表示。 如果需要删除 ProductImage，我们需要使用存储库方法从数据库中删除一行，并且还需要从存储介质中删除与该图像关联的文件。 IO 操作是否属于存储库，或者是否有更合适的位置？

在类似我刚才描述的情况下，我一直在做的一件事是在我的存储库中提供静态方法，这些方法通过使用存储在数据库中的文件名和预定义的目录模式以编程方式生成它，为我提供给定 ProductImage 的路径。 这是否超出了存储库的预期用途？

---

编辑

如果这样的操作不属于存储库，那么在 MVC 模式中类似的操作应该放在哪里？ 在我看来，在控制器和存储库之间设置另一层可能是有意义的，该层根据需要调用存储库，并且可以从控制器静态调用。

Answer 1

存储库的作用是将应用程序与它所使用的数据的存储方式和位置隔离开来。 在此基础上，存储库绝对是在此上下文中处理数据库和基于文件的活动的正确位置。

Answer 2

我最近设计了一个新的存储库，并为同样的问题而苦苦挣扎。 我最终将我的附加方法包含在存储库中。

现在回想起来，我觉得更好的解决方案是让我的存储库更加集中，并将我的附加方法放入与我的存储库紧密集成的服务中。

对于上面的示例，您可以在 ProductsService 中使用“DeleteProductImage”方法，该方法将调用 ProductsRepository.DeleteImage & 然后还要处理从存储介质中删除图像。

这可以使您的存储库保持干净并仅专注于“DeleteImage”逻辑，同时仍然为您提供需要调用的单个方法（“DeleteProductImage”），该方法负责调用存储库来删除图像，同时还处理与存储介质的交互以及删除图像时可能需要发生的任何其他与您的存储库不直接相关的事情。

Answer 3

我认为对存储库模式更大的担忧是您违反了单一责任原则。 您的类应该承担一项职责，例如操作数据库中的数据。 您应该有一个不同的类来处理文件 IO，并且您可以将函数分组到一个类中的上一层。

类的更改应该只有一个原因，而处理文件 IO 和数据库调用的存储库类将有两个原因。 文件系统布局的更改或数据库的更改。

编辑

为了解决您的编辑问题，以下是我在 MVC 场景中实现这一点的方法（这也假设您正在使用某种依赖注入来使生活更轻松）。

// Controller class
public class ProductsController
{
    private IProductService _productService;

    public ProductsController(IProductService productService)
    {
        _productService = productService
    }

    public void RemoveImage(int productId, int imageId)
    {
        _productService.RemoveImage(productId, imageId)
    }
}

public class ProductService: IProductService
{
    private IProductRepository _productRepository;
    private IProductImageManager _imageManager;

    public ProductService(IProductRepository productRepository, IProductImageManager imageManager)
    {
      _productRepository = productRepository;
      _imageManager = imageManager;
    }

    public void RemoveImage(int productId, int imageId)
    {
        // assume some details about locating the image are in the data store
        var details = _productRepository.GetProductImageDetails(productId, imageId);
        // TODO: error handling, when not found?
        _imageManager.DeleteImage(details.location);
        _productRepository.DeleteImage(productId, imageId)
    }
}

然后，您可以根据任何有意义的接口来实现 IProductImageManager 和 IProductRepository，并根据您的特定需求进行具体实现。