команды это как контроллеры . только одают и получают

команды это как контроллеры . только одают и получают

Ты про какие команды?

Консольные?

а еще есть какие команды ?

Есть

Те, что лаунчатся с помощью DispatchesJobs

В них и логика

А если джопсы не нужны ?

Тогда не получится использовать команды. Вопрос один - нужны ли команды

Джобс умер как Френк Синатра

Там где создашь там и будет слой

Смысл вытащить логику с контроллера и модели . а там ей не место и разбить ее на отдельные классы. Модель тяжело тестировать

Давай тогда начнем с логики. Что это такое?

Бизнес логика

Вынимание модели по slug - это бизнес логика?

нет.

Приведи, плиз, пример бизнес логики

Отправка почты. разные оповещения, обработка и сохранение чего-то в бд

То есть create - это бизнес логика?

У меня эта операция находится в репозитории

Что такое репозиторий?

И я ее не собираюсь оттуда перемещать

В ларе не классический мвс?

То есть create - это бизнес логика?

Обработка и создание это уже БЛ

Для обращения внутрь модели я пишу методы в классе Model. Для обращения внутрь всей коллекции моделей я пишу методы в классе Repository

Create - изначально в репозитории

Для ограничения выборки использую класс Criteria

Для активных действий над коллекцией - класс Collection

Для обращения внутрь модели я пишу методы в классе Model. Для обращения внутрь всей коллекции моделей я пишу методы в классе Repository

я так понимаю ты не пишешь тесты ?

Неа

Ну это было понятно

?

Думаешь будут траблы?

?

у тебя триггер на слово тесты ?

Честно - не врубаюсь для чего они

агась

тестозависимость она такая

у тебя триггер на слово тесты ?

Кстати - триггеры на слова в слаке ништяк

Думаешь будут траблы?

Ну при тестировании ты бы делил логику между классами

Ну при тестировании ты бы делил логику между классами

Ээм. Я и так делю. Или нет?

Нее. Раз в модели что-то там выбирается и еще что-то делается

Нее. Раз в модели что-то там выбирается и еще что-то делается

Что именно?

Модель обращается внутрь себя

Только

Давай я зашарю модуль и вы зацените. У меня ощущение, что мы говорим чуть разными словами

Можно

Не идеал, конечно ))

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 ?

Вот я и говорю - не идеал там. Это тот полугодичный модуль - там есть ошибки. То что ты показываешь - должно быть в критерии

live()

Так молчите, аж страшно )

600 строк, много что-то На сколько я вижу модель это склад всячины и она выступает в роли QueryBuilder, Response(!!!!), модель знает что она еще что она кешируется хм.

Хорошо, можешь показать свою старенькую? Есть не под НДА?

любую модель ?

Что считаешь нужным

Ну модели пустые там ничего интересного.

https://gist.github.com/exileed/1ca3c26ca8a9d7f5c4bc255c122d5d20

это код этого бота

/link@yetanotherstatsbot

https://yasb.exileed.com/t/laravel_pro

https://gist.github.com/exileed/7cb85920ea7f6336cd58a544af645d2a вот такой был раньше контроллер. Счас все ушло в еще 1 сервис. ибо выводов уже много

/link@yetanotherstatsbot

Whoops, looks like something went wrong.

Это я обновляю

ERROR: S client not available

https://yasb.exileed.com/t/laravel_pro

какой же блядский синтаксис изобразили для множественных юзов, простите

каждый раз как вижу блюй берет

use App\Supports\Telegram\Statistics\Chat\{ MessagePerDay, TopChats, TopMessageUsers, TopRatingUsers }

я про это

А какие критерии включает в себя оценка рейтинга?

какой же блядский синтаксис изобразили для множественных юзов, простите

+

@ExileeD кстати это иде поддерживает уже?

@ExileeD кстати это иде поддерживает уже?

ага

но все равно блевать тянет. как rfc увидел принятый офигел

А какие критерии включает в себя оценка рейтинга?

кол-во сообщений, тип, полезность для сообщества и т.д

Дизайн норм, мне понравился. Он обычно включен?

Он всегда включен, если я только не обновляю

Я попробовал сейчас ответить на сообщение бота

Дизайн норм, мне понравился. Он обычно включен?

да вроде стата и сейчас доступна

https://yasb.exileed.com/t/laravel_pro

Прикольно, буду юзать нет-нет)

По коду скажу что разрядил бы пустыми строками кое где - это раз. Имхо - реально читаемей будет в трудных условиях (экран телефона, например).

Два - то что это просто перечисление действий в контроллере

Правильно?

wut?

я бы наоборот пустые строки выкинул

сколько мусора

я бы наоборот пустые строки выкинул

Именно поэтому я и говорю имхо

У меня такую херь форматтер вырезает

сколько мусора

Плеер на кофе лучше не светить, понял )))

https://gist.github.com/exileed/7cb85920ea7f6336cd58a544af645d2a вот такой был раньше контроллер. Счас все ушло в еще 1 сервис. ибо выводов уже много

https://gist.github.com/Piterden/ef97abd09f1c85205403730b78a9848b

Жиесть

про сервисы мы не слышали

Да не знаю, код не мой, я просто навел там чуть порядка.