Помощь проектирование заказа менеджер класс

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 


    } 
} 

Теперь, дело здесь иметь гибкий/Testible дизайн. Я очень обеспокоен тем, что могу написать твердые модульные тесты вокруг этого метода 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 и т. Д.) И может использовать класс Factory (ShippingManager) для передачи правильного ответа или зависят от МОК, чтобы сделать это за вас.

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

Вы также можете просто вводить в IShipper и конкретные классы ITaxer в ваш OrderManager и рассчитать метод просто вызывает в этих классах .... и может использовать МОК прекрасно справиться с этим.

Answer 3

Только мысль:

Ваш метод Calculate() не принимает никаких параметров, не возвращая ничего и действуя на частных полях не так, как я бы это сделать. Я бы написал его как статический метод, который принимает некоторые числа, IShippingProvider и ITaxJurisdiction и возвращает общую сумму в долларах. Таким образом, у вас есть возможность кэшировать дорогие звонки в UPS и ваши налоговые таблицы, используя memoization.

Может быть, я предвзято отношусь к общедоступным методам, которые работают так. Они сожгли меня в прошлом, пытаясь связать с элементами управления, использовать генераторы кода и т. Д.

EDIT: что касается инъекции зависимостей/IOC, я не вижу необходимости. Для этого были созданы интерфейсы. Вы не собираетесь загружать целые массивные дурацкие классы, просто некоторые реализации одного и того же веса/кода zipcode.

Вот что я бы сказал, если бы я был вашим начальником.

Answer 4

Я бы взял метод Calculate в класс. В зависимости от ваших обстоятельств OrderCalculator может потребоваться знать НДС, Валюту, Скидки, ...

Просто подумайте.