Size: a a a

2021 February 18

AS

Aliaksandr Sadouski in Evolution CMS
Pathologic
розетка может реализовать ведь и несколько интерфейсов
Так и класс может несколько реализовывать. Но мы же заранее знаем чего в какие дырки розетки тыкать (какие там интерфейсы). А тут получается, про 2 дырки мы знаем, но есть еще 5, причем чтобы тыркать в одну надо сначала потыркать в другие
источник

AS

Aliaksandr Sadouski in Evolution CMS
Просто может там еще какая логика заложена, чтоб использовать отдельно валидацию т .п.
источник

AS

Aliaksandr Sadouski in Evolution CMS
Если нет, так там одного метода process() везде б хватило из публичных )
источник

P

Pathologic in Evolution CMS
в целом да
источник

P

Pathologic in Evolution CMS
просто @Serious_t91 решил не заморачиваться
источник

P

Pathologic in Evolution CMS
так получается, что хорошая идея не продумывается до конца, а через какое-то время вроде и понятно, что нужно переделать, но уже поздно
источник

AS

Aliaksandr Sadouski in Evolution CMS
Ну вот я глянул еще раз - эти же все сервисы через общий фасад DocumentManager и разруливаются, и там везде только process и используется
источник

P

Pathologic in Evolution CMS
та же проблема с доклистером, например
источник

P

Pathologic in Evolution CMS
я предлагал кроме интерфейса и абстрактный класс сделать
источник

P

Pathologic in Evolution CMS
чтобы код сервисов был еще проще
источник

P

Pathologic in Evolution CMS
вот я сейчас присматривался, как сделать нормальное событие для документов и вижу, что все плохо
источник

P

Pathologic in Evolution CMS
а когда мы обсуждали, так никто и не заметил подвоха
источник

P

Pathologic in Evolution CMS
Aliaksandr Sadouski
Ну вот я глянул еще раз - эти же все сервисы через общий фасад DocumentManager и разруливаются, и там везде только process и используется
мне кажется, что такая схема лучше modxapi c огромными классами
источник

P

Pathologic in Evolution CMS
а потом окажется, что хуйня это а не схема и все неудобно
источник

AS

Aliaksandr Sadouski in Evolution CMS
Так всегда так и бывает :)
источник

AS

Aliaksandr Sadouski in Evolution CMS
Например, чтоб подшаманить свое хранение тв опять не подменишь пару методов, а надо полностью весь пакет заменять
источник

AS

Aliaksandr Sadouski in Evolution CMS
Потому что все жёстко вшито в два класса
источник

P

Pathologic in Evolution CMS
не жестко
источник

P

Pathologic in Evolution CMS
полностью весь пакет заменять не нужно
источник

P

Pathologic in Evolution CMS
заменяешь менеджер и какие-то сервисы
источник