帮助设计订单管理器类

Question

所以我有一个订单管理器类，如下所示：

public class OrderManager
{
     private IDBFactory _dbFactory;
     private Order _order;

     public OrderManager(IDBFactory dbFactory)
     {
         _dbFactory = dbFactory;
     }


     public void Calculate()
     {

          _order.SubTotal
          _order.ShippingTotal
          _order.TaxTotal

          _order.GrandTotal


     }
}

现在，这里的重点是要有一个灵活/可测试的设计。 我非常关心能否围绕这个Calculate 方法编写可靠的单元测试。

注意事项：

     1.  Shipping has to be abstracted out, be loose coupled since the implementation of shipping could vary depending on USPS, UPS, fedex etc.  (they have their own API's).
     2. same goes with calculating tax

我是否应该创建一个税收和运输管理器类，并在构造函数中拥有一个税收/运输工厂？ （我到底是如何设计我的 OrderManager 的）类？

（就我“缺少的”而言，我唯一能想到的是 IoC，但我不介意这一点，并且在我看来不需要额外的抽象级别）。

Answer 1

好吧，您已经在方法中转向依赖注入，那么为什么不全力以赴并使用某种 IoC 容器来为您处理这个问题呢？

是的，如果您希望将其抽象化，请为其创建一个单独的类。如果您想真正对剩下的内容进行单元测试，请抽象出接口并使用模拟测试。问题是，你这样抽象得越多，需要做的工作就越多，你就会发现自己越希望使用某种类型的 IoC 框架。

您建议构造函数注入，这是一种常见的方法。您还会遇到属性注入（无参数构造函数，而是设置属性）。还有一些框架要求您实现某种初始化接口，以允许 IoC 框架在方法调用中为您进行初始化。使用您觉得最舒服的任何东西。

Answer 2

我确实认为 IOC 将有助于实例化正确的具体类，但您仍然需要按照您想要的方式进行设计。我确实认为您需要使用一个接口来抽象运输，您可以使用每个托运人（USPS、UPS、FEDEx 等）的类来实现该接口，并且可以使用工厂类（ShippingManager）来传递正确的一个或取决于国际奥委会为你做这件事。

public interface IShipper
{
//whatever goes into calculating shipping.....
 decimal CalculateShippingCost(GeoData geo, decimal packageWeight);
}

您还可以将 IShipper 和 ITaxer 具体类注入到 OrderManager 中，然后计算方法只需调用这些类......并且可以很好地使用 IOC 来处理该问题。

Answer 3

只是一个想法：

你的Calculate()方法不带参数，不返回任何内容并且作用于私有字段，这不是我会做的。我会将其编写为静态方法，该方法接受一些数字、IShippingProvider 和 ITaxJurisdiction，并返回美元总额。这样，您就有机会使用记忆功能缓存对 UPS 的昂贵呼叫和税表。

可能是我对这样的公共方法存在偏见。他们过去试图绑定到控件、使用代码生成器等让我很恼火。

编辑：至于依赖注入/IOC，我不认为有必要。这就是接口的用途。您不会加载一整套古怪的类，而只是加载相同权重/邮政编码组合的一些实现。

如果我是你的老板我也会这么说。

Answer 4

我会将计算方法放入一个类中。根据您的情况，OrderCalculator 可能需要了解增值税、货币、折扣……

只是一个想法。