c# - 存储库中添加和更新方法的特定实体对象

Question

我正在尝试想出一种方法来设计一个存储库，其中添加和更新只接受它可以添加/更新的数据/属性的确切数量。

我有以下设计:

public interface IProduct
{
    int Id { get; set; }
    string Name { get; set; }
    decimal Price { get; set; }
    DateTime Created { get; set; }
    DateTime Updated { get; set; }
}

public interface IProductRepository
{
    void Add(IProduct product);
    void Update(IProduct product);
    IProduct Get(int id);
    IEnumerable<IProduct> GetAll();
}

但是，Created 和 Updated 属性并不是我想要在数据库外部修改的内容。 Id 在添加时都不相关，所以我尝试了以下操作:

public interface IProductAdd
{
    string Name { get; set; }
    decimal Price { get; set; }
}

public interface IProductUpdate
{
    int Id { get; set; }
    string Name { get; set; }
    decimal Price { get; set; }
}

并相应地更新了存储库:

public interface IProductRepository
{
    void Add(IProductAdd product);
    void Update(IProductUpdate product);
    IProduct Get(int id);
    IEnumerable<IProduct> GetAll();
}

现在每个单独的方法中只存在相关属性。

然后我可以创建一个实现所有产品接口(interface)的类:

public class Product : IProduct, IProductAdd, IProductUpdate
{
    public int Id { get; set; }
    public string Name { get; set; }
    public decimal Price { get; set; }
    public DateTime Created { get; set; }
    public DateTime Updated { get; set; }
}

所以我的问题是:这样做是正确的方法吗？

我的想法:

我本可以选择更改 Repository 上的 Add 和 Update 方法以接受每一位产品数据作为参数，例如 Update(int id, string name, decimal price)，但它当产品包含的信息量增加时，将很快失控。

我目前的解决方案涉及重复。如果产品应包含 Description 属性，我将不得不在不同的 3 个界面中指定它。我可以让接口(interface)相互实现来解决这个问题......

public interface IProductAdd
{
    string Name { get; set; }
    decimal Price { get; set; }
    string Description { get; set; }
}

public interface IProductUpdate : IProductAdd
{
    int Id { get; set; }
}

public interface IProduct : IProductUpdate
{
    DateTime Created { get; set; }
    DateTime Updated { get; set; }
}

...但是如果 IProductAdd 有一些 IProductUpdate 不应该有的东西，我就会遇到麻烦。

相关问题:假设我想将产品放入类别中，并且可以直接访问每个产品上的类别。

public interface ICategory
{
    int Id { get; set; }
    string Name { get; set; }
    string Description { get; set; }
}

public interface IProduct
{
    int Id { get; set; }
    string Name { get; set; }
    decimal Price { get; set; }
    DateTime Created { get; set; }
    DateTime Updated { get; set; }
    ICategory Category { get; set; }
}

当我更改产品时，我想指定类别的 ID(因为我添加/更新的是关系，而不是类别本身):

public interface IProductAdd
{
    string Name { get; set; }
    decimal Price { get; set; }
    int CategoryId { get; set; }
}

public interface IProductUpdate
{
    int Id { get; set; }
    string Name { get; set; }
    decimal Price { get; set; }
    int CategoryId { get; set; }
}

这导致以下实现:

public class Product : IProduct, IProductAdd, IProductUpdate
{
    public int Id { get; set; }
    public string Name { get; set; }
    public decimal Price { get; set; }
    public DateTime Created { get; set; }
    public DateTime Updated { get; set; }
    public ICategory Category { get; set; }
    public int CategoryId { get; set; }
}

请看，我是使用 Category.Id 还是 CategoryId？不理想，IMO。

所以我想无论我怎么做，我都会看到问题。我太挑剔了吗？我看错了吗？

我是否应该将它们完全分开，因为它们是不同的东西(例如 [实体]/[添加实体参数]/[更新实体参数])？

Answer 1

我认为您在没有正确分离图层的情况下使事情过于复杂。在我看来，前两个类是应该如何完成的。

据我了解，您的整个问题是您不希望错误地修改 Created 和 Updated 属性。但是，您混淆了数据和业务问题。说产品的创建日期应该设置在正在创建的产品上是创建新产品的业务逻辑的一部分，并且说产品的更新日期应该在 x 和 y 出现时更新也是更新 a 的逻辑过程的一部分产品的数据。这与验证产品属性是否有效、用户是否获得授权等属于同一类型的流程，所有这些都是业务流程问题，而不是数据存储问题。

您的存储库应该只是数据层的一部分，它唯一关心的是如何从数据库中检索请求的产品、如何更新数据库中的产品或在数据库中创建产品。就是这样。

您需要一个专门的业务层来处理添加或更新产品信息的所有业务逻辑。然后，您将在该层中调用一个方法，其中包含您要添加的产品的名称和价格，在该方法中，您将执行您想要执行的任何验证，确定用户是否有权进行这些编辑，以及如果需要，设置 CreatedDate 或 UpdatedDate。然后此方法会将 Product 实体传递到您的存储库以将其保存在数据库中。

以这种方式分离逻辑将使您想要更改有关如何管理诸如 UpdatedDate 之类的事情的逻辑时变得容易得多(也许您想要某些操作来更改该日期，但不是全部行动)。如果您尝试在您的存储库/数据层中处理所有这些，当您摆脱琐碎的用例时，它很快就会变得不知所措和困惑。

还有一点。 IProduct 是一个业务实体，这意味着您根本不必将它暴露给您的表示层。因此，如果您不想让开发人员冒接触某些属性的风险，您可以使用 MVC 架构通常称为 ViewModels 的东西。本质上，这些是在表示层上使用的数据结构，然后业务层可以将这些 ViewModel 转换为实际的业务实体。

例如，您可以:

public class ProductViewModel
{
    int Id { get; set; }
    string Name { get; set; }
    decimal Price { get; set; }
    int CategoryId { get; set; }
}

您的表示层会将填写好的 ProductViewModel 传递到业务层的 AddProduct() 或 UpdateProduct() 方法中，然后检索指定数据库的 IProduct 实体，并使用 ProductViewModel 确定如何更新(或创建新的)数据库实体。这样，您永远不会公开这两个 DateTime 属性，但仍然可以完全控制它们的设置方式和时间。