Контроллеры для этого и нужны, модель служит для работы с физическим уровнем, а контроллер уже дергает модель так как ему угодно

Контроллер только посредник. Никакой бизнес логики там быть не должно

какие люди в голивуде

Контроллер только посредник. Никакой бизнес логики там быть не должно

Во вьюхе не должно

вынеси в модель и не парься, не передавай с with а бери напрямую

а как модель вынести :? есть какой-то пример я новичок просто

какие люди в голивуде

Приходится холиварить))

а как модель вынести :? есть какой-то пример я новичок просто

покажи фулл код контроллера

Контроллеры для этого и нужны, модель служит для работы с физическим уровнем, а контроллер уже дергает модель так как ему угодно

Давай не будем

Матчасть лучше изучать

выбрать данные и прокинуть в респонс/вьюху

это не бизнесс логика

выборкой занимается модель отрисовкой вьюха никакой принцип не нарушен

посредник между request/response

Матчасть лучше изучать

Пускай будет так, если тебе с этой мыслью будет проще уснуть))

Посредник не делает sql запросы

Не должен

Посредник не делает sql запросы

а контроллер их и не делает

он делегирует это модели)

Матчасть лучше изучать

а контроллер их и не делает

Where не надо там

Матчасть лучше изучать

Хочешь поговорить об этом?

Может стоит хотябы почитать про тупыте толстые уродливые контроллеры?

Или хотя бы ларакасты посмотреть?

покажи фулл код контроллера

https://codepad.remoteinterview.io/QGAQCLMMKZ

Where не надо там

с этим согласен, больше претендует на поведение у модели но говорить что не стоит делать выборки, уж точно не стоит

Или хотя бы ларакасты посмотреть?

чего мы там не видели)

На скрине вообще не очень название модели Type

а как модель вынести :? есть какой-то пример я новичок просто

а для чего тебе там айдишники какую-то юайку отрисовать по id => name ?

Я бы назвал с точки зрения бизнеса эту модель и сделал внутри метод. И вызвал бы внутри контроллера этот метод. Его можно переиспользовать. Добавится доб логика выборки, то сделаешь один раз в модели а не в 5 контроллерах

PartnerType::getList();

Типа такого

Я бы назвал с точки зрения бизнеса эту модель и сделал внутри метод. И вызвал бы внутри контроллера этот метод. Его можно переиспользовать. Добавится доб логика выборки, то сделаешь один раз в модели а не в 5 контроллерах

А если тебе надо выбрать что-то, что ты один раз юзать будешь, пусть какая то статистика

И вообще через DI модель делаю

Ты для этого будешь модель расширять?

ну у меня эта выборка будет работать только здесь

просто кмк, return view('view.name',$types)->with['title' => 'Успех'); логичнее

Ты никогда не знаешь будующиэ требований

Тогда в лоб вопрос - зачем нужны контроллеры, если там не должно быть бизнеса?

Я бы порекомендовал хотя бы серию ларакастов где джефри делает форум. Там все наглядно

Тогда в лоб вопрос - зачем нужны контроллеры, если там не должно быть бизнеса?

Контроллеры принимают запросы и отдают респонсы

Я бы порекомендовал хотя бы серию ларакастов где джефри делает форум. Там все наглядно

я бы не ставил ларакасты во главу угла там все делают максимально просто

для долбоебов, что бы порог вхождения был ниже)

Они общаются с нужными сервисами для того чтобы вернуть респонс

Ок пусть не ларакасты. Тогда вообще стоит почитать про mvc, mvp

насколько я понял тред о $types = Model::select('field','value'); и $types = $this->manager->getTypes('field','value');

Так а почему бы в таком случае не ебашить сразу в модель?) мне кажется ты не до конца mvc понимаешь. А если еще тебя познакомить с mvvc так совсем в обморок можешь упасть)

Ок пусть не ларакасты. Тогда вообще стоит почитать про mvc, mvp

ларавель не mvc

а mvcs

Mvp я бы назвал

Так а почему бы в таком случае не ебашить сразу в модель?) мне кажется ты не до конца mvc понимаешь. А если еще тебя познакомить с mvvc так совсем в обморок можешь упасть)

Ты можешь вообще в один файл ебашить. Разрешаю

Mvp я бы назвал

Тебе рассказать что такое mvp и почему лара не mvp? Начать и закончить даже можно с того, куда приземляются запросы, не на presentation layer, если юзать blade. Если юзать vue/reack/knockout etc то еще можно

https://ru.m.wikipedia.org/wiki/Model-View-Controller

Но из коробки это mvc

Тут выдержки разрабов zend

Тебе рассказать что такое mvp и почему лара не mvp? Начать и закончить даже можно с того, куда приземляются запросы, не на presentation layer, если юзать blade. Если юзать vue/reack/knockout etc то еще можно

Не надо мне рассказывать

Ок, огнетушитель принести?)

Тебя бомбит?. Тушись

Разбераюсь с Кроном и шедулерами, создал command, сказал выполнять каждую минуту, инсертить в базу значения, чисто для теста. Запускаю артисаном команду, выполняется один раз и все. Что не так делаю?

Интересно, а ты вот контрибьютил в лару или так, просто книжек начитался и по-академически вопросы решаешь?)

Разбераюсь с Кроном и шедулерами, создал command, сказал выполнять каждую минуту, инсертить в базу значения, чисто для теста. Запускаю артисаном команду, выполняется один раз и все. Что не так делаю?

Тебе на крон надо поставить php artisan schedule:run чтоб ее дергало каждую минуту

crontab -e

Интересно, а ты вот контрибьютил в лару или так, просто книжек начитался и по-академически вопросы решаешь?)

Официальную русскую доку переводил. Старожил из гиттера

А что?

Приди туда и спроси. Посмотри там мою карму

Шедуль артисана только посредник между твоим кодом и кроном

Меня там знают

Ну мне просто стало интересно, в чате node js часто приходилось спорить с буратинами

ERROR: S client not available

Там прям не кодеры а мухи

Тут хоть интересно

В свое время в гиттере было бурно. Много интересных ребят сидели.

@jhaoda только зануда все время доку заставлял читать. Но спасибо ему

На самом деле потом входит в привычку все в контроллере писать и код разрастается. У себя на проекте мы делал репозитории, в которых были все выборки

И сервисные классы которые всем этим и другим управляли

Легче потом тестами покрывать. Полный отказ от фасадов ещё был и DI

чуть получше,перед return nl добавь, так нужно + подумай над тем, что бы вынести это отдельным методом в модель

чуть получше,перед return nl добавь, так нужно + подумай над тем, что бы вынести это отдельным методом в модель

не встречал nl nl return view(

nl это new line

return'ы выделяют явно таким образом

не встречал nl nl return view(

кинь код на paste.laravel.io

кинь код на paste.laravel.io

https://paste.laravel.io/b285d8f2-1e67-4c18-9d07-dd5e78b6d408

https://paste.laravel.io/b285d8f2-1e67-4c18-9d07-dd5e78b6d408

для чего тебе отдельный контроллер для Add?

https://paste.laravel.io/b285d8f2-1e67-4c18-9d07-dd5e78b6d408

замени валидацию на формреквест

сделай просто PartnerController

Шедуль артисана только посредник между твоим кодом и кроном

Спасибо! А втаком случае в крон табе 0ли прописывать ?

замени валидацию на формреквест

$validatedData = $request->validate([

$validatedData = $request->validate([

не, пусть тебе прилетает в контроллер не реквест

а инстанс формреквеста, уже провалидированный

https://laravel.com/docs/5.6/validation#creating-form-requests

Спасибо! А втаком случае в крон табе 0ли прописывать ?

Когда хочешь например 0 часов или 0 минут

замени валидацию на формреквест

+

5 звездочек - каждую минуту

5 звездочек - каждую минуту

Но в таком случае зачем указывать в ларе количество дерганий?

зачем тогда это шедуль

Смотри, ты указываешь один раз пять звездочек в кроне а дальше кодом решаешь когда и как запускать таски

Чтоб не делать тысячу крон задач

Смотри, ты указываешь один раз пять звездочек в кроне а дальше кодом решаешь когда и как запускать таски

В таком случае крон будет каждую минуту смотреьть в задачу а лара уже на своих счетчиках решает внутри когда и как что запускать

В таком случае крон будет каждую минуту смотреьть в задачу а лара уже на своих счетчиках решает внутри когда и как что запускать

Именно

Именно

Понял, спасибо огромное