Dmitriy
В ларе не классический мвс?
Denis
Для обращения внутрь модели я пишу методы в классе Model. Для обращения внутрь всей коллекции моделей я пишу методы в классе Repository
Denis
Create - изначально в репозитории
Denis
Для ограничения выборки использую класс Criteria
Denis
Для активных действий над коллекцией - класс Collection
Dmitriy
Для обращения внутрь модели я пишу методы в классе Model. Для обращения внутрь всей коллекции моделей я пишу методы в классе Repository
я так понимаю ты не пишешь тесты ?
Denis
Неа
Dmitriy
Ну это было понятно
SarcasmIO
🙃
Denis
Думаешь будут траблы?
Denis
Честно - не врубаюсь для чего они
SarcasmIO
агась
Dmitriy
тестозависимость она такая
Dmitriy
Нее. Раз в модели что-то там выбирается и еще что-то делается
Denis
Модель обращается внутрь себя
Denis
Только
Denis
Давай я зашарю модуль и вы зацените. У меня ощущение, что мы говорим чуть разными словами
Dmitriy
Можно
Denis
Denis
Не идеал, конечно ))
Dmitriy
public function scopeLive(Builder $query)
{
return $query
->where('enabled', 1)
->where('publish_at', '<=', date('Y-m-d H:i:s'))
->orderBy('publish_at', 'DESC');
}
Зачем из модели делать QueryBuilder ?
Denis
Вот я и говорю - не идеал там. Это тот полугодичный модуль - там есть ошибки.
То что ты показываешь - должно быть в критерии
Denis
live()
Denis
Так молчите, аж страшно )
Dmitriy
600 строк, много что-то
На сколько я вижу модель это склад всячины и она выступает в роли QueryBuilder, Response(!!!!), модель знает что она еще что она кешируется хм.
Denis
Хорошо, можешь показать свою старенькую? Есть не под НДА?
Dmitriy
любую модель ?
Denis
Что считаешь нужным
Dmitriy
Ну модели пустые там ничего интересного.
Dmitriy
https://gist.github.com/exileed/1ca3c26ca8a9d7f5c4bc255c122d5d20
Dmitriy
это код этого бота
Dmitriy
/link@yetanotherstatsbot
Yet Another Stats (β)
https://yasb.exileed.com/t/laravel_pro
Dmitriy
https://gist.github.com/exileed/7cb85920ea7f6336cd58a544af645d2a вот такой был раньше контроллер.
Счас все ушло в еще 1 сервис. ибо выводов уже много
Denis
/link@yetanotherstatsbot
Denis
Whoops, looks like something went wrong.
Dmitriy
Это я обновляю
Yet Another Stats (β)
https://yasb.exileed.com/t/laravel_pro
Юрий
какой же блядский синтаксис изобразили для множественных юзов, простите
Юрий
каждый раз как вижу блюй берет
Юрий
use App\Supports\Telegram\Statistics\Chat\{
MessagePerDay, TopChats, TopMessageUsers, TopRatingUsers
}
Юрий
я про это
Denis
А какие критерии включает в себя оценка рейтинга?
Юрий
@ExileeD кстати это иде поддерживает уже?
Юрий
но все равно блевать тянет. как rfc увидел принятый офигел
Dmitriy
А какие критерии включает в себя оценка рейтинга?
кол-во сообщений, тип, полезность для сообщества и т.д
Denis
Дизайн норм, мне понравился. Он обычно включен?
Dmitriy
Он всегда включен, если я только не обновляю
Denis
Я попробовал сейчас ответить на сообщение бота
Юрий
https://yasb.exileed.com/t/laravel_pro
Denis
Прикольно, буду юзать нет-нет)
Denis
По коду скажу что разрядил бы пустыми строками кое где - это раз. Имхо - реально читаемей будет в трудных условиях (экран телефона, например).
Denis
Два - то что это просто перечисление действий в контроллере
Denis
Правильно?
Юрий
wut?
Юрий
я бы наоборот пустые строки выкинул
Юрий
Юрий
сколько мусора
Denis
Denis
https://gist.github.com/exileed/7cb85920ea7f6336cd58a544af645d2a вот такой был раньше контроллер.
Счас все ушло в еще 1 сервис. ибо выводов уже много
https://gist.github.com/Piterden/ef97abd09f1c85205403730b78a9848b
Anonymous
Жиесть
Anonymous
про сервисы мы не слышали
Denis
Да не знаю, код не мой, я просто навел там чуть порядка.
Denis
Мой есть выше в архиве, если интересно - там тоже косяков хватает )))
Anonymous
Где 666 челевечег
Anonymous
Закидывай в след раз на pastebin
Юрий
https://gist.github.com/Piterden/ef97abd09f1c85205403730b78a9848b
Перед неймспейсом пустая строка должна быть. После тоже, перед юзами. Пхпдок к классу где? Зато есть заглушка под него. Между функциями пустая строка где?
Dan
омагадомагад сколько кода-говнокода омагад, срочно под кат прячь
Anonymous
@MatteProxy сорь