Класс AbstractPriceCalculator имеет свойство protected float $price и метод public

Question

Класс AbstractPriceCalculator имеет свойство protected float $price и метод public

calculatePrice(): float.

Чтобы посчитать цену свойство $price не должно быть null. Однако из-за особенностей клиентского кода значение для $price неизвестно до момента создания экземпляра AbstractPriceCalculator. То есть нельзя задать $price в конструкторе.

Добавил сеттер для $price. Однако метод calculatePrice могут вызвать до установки $price через сеттер. Как можно обойти? Выбрасывать исключение в calculatePrice, если $price === null? Но наследники могут переопределить метод и не выбрасывать исключение

#backend #php #programming #russian

0

15.04.2022

10 ответов

33 просмотра

W. Blazkowicz Автор вопроса

Ярослав Халюзов
0 присваивай, если null. Меньше исключений будет

да, но это может в будущем породить баги и проблемы бизнесу из-за того, что $price будет равно нулю

0

15.04.2022

Ярослав Халюзов

W. Blazkowicz
да, но это может в будущем породить баги и проблем...

Хммм, тогда может не стоит вызывать calculatePrice до setPrice?

0

15.04.2022

Nikolay Underground

W. Blazkowicz
да, но это может в будущем породить баги и проблем...

В конструкторе делай валидацию

0

15.04.2022

smdc0

W. Blazkowicz
да, но это может в будущем породить баги и проблем...

Вариант с нулем по умолчанию хорош для отладки — это исключает неопределенное поведение. Я бы постарался сделать так: - При создании класса-потомка от AbstractPriceCalculator в конструкторе будет нуль для price. - При записи или других действиях невозможно довести до конца, если price === 0

0

15.04.2022

W. Blazkowicz Автор вопроса

Nikolay Underground
В конструкторе делай валидацию

Как уже сказал, из-за особенностей легаси клиентского кода нельзя задать $price в конструкторе

0

15.04.2022

🐴

По-хорошему это значит, что класс увлькуляятор не подходит для тсоей задачи. Может быть тебя спасет какой-нить враппер вроде new CalculatorWithPrice($calculator, $price) который ты будешь передавать дальше по код. А может и не спасет. Попробуй не натягивать сову на глобус, а переработать модель под твою задачу.

0

15.04.2022

🐴

FYI: описанная проблема, вернее, одно из ее проявлений, называется temporal coupling, погугли, может почерпнешь вдохновение какое-то

0

15.04.2022

Evgeniy Kuvshinov

цены во float мммм

0

15.04.2022

W. Blazkowicz Автор вопроса

🐴
FYI: описанная проблема, вернее, одно из ее проявл...