А вот ещё вопрос, чтобы разбавить скучный вечер воскресенья (всем

офк нечего делать).
Есть зависимость на уровне метода
class Tariff {
function makeInvoice(invoiceId, promocode) {
promocode->use(invoiceId, this->id);
// посчитать сумму для инвойса с использованием тарифа и промокода и создать инвойс
}
}
Вроде бы это естественный coupling - при создании инвойса - нужно учесть скидку из промокода, но объект промокода для соблюдения инвариантов тарифа не нужен. Может вынести это на уровень command handler-a? C null-промокодом по идее не должно быть никаких иф-ов в handler-е (но это не точно).
Что-то типа
promocode->use()
sum = tariff->calcSum(promocode)
tariff->makeInvoice(sum, invoiceId, ...)
И ещё мне не нравится неявное изменение объекта промокода в методе создания инвойса 🐒

1 ответов

5 просмотров

Глянь на то как это сделано в Sylius например, и все станет ясно и понятно

Похожие вопросы

Обсуждают сегодня

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
Magic
6
сделал сайт, прикрутил в боте сайт, и виджет логина. как автоматически логинить пользователя в аккаунт(телеграм), при входе с бота?
Александра Чернивецкая
5
Объясните, пожалуйста, почему компилятор ругается на использование в условии неинициализированной переменной: int x; Task.Run(async () => { x = await somefunc(); }).Wait...
Александр
5
Ребят, подскажите, пожалуйста, почему в префиксе к ассетам, которые генерируются через фильтр | theme в шаблоне, стал вдруг появляться index.php? Вот так выглядит ссылка на а...
Виталий
1
Всем привет. Ребята, подскажите, пожалуйста. у ботов есть ограничение на отправку сообщений - 30 сообщений в секунду, эти ограничения накладываются на все сообщения? или на со...
Artem Stormageddon
4
Блин, ребята, сори за тупые вопросы. А можно ли как-то открыть вебапку по нажатию на кнопку в меню(которое появляется слева, команды)?
Artem Stormageddon
3
а плаксы из-под питона умеют только в комфортных условиях что-то выдавить из себя?)
Lencore
9
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
13
Это может быть все-таки не флудвейт? у меня ботфазер принимает изменения и отображает даже что они изменились, на видео видно что он прислал якобы уже измененное описание, н...
OVERLINK
13
Коллеги, может знает кто, можно ли цвет бейджа счётчика в BackendMenu менять без бубнов?
Alex Blaze
3
Карта сайта