Разбираясь с принципами SOLID возникли некоторые вопросы в понимании. А именно, нормально ли изменяеть конструктор класса, если появилась необходимость расширить функциональность класса? Часто возникает проблема с тем, что надо заинжектить в класс дополнительный класс, чтобы добавить функционал.
Мне кажется, что в таком случае нарушается принцип открытости/закрытости. Вопрос в том, как можно избежать нарушения этого принципа на следующем примере.
Есть интерфейс
public interface ShopFactory {
List<Discount> getDiscounts();
List<Sale> getSales();
}
И его имплементация (интерфейс я менять не могу, т.к этот интерфейс может реализовывать кто угодно, подключивший мою библиотеку в свой проект).
public class CountableDefaultShopFactory implements ShopFactory {
Counter discountsCounter;
Counter salesCounter;
public CountableDefaultShopFactory(Counter discountsCounter, Counter salesCounter) {
this.discountsCounter = discountsCounter;
this.salesCounter = salesCounter;
}
@Override
List<Discount> getDiscounts() {
discountsCounter.count();
return Discount.defaultDiscounts();
}
@Override
List<Sale> getSales() {
salesCounter.count();
return Sale.defaultSales();
}
}
Выглядит довольно просто. CountableDefaultShopFactory
реализует ShopFactory
, переопределяет два метода и принимает в конструкторе два объекта типа Counter
, которые будут использоваться для подсчета кол-ва раз вызванного метода. В результате каждый метод возвращает результат вызовом статического метода.
Теперь предположим, что необходимо добавить функционала в этот класс и он будет возвращать еще список объектов типа Coupon
. Только брать он их уже будет не из статического метода у класса Coupon
, а из-за базы данных например. Предположим, что у меня есть DAO класс, который возвращает эти данные.
Таким образом, мой класс принимает следующий вид
public class CountableDefaultShopFactory implements ShopFactory {
Counter discountsCounter;
Counter salesCounter;
Counter couponsCounter;
CouponDAO couponDAO;
public DefaultShopFactory(Counter discountsCounter, Counter salesCounter, Counter couponsCounter, CouponDAO couponDAO) {
this.discountsCounter = discountsCounter;
this.salesCounter = salesCounter;
this.couponsCounter = couponsCounter;
this.couponDAO = couponDAO;
}
@Override
List<Discount> getDiscounts() {
discountsCounter.count();
return Discount.defaultDiscounts();
}
@Override
List<Sale> getSales() {
salesCounter.count();
return Sale.defaultSales();
}
@Override
List<Coupon> getCoupons() {
couponsCounter.count();
return couponDAO.getDefaultCoupons();
}
}
Как видно, то пришлось модифицировать конструктор, а именно добавить еще параметры couponsCounter
(что я считаю нормально) и couponDAO
.
По хорошему я считаю, что класс CountableDefaultShopFactory
не должен знать ничего о DAO
слое и тут собственно возникает вопрос в том, как лучше это сделать? И как бы вы это сделали?
Возможно есть готовые паттерны для таких случаев, я к сожалению не нашел похожего.
Заранее спасибо.
CountableDefaultShopFactory
либо нет конструктора, либо вы имя перепутали сDefaultShopFactory