Ilia
А JRegistry объект проверить на существование is_object правильно?
Dmitry
А JRegistry объект проверить на существование is_object правильно?
А почему нет? Это же объект
Dmitry
Хм ;)
Ilia
Было дело @Septdir высказался на мой файл: "Лучше бы я этого не видел". Так вот если есть где почитать как делать не надо, я бы почитал. Но я думаю разные темы затронуты моим кодом который возмутил. Возможно не знание архитектуры Joomla, возможно незнание в части php, возможно в построении структуры php кода или еще чего. Если увидеть каждый мой косяк по отдельности (локализовать направление для изучения) было бы круто.
Vladimir
Что ещё можно для toc использовать? Руки не подходят :))
вообще я говорил давно обновить плагин
Dmitry
Истек срок
Dmitry
Вот не помню, зачем брал про
Vladimir
там платная в пень не уперлась
Dmitry
Что-то там было
Vladimir
а если мы остаёмся в статье - то пофиг
Dmitry
Логично
Vladimir
так что предлагаю просто поставить free версию
Vladimir
но честно говоря я в шоке.
Vladimir
за год он превратился в тыкву
Vladimir
и там кеш
Dmitry
А, ок
Igor
Было дело @Septdir высказался на мой файл: "Лучше бы я этого не видел". Так вот если есть где почитать как делать не надо, я бы почитал. Но я думаю разные темы затронуты моим кодом который возмутил. Возможно не знание архитектуры Joomla, возможно незнание в части php, возможно в построении структуры php кода или еще чего. Если увидеть каждый мой косяк по отдельности (локализовать направление для изучения) было бы круто.
Ну начните с простого. С формата кода PHPDoc и использования namespaces
Igor
Было дело @Septdir высказался на мой файл: "Лучше бы я этого не видел". Так вот если есть где почитать как делать не надо, я бы почитал. Но я думаю разные темы затронуты моим кодом который возмутил. Возможно не знание архитектуры Joomla, возможно незнание в части php, возможно в построении структуры php кода или еще чего. Если увидеть каждый мой косяк по отдельности (локализовать направление для изучения) было бы круто.
Igor
Ну а главное помните что ваш код может перейти по наследству и желательно чтобы не было "Тут легче все переписать чем разбираться"
Ilia
Igor
Это не очень. Это про то что у J! есть свое логирование?
Нет. просто у вас пишутся одинаковые логи даже если parent::save вернте false
Ilia
Нет. просто у вас пишутся одинаковые логи даже если parent::save вернте false
+ Да не видел этого в упор
Igor
И да на кой var_export когда есть json
Igor
Когда я учился писать, мне всегда мой код казался каким-то детским так что я убил кучу времени чтобы он выглядел по взрослому и понятно.
И в итоге и мне удобно копипасить у себя и людям читать.
Igor
Это описание в комментах? или что-то еще?
Формат строк, коментарии проверки каждого чиха.
Сейчас вышлю переписанный это кусок.
Ilia
Формат строк, коментарии проверки каждого чиха.
Сейчас вышлю переписанный это кусок.
Дай просто свой кусок я на примере посмотрю. Пойму заодно - Смогу ли я разобраться в чужом?
Dmitry
Формат строк, коментарии проверки каждого чиха.
Сейчас вышлю переписанный это кусок.
Да, комменты на чих точно не нужны
Dmitry
Комменты нужны там, где код непонятный ))
Dmitry
Или пояснения, почему сделано именно так
Dmitry
Потому что это чаще всего забывается
Igor
Дай просто свой кусок я на примере посмотрю. Пойму заодно - Смогу ли я разобраться в чужом?
https://github.com/SeptdirWorkshop/SWJProjects/blob/master/com_swjprojects/admin/models/project.php#L341
Ну вот к примеру. Хотя тут довольно слоный save
Dmitry
@platonische почитай книжку "Совершенный код". https://www.ozon.ru/product/sovershennyy-kod-master-klass-138437220/
Очень классная
Dmitry
Она в принципе о подходе к написанию кода
Dmitry
https://github.com/SeptdirWorkshop/SWJProjects/blob/master/com_swjprojects/admin/models/project.php#L495
Vladimir
Dmitry
https://github.com/SeptdirWorkshop/SWJProjects/blob/master/com_swjprojects/admin/models/project.php#L495
@Septdir эт чих 😂
Vladimir
пойдет?
Dmitry
Dmitry
Вообще надо такой save разбивать на несколько
Dmitry
В идеале
Ilia
Igor
@Septdir эт чих 😂
Этот чих меня не раз спасал. когда код проекта доходит то 20-30к строк. а тебе надо логику modified отрефакторить
Dmitry
Все эти if отрефакторить в методы
Vladimir
Ilia
Спасибо. Не все сразу, но идет потихонечку мое обучение
Dmitry
Тупо extract method
Ilia
Меня уже xdebug выдернул на новый уровень. Но не могу все что пишите и знаете в одночасье забрать
Dmitry
В одночасье не надо ;)
Igor
save я переписал
Igor
правда вслепую не уверен что будет работать как надо =)
Dmitry
Не мне тебя учить 😊
Igor
Ну это на твоё усмотрение :)
Я сейчас довольно многое переношу в helpers а еще использую свзязь моделей.
Ilia
Я сейчас довольно многое переношу в helpers а еще использую свзязь моделей.
А принцип выбора Хелпер или модель есть?
Dmitry
Я сейчас довольно многое переношу в helpers а еще использую свзязь моделей.
На статику переходишь?))
Dmitry
Трейты тогда уж
Igor
Меня уже xdebug выдернул на новый уровень. Но не могу все что пишите и знаете в одночасье забрать
о забыл, еще свежую плюшку при подключении моделей внтури моделей можно еще сделать так
$logModel->set('context', $this->context);
Так в плагинах будет видно откуда пошел save
Igor
На статику переходишь?))
Не совсем, скорее просто для удобсва всякие калькуляции или форматирование
Dmitry
Dmitry
Кстати, а set не деприкейтед?
Igor
А принцип выбора Хелпер или модель есть?
Ну тут ответ в самих названиеях helper это помошник.
Простой пример получить список всех id дочерних категорий.
Если это надо всего лишь один раз делашь прямо. там.
Если дважды но в одной моделе то делаешь отдельный метод.
Ну если в 5 моделях то в helper и пусть держит результат в раме