4

在工作中，我投入到开发一个遗留的企业级应用程序中，该应用程序仍在生产中，并且由于糟糕的设计和不稳定而在过去几个月中停滞不前。

所以我们已经开始使用 EF5 并将一些设计模式/层应用到我们的应用程序中。

我很难理解的是：在我们的案例中，服务层到底应该做什么？它会是过度架构，还是会在不增加不必要的复杂性的情况下提供一些好处？

让我们向您展示我们目前所拥有的：

  • 我们引入了 EF（带有 POCO 的 Code First）来映射我们的遗留数据库（工作得相当好）
  • 我们已经为我们在新数据层中需要的大多数东西创建了存储库（具体实现，我没有看到使用通用存储库分离关注点的任何好处..）

现在在特定情况下，它是关于计算文章的价格 - 通过直接从 arcile 或从文章所在的组获取价格（如果没有指定价格）。它变得更加复杂，因为还涉及不同的价目表（取决于订单的完整价值）并且取决于也可以有特价等的客户。

所以我的主要问题是：谁负责获得正确的价格？

我的想法是：订单必须知道它包含的项目。另一方面，这些商品必须知道它们的价格是多少，但订单必须不知道如何计算商品的价格，只需要汇总它们的成本即可。

目前我的代码摘录：

ArticlePrice（POCO，即将由 Fluid API 交换的映射）

[Table("artikeldaten_preise")]
public class ArticlePrice : BaseEntity
{
    [Key]
    [Column("id")]
    public int Id { get; set; }

    [Column("einheit")]
    public int UnitId { get; set; }

    [ForeignKey("UnitId")]
    public virtual Unit Unit { get; set; }

    [Column("preisliste")]
    public int PricelistId { get; set; }

    [ForeignKey("PricelistId")]
    public virtual Pricelist Pricelist { get; set; }

    [Column("artikel")]
    public int ArticleId { get; set; }

    [ForeignKey("ArticleId")]
    public virtual Article Article { get; set; }

    public PriceInfo PriceInfo { get; set; }

}

文章价格库：

   public class ArticlePriceRepository : CarpetFiveRepository
{
    public ArticlePriceRepository(CarpetFiveContext context) : base(context) {}

    public IEnumerable<ArticlePrice> FindByCriteria(ArticlePriceCriteria criteria)
    {
        var prices = from price in DbContext.ArticlePrices
                     where
                         price.PricelistId == criteria.Pricelist.Id
                         && price.ArticleId == criteria.Article.Id
                         && price.UnitId == criteria.Unit.Id
                         && price.Deleted == false
                     select price;

        return prices.ToList();
    }
}

public class ArticlePriceCriteria
{
    public Pricelist Pricelist { get; set; }
    public Article Article { get; set; }
    public Unit Unit { get; set; }

    public ArticlePriceCriteria(Pricelist pricelist, Article article, Unit unit)
    {
        Pricelist = pricelist;
        Article = article;
        Unit = unit;
    }
}

PriceService（确实有一种可怕的代码气味......）

public class PriceService
{
    private PricelistRepository _pricelistRepository;
    private ArticlePriceRepository _articlePriceRepository;
    private PriceGroupRepository _priceGroupRepository;

    public PriceService(PricelistRepository pricelistRepository, ArticlePriceRepository articlePriceRepository, PriceGroupRepository priceGroupRepository)
    {
        _pricelistRepository = pricelistRepository;
        _articlePriceRepository = articlePriceRepository;
        _priceGroupRepository = priceGroupRepository;
    }

    public double GetByArticle(Article article, Unit unit, double amount = 1, double orderValue = 0, DateTime dateTime = new DateTime())
    {
        var pricelists = _pricelistRepository.FindByDate(dateTime, orderValue);

        var articlePrices = new List<ArticlePrice>();

        foreach (var list in pricelists)
            articlePrices.AddRange(_articlePriceRepository.FindByCriteria(new ArticlePriceCriteria(list, article, unit)));

        double price = 0;
        double priceDiff = 0;

        foreach (var articlePrice in articlePrices)
        {
            switch (articlePrice.PriceInfo.Type)
            {
                    case PriceTypes.Absolute:
                        price = articlePrice.PriceInfo.Price;
                        break;
                    case PriceTypes.Difference:
                        priceDiff = priceDiff + articlePrice.PriceInfo.Price;
                    break;
            }
        }

        return (price + priceDiff) * amount;
    }

    public double GetByPriceGroup(PriceGroup priceGroup, Unit unit)
    {
        throw new NotImplementedException("not implemented yet");
    }

    //etc. you'll get the point that this approach might be completely WRONG

}

我的最后一个问题是：如何正确建模我的问题？是否正确，我正在对代码进行过度架构？我的服务层将如何正确显示？我宁愿拥有 ArticlePriceService、ArticleGroupPriceService 等吗？但是谁会连接这些部分并计算出正确的价格呢？例如，这会是具有“GetPrice”方法的 OrderItemService 的责任吗？但话又说回来 orderItemService 必须知道其他服务..

请尝试为我提供有关架构的可能解决方案，以及哪个对象/层做什么。

如果您需要更多信息，请随时问我其他问题！

4

2 回答 2

0

您确实提出了一个简单的场景，存储库本身可能就足够了。
你有更多的存储库吗？
您是否希望您的应用程序增长，并使用更多的存储库？

我见过的大多数应用程序/示例都建议使用抽象数据层的服务层，并且开销并不大。

当您想从几个不同的存储库中获取数据，然后对数据执行某种聚合/操作时，可能会弹出使用服务的一个原因。
然后，服务层将提供操作逻辑，而服务消费者不必处理几个不同的存储库。
您还应该考虑可能希望在一个事务中更改多个实体的情况（含义 - 多个存储库），并且仅当所有更新操作成功时才​​将更改保存到数据库。
这种情况应该意味着使用工作单元模式，并且可能会结束使用服务层，以启用正确的单元测试。

于 2013-06-13T13:48:53.850 回答
0

当我开始研究对象和架构时，我的主要问题是给类起个好名字。

对我来说，您的服务似乎应该称为“ShopService”（或类似的名称）。那么你的方法GetByArticle，应该命名为GetPriceByArticle。

将服务名称更改为比价格更大的东西的想法会更有意义，并且还会解决其他问题（例如您想知道的 OrderPriceService）。

也许您可以问自己“与此服务交互的我的页面或窗口的名称是什么？” 只有一个或多个吗？如果更多，它们有什么共同点？这可以帮助您为您的服务找到一个好名字，从而为您的服务找到不同的方法来获取每个需要的内容。

告诉我更多。我会很乐意帮忙的。

于 2015-02-03T14:24:57.150 回答