Как правильно писать под MVC на laravel пример?

Question

[cryp24]

Не как не могу понять как правильно писать под MVC, в частности проблемы с пониманием что пихать в контроллер а что в модель, с view более менее понятно, прошу подсказать на моем примере ибо теорию уже зачитался.

Итак есть парсер новостей, на данный момент это выглядит так:
В App создал модель, в ней создал функцию (код большой поэтому напишу просто блоками 1-2-3-4-5)

public static function pars() {

1) Парсим страницу + распарсиваем нужные данные. На выходе имеем массив спарсинных данных для вставки в бд.
2) Сверяемся с базой на наличие дублей и формируем новый массив для вставки без дублей. На выходе имеем массив уникальных новостей.
3) Далее идет вставка этого массива в бд
4) После поступления новых данных, идет кое какая проверка по новым новостям на основании которой, удаляются некоторые старые.
5) Затем идет выборка по определенным критериям из этих новостей и отправка мне смс, если такие есть

return;
}

соответственно в web.php

Route::get('/pars', function () {

    App\News::pars();
    return ;
});

На выходе имею парсер site/pars который прикручивается на крон и все готова.

Как вы заметили в этой схеме нет контроллера. Как правильно распределить код из моего примера чтоб соответствовало модели MVC?

Comments

[JhaoDa]

Так же, как и на любом другом фреймворке.

Как вы заметили в этой схеме нет контроллера. Как правильно распределить код из моего примера чтоб соответствовало модели MVC?

Я бы сначала беспокоился о принципиальном нарушении всех лучших практик, а не об отсутствующем контроллере...

Answer 1

[Александр Аксентьев]

На выходе имею парсер site/pars который прикручивается на крон и все готова.

И всё готово максимально кривокосо, а не так как хотелось бы https://laravel.com/docs/5.8/scheduling
И контроллер тут вообще не нужен, и натягивание аббревиатуры MVC на всё что угодно.
И метод в модели который что-то делает тоже, хотя сама модель там вообще не используется https://laravel.com/docs/5.8/providers
И сам парсинг всё-в-одном https://laravel.com/docs/5.8/queues

Answer 2

[Роман]

Судя по коду который я видел там и сям в Ларавеле принято пихать логику как раз таки не в модель а в контроллер :)

Модель обычно используют для взаимодействия с БД и настройки отношений.

Comments

[JhaoDa]

Ты видел плохой код. Логику надо пихать туда, где ей место. Если это такая логика, что ей место в модели, то там она и должна быть. А если ей место в классе-сервисе, то... ну ты понял (надеюсь).

[cryp24]

Это все прекрасно что вы друг другу пояснили ))), но как насчет моего случая, в моем парсере есть взаимодействие с бд, значит модель всеже нужна?. Может его нужно разбить на несколько частей, тогда вопрос куда какую часть записать.

[JhaoDa]

cryp24, я предлагаю начать с изучения документации.

[cryp24]

JhaoDa, по документации не как непонятно, для того и пример опубликовал

[JhaoDa]

cryp24, суровая реальность, чувак. Отвечать на твои более-менее чёткие вопросы — пожалуйста, учить тебя чтению документации и программированию — нет.

[cryp24]

JhaoDa, На данный момент вместо твоего "пожалуйста" я вижу только общий посыл мол иди изучай сам. Ок к тебе вопросов нет.

[JhaoDa]

cryp24, ну так я уже сказал — тебе не о контроллерах надо беспокоиться, а в принципе о неправильном подходе. Правильный подход описан в документации, выше дали прямых ссылок.

[Роман]

cryp24, в общем суть здесь в том, что нет какой-то единой общепринятой догмы на этот счет :)

Наверное каждый приходит к пониманию своего стиля методом проб и ошибок.

Как-то так ))