КГ
меня самого долго пинали базовым классом для саг, но объективно я не вижу ни единого варианта его замены на что-то более "адекватное", что не заставило бы клиентский код экстендить что-либо
Дык может у людей уже есть свой базовый класс, а
Size: a a a
2020 December 23
КГ
а экстендить свой базовый от твоего по каким то причинам не хотят)
ИЛ
значит надо делать свой логгер от абстрактного и включать его зависимостью
КГ
А трейт с интерфейсом навесить дополнительно легко почти без ограничений.
Ну то есть понятно, что уже звучит как хуета, но должна ли либа ограничивать возможность делать дичь которая не особо связана с либой)
Ну то есть понятно, что уже звучит как хуета, но должна ли либа ограничивать возможность делать дичь которая не особо связана с либой)
ИЛ
а это вот, логгер и не логгер в куче - это смешивание интерфейсов
ИЛ
есть секта адептов "выпилим аннотации, так это код фреймворка", но строго говоря это не имеет особого значения. Ни базовый класс, ни трейт не приведут тебя ни к каким проблемам. И с точки зрения разраба либы это самый дешёвый и безотказный вариант
из дерева пусть выпиливают
ИЛ
а делать EventSourcedTrait и лепить для него AggregateInterface - ну такое
поскольку никто не придумал столь же ценную вторую штуку, то я согласен, что этому место в базовом абстрактном классе
MM
А трейт с интерфейсом навесить дополнительно легко почти без ограничений.
Ну то есть понятно, что уже звучит как хуета, но должна ли либа ограничивать возможность делать дичь которая не особо связана с либой)
Ну то есть понятно, что уже звучит как хуета, но должна ли либа ограничивать возможность делать дичь которая не особо связана с либой)
а если у них в базовом классе метод с таким же именем?)
КГ
а если у них в базовом классе метод с таким же именем?)
Поэтому и написал "почти" без ограничений)
MM
ну т.е. ничего не изменилось)
КГ
ну т.е. ничего не изменилось)
Изменилось. Может у меня уже есть наследование от другого вендора
MM
ну, в данном случае ты практически гарантированно делаешь хуиту и тебя трейты не спасут, только ухудшат положение
КГ
ну, в данном случае ты практически гарантированно делаешь хуиту и тебя трейты не спасут, только ухудшат положение
Да, но твои ограничения всё равно умнее никого не сделают, а либо уменьшат аудиторию либо вынуят ещё какую хуету сделать чтобы твою либу интегрировать
ИЛ
cs fixer остался
MM
Да, но твои ограничения всё равно умнее никого не сделают, а либо уменьшат аудиторию либо вынуят ещё какую хуету сделать чтобы твою либу интегрировать
да и пофиг)
AD
cs fixer остался
Прод не обновится(((
ИЛ
Прод не обновится(((
ну справедливости ради, он просто не работает, а обновлению не мешает
AD
ну справедливости ради, он просто не работает, а обновлению не мешает
Поставь две версии пхп)
КГ
да и пофиг)
Скажем тебе ничего не стоит вынести функционал из твоего базового класса в трейт и вынести интерфейс. И обратная совместимость цела, и теперь противники наследования смогу затащить твой трейт)
Повторюсь - да хуета, но форсить нормальный код нужно чуток другими способами
Повторюсь - да хуета, но форсить нормальный код нужно чуток другими способами
MM
Скажем тебе ничего не стоит вынести функционал из твоего базового класса в трейт и вынести интерфейс. И обратная совместимость цела, и теперь противники наследования смогу затащить твой трейт)
Повторюсь - да хуета, но форсить нормальный код нужно чуток другими способами
Повторюсь - да хуета, но форсить нормальный код нужно чуток другими способами
ты меня с тейлором перепутал