AM
не пушнул :)
Size: a a a
2020 March 04
DS
но не факт что так надо
Это работает с еще фиг знает какой версии php и много где используется, только не в абстрактном виде.
А
Это работает с еще фиг знает какой версии php и много где используется, только не в абстрактном виде.
да, но может считаться плохой практикой, типа хаком
AM
Я понимаю что технически это работает. Но это явно тот ещё хак. На это указывает то, что get() болтается бесполезно там где не должен.
AM
То есть притянуто за уши дерево наследования.
DS
Я понимаю что технически это работает. Но это явно тот ещё хак. На это указывает то, что get() болтается бесполезно там где не должен.
Это однозначно лучше, чем публичный сеттер будет торчать.
DS
не пушнул :)
Допушил, не подхватил файлы в коммит😏
AM
ну если так сравнивать A и B, то да, я согласен что лучше
А
я бы ещё попросил причесать порядок методов в классах, чтобы public были вверху, private внизу
DS
я бы ещё попросил причесать порядок методов в классах, чтобы public были вверху, private внизу
Я все хотел спросить, нормально ли это у вас, а то оно повсеместно такое встречается
А
Я все хотел спросить, нормально ли это у вас, а то оно повсеместно такое встречается
не всегда на это падает внимание, но лучше в методах и свойствах следовать порядку
AM
6. Нужны тесты на delegateLookup.
А
Я понимаю что технически это работает. Но это явно тот ещё хак. На это указывает то, что get() болтается бесполезно там где не должен.
а про
get это где?DS
не всегда на это падает внимание, но лучше в методах и свойствах следовать порядку
Ну я всегда сортирую public > protected > private
DS
6. Нужны тесты на delegateLookup.
Через пару часов сделаю, сейчас мне надо отъехать на время.
А
Ну я всегда сортирую public > protected > private
я бы ещё добавил, что первыми среди публичных с префиксом __
AM
а про
get это где?DS
Методы посортирую перед самым мержем, потому как с диффом проблемы будут
AM
давай отдельно
AM
а то потом чёрт ногу сломит...