Выбираем меньшее из двух зол (DI vs Copy/Paste)

@Pro100Tom · Регистрация: 29.10.2012

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

Всем привет!

У меня возникла очень трудная диллема. Есть калькулятор скидки. Скажем, звучит так: возьми товар за £20.

Калькулятор скидки считает эту скидку. То есть, если цена продукат £500, то скидка будет £480. Дело тут вот в чём. В коде (написаном не мною, и который менять не стоит) есть логика вытаскивания цены продукта. То есть, чтобы посчитать скидку, нужно получить цену продукта. Для этого используется класс, который в зависимости от флага в базе данных возвращает либо цену без налога, либо цену с налогом. Налог 20%, то есть в зависимости от значения в базе данных, калькулятор скидки будет выплёвывать разное значение: если налог не учитывается, то скидка, как уже сказал будет £480. А если учитывается, то скидка будет £580, так как сервис получения цены вернет 600 (20% сверху от £500).

Мне нужно изменить код таким образом, чтобы этот флаг шел не из базы данных, а из другого класса. Есть два варианта:
1. Этот сервис, который используется для получения цены подключается в конструктор калькурятора скидки и можно туда вставить другой сервис, который будет не из базы вытаскивать значение, а из другого класса, как мне нужно.
2. Я могу создать новый класс калькулятора скидки и просто вставить скопированный код и в том месте, где сервис возвращает цену товара, поменять логику так как мне нужно.

Проблема первого варианта заключается в том, что мое решение зависит от того, что тот сервис обязательно должен использоваться для получения цены. Еcли кто-то что-то поменяет там, то мои изменения перестанут работать. Плюс, я таким образом меняю функционал оригинального калькулятора скидки. Не знаю, норм ли менять его. Дополнять - пожалуйста, а вот менять - не уверен. И еще одна проблема заключается в том, что тот сервис имеет много публичных методов; и если я изменю тот один, так как мне нужно, то могут возникнуть логические ошибки по отношению к другим публичным методам; там, как у Окси всё переплетено.

Проблема второго варианта заключается в copy/paste. Если найдётся какой-то баг, то придется его устранять в обоих файлах.

Что делать?

@IamRain · 01.07.2022, 12:54

Вы слишком много факторов вносите в ваше изложение - сложно анализировать.

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

Дополнять - пожалуйста, а вот менять - не уверен.

Используйте разнеовидность первого варианта, но просто расширяйте интерфейс условного IDiscountCalculator методом, аля:

C#
1
Task<decimal> GetFixedPriceDiscountValue(decimal fixedPrice, IPriceProvider priceProvider);

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

@Pro100Tom · 01.07.2022, 13:07 **[ТС]**

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

Необязательно инжектить IPriceProvider в конструктор

Не могу поменять, это core framework behaviour. Надо плясать вокруг такой имплементации. Во фреймворке зависимости инджектятся в конструктор.

@IamRain · 01.07.2022, 13:34

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

Во фреймворке зависимости инджектятся в конструктор.

Pro100Tom, не важно как они инжектятся, если они в принципе инжектятся, то и получить экземпляр тоже можно (через ваш кастомный ServiceLocator). Просто обозначение явно как зависимости в виде параметра для конкретной операции - нормальная практика.
Не можете - не делайте. Сделайте так:

C#
1
Task<decimal> GetFixedPriceDiscountValue(decimal fixedPrice); // let it be

Добавлено через 15 минут
Боитесь поломать существующий IPriceDiscounter? - используйте Decorator.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
class CustomPriceDiscounter: IPriceDiscounter
{
   
   CustomPriceDiscounter(IPriceProvider priceProvider, IPriceDiscounter original)
   {
          // reuse everywhere except of  GetFixedPriceDiscountValue
          this.original = original;
          this.priceProvider = priceProvider;
   }
   ...
  
   Task<decimal> GetFixedPriceDiscountValue(decimal fixedPrice)
   {
      //your logic here, but original would throw NotSupportException, for instance
   }      
}

Такие штуки, как по мне, просто в потоке сознания делаются, без лишних раздумываний. Несложно ведь.

@Pro100Tom · 01.07.2022, 15:44 **[ТС]**

IamRain,

Если использовать декоратор или просто если наследовать калькулятор скидки, то мне не нужен будет тот сервис, что возвращает мне цену; я бы использовал там другую имплементацию; очень похожую, почти ту же самую, но всё-таки другую.

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

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .	[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .

@Pro100Tom 154 / 31 / 11 Регистрация: 29.10.2012 Сообщений: 397

	Выбираем меньшее из двух зол (DI vs Copy/Paste) 01.07.2022, 12:16. Показов 591. Ответов 4 Метки нет (Все метки) Всем привет! У меня возникла очень трудная диллема. Есть калькулятор скидки. Скажем, звучит так: возьми товар за £20. Калькулятор скидки считает эту скидку. То есть, если цена продукат £500, то скидка будет £480. Дело тут вот в чём. В коде (написаном не мною, и который менять не стоит) есть логика вытаскивания цены продукта. То есть, чтобы посчитать скидку, нужно получить цену продукта. Для этого используется класс, который в зависимости от флага в базе данных возвращает либо цену без налога, либо цену с налогом. Налог 20%, то есть в зависимости от значения в базе данных, калькулятор скидки будет выплёвывать разное значение: если налог не учитывается, то скидка, как уже сказал будет £480. А если учитывается, то скидка будет £580, так как сервис получения цены вернет 600 (20% сверху от £500). Мне нужно изменить код таким образом, чтобы этот флаг шел не из базы данных, а из другого класса. Есть два варианта: 1. Этот сервис, который используется для получения цены подключается в конструктор калькурятора скидки и можно туда вставить другой сервис, который будет не из базы вытаскивать значение, а из другого класса, как мне нужно. 2. Я могу создать новый класс калькулятора скидки и просто вставить скопированный код и в том месте, где сервис возвращает цену товара, поменять логику так как мне нужно. Проблема первого варианта заключается в том, что мое решение зависит от того, что тот сервис обязательно должен использоваться для получения цены. Еcли кто-то что-то поменяет там, то мои изменения перестанут работать. Плюс, я таким образом меняю функционал оригинального калькулятора скидки. Не знаю, норм ли менять его. Дополнять - пожалуйста, а вот менять - не уверен. И еще одна проблема заключается в том, что тот сервис имеет много публичных методов; и если я изменю тот один, так как мне нужно, то могут возникнуть логические ошибки по отношению к другим публичным методам; там, как у Окси всё переплетено. Проблема второго варианта заключается в copy/paste. Если найдётся какой-то баг, то придется его устранять в обоих файлах. Что делать? 0

@Pro100Tom 154 / 31 / 11 Регистрация: 29.10.2012 Сообщений: 397
	01.07.2022, 15:44 [ТС]
	IamRain, Если использовать декоратор или просто если наследовать калькулятор скидки, то мне не нужен будет тот сервис, что возвращает мне цену; я бы использовал там другую имплементацию; очень похожую, почти ту же самую, но всё-таки другую. Это нормально, если моя имплементация будет очень похожа на оригинальную и только отличаться логикой получения цены (это всего пару строк). Но проблема тогда заключатся в том, что возникает дубликация кода, и если будет найдет баг, то надо исправлять в двух файлах. 0