Принцип open/closed и контруктором

@randomhero · Регистрация: 31.12.2016

Студворк — интернет-сервис помощи студентам

Разбираясь с принципами SOLID возникли некоторые вопросы в понимании. А именно, нормально ли изменяеть конструктор класса, если появилась необходимость расширить функциональность класса? Часто возникает проблема с тем, что надо заинжектить в класс дополнительный класс, чтобы добавить функционал.

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

Есть интерфейс

Java
1
2
3
4
public interface ShopFactory {
    List<Discount> getDiscounts();
    List<Sale> getSales();
}

И его имплементация (интерфейс я менять не могу, т.к этот интерфейс может реализовывать кто угодно, подключивший мою библиотеку в свой проект).

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
public class CountableDefaultShopFactory implements ShopFactory {
 
    Counter discountsCounter;
    Counter salesCounter;
 
    public DefaultShopFactory(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 класс, который возвращает эти данные.

Таким образом, мой класс принимает следующий вид

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
public class CountableDefaultShopFactory implements ShopFactory {
 
    Counter discountsCounter;
    Counter salesCounter;
    Counter couponsCounter;
    CouponDAO couponDAO;
 
    public CountableDefaultShopFactory(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 слое и тут собственно возникает вопрос в том, как лучше это сделать? И как бы вы это сделали?

Возможно есть готовые паттерны для таких случаев, я к сожалению не нашел похожего.

@korvin_ · 31.12.2016, 19:28

Сообщение от randomhero

По хорошему я считаю, что класс CountableDefaultShopFactory не должен знать ничего о DAO слое и тут собственно возникает вопрос в том, как лучше это сделать? И как бы вы это сделали?

Если CouponDAO — это интерфейс, то не вижу проблем. Ну, можно использовать Supplier<List<Coupon>>, например

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
public class CountableDefaultShopFactory implements ShopFactory {
 
    ...
    private Supplier<List<Coupon>> couponListSupplier;
 
    public CountableDefaultShopFactory(..., Supplier<List<Coupon>> couponListSupplier) {
        ...
        this. couponListSupplier = couponListSupplier;
    }
 
    ...
 
    @Override
    List<Coupon> getCoupons() {
        ...
        return couponListSupplier.get();
    }
}

@randomhero · 01.01.2017, 13:35 **[ТС]**

@korvin_, в таком случае все равно приходится изменять конструктор (плюс все места в проекте где этот конструктор использовался) и следовательно это нарушает принцип open/closed, не так ли?

@korvin_ · 01.01.2017, 15:57

Сообщение от randomhero

все равно приходится изменять конструктор

И что? Конструкторы не полиморфны и относятся к конкретным классам.

Сообщение от randomhero

плюс все места в проекте где этот конструктор использовался

По-хорошему тебе и так нужно всё это рефакторить, раз возникают такие проблемы. Например

Java
1
2
3
@FunctionalInterface
public interface ListSupplier<T> extends Supplier<List<T>> {
}

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
public interface ShopFactoryContext {
 
    Counter getDiscountsCounter();
 
    Counter getSalesCounter();
 
    Counter getCouponsCounter();
 
    ListSupplier<Discount> getDiscountsSupplier();
 
    ListSupplier<Sale> getSalesSupplier();
 
    ListSupplier<Coupon> getCouponsSupplier();
}

было

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
public class CountableDefaultShopFactory implements ShopFactory {
 
    private Counter discountsCounter;
    private Counter salesCounter;
    private ListSupplier<Discount> discountsSupplier;
    private ListSupplier<Sale> salesSupplier;
 
    public CountableDefaultShopFactory(ShopFactoryContext context) {
        this.discountsCounter = context.getDiscountsCounter();
        this.discountsSupplier = context.getDiscountsSupplier();
        this.salesCounter = context.getSalesCounter();
        this.salesSupplier = context.getSalesSupplier();
    }
 
    @Override
    public List<Discount> getDiscounts() {
        discountsCounter.count();
        return discountsSupplier.get();
    }
 
    @Override
    public List<Sale> getSales() {
        salesCounter.count();
        return salesSupplier.get();
    }
}

стало

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
public class CountableDefaultShopFactory implements ShopFactory {
 
    private Counter discountsCounter;
    private Counter salesCounter;
    private Counter couponsCounter;
    private ListSupplier<Discount> discountsSupplier;
    private ListSupplier<Sale> salesSupplier;
    private ListSupplier<Coupon> couponsSupplier;
 
    public CountableDefaultShopFactory(ShopFactoryContext context) {
        this.discountsCounter = context.getDiscountsCounter();
        this.discountsSupplier = context.getDiscountsSupplier();
        this.salesCounter = context.getSalesCounter();
        this.salesSupplier = context.getSalesSupplier();
        this.couponsCounter = context.getCouponsCounter();
        this.couponsSupplier = context.getCouponsSupplier();
    }
 
    @Override
    public List<Discount> getDiscounts() {
        discountsCounter.count();
        return discountsSupplier.get();
    }
 
    @Override
    public List<Sale> getSales() {
        salesCounter.count();
        return salesSupplier.get();
    }
 
    @Override
    public List<Coupon> getCoupons() {
        couponsCounter.count();
        return couponsSupplier.get();
    }
}

А собственно ShopFactoryContext должен создаваться в одном месте.

Вообще почитай про dependency injection, в частности, например, spring предоставляет механизмы.

Новые блоги и статьи Все статьи Все блоги /
http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Первый деплой lagorue 16.01.2026 Не спеша развернул своё 1ое приложение в kubernetes. А дальше мне интересно создать 1фронтэнд приложения и 2 бэкэнд приложения развернуть 2 деплоя в кубере получится 2 сервиса и что-бы они. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит: токи, напряжения и их 1 и 2 производные при t = 0;. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .

@randomhero 0 / 0 / 0 Регистрация: 31.12.2016 Сообщений: 2
	01.01.2017, 13:35 [ТС]
	@korvin_, в таком случае все равно приходится изменять конструктор (плюс все места в проекте где этот конструктор использовался) и следовательно это нарушает принцип open/closed, не так ли? 0