AZ
Добрый день, может кто-то объяснить этот код нормальный или плохой. Если плохой накидайте несколько аршументов.
Сам метод находится в Модели
Сам метод находится в Модели
Size: a a a
2020 March 05
pf
Может кто то поделится laravel nova хочу затестить перед покупкой, буду благодарен.
да да, перед покупкой)) все так говорят
AP
Добрый день, может кто-то объяснить этот код нормальный или плохой. Если плохой накидайте несколько аршументов.
Сам метод находится в Модели
Сам метод находится в Модели
ну тут с какой стороны посмотреть
F
Добрый день, может кто-то объяснить этот код нормальный или плохой. Если плохой накидайте несколько аршументов.
Сам метод находится в Модели
Сам метод находится в Модели
Плохой, не нужно писать всю работу писать в моделе, нужно использовать контроллеры, и валидацию проводить в Request
AP
какбы модель занимается какими-то делами, которыми не должна
AZ
Слабые аргументы, человек мне не поверит) Для меня этот код просто ужасен, но мне не хватает опыта объяснить почему, и человек мне отвечает, раз в ларавель есть такое то это ок и этот код прямо очень хорош.
DK
Можно сказать что она делает валидацию :D Но это можно все решить в реквестах и контролере
DK
Слабые аргументы, человек мне не поверит) Для меня этот код просто ужасен, но мне не хватает опыта объяснить почему, и человек мне отвечает, раз в ларавель есть такое то это ок и этот код прямо очень хорош.
я бы предложил логику разнести в Request и контрллер
AZ
А вообще это нормальная практика в модели вызывать фасады? по сути это почти глобальная переменная.
DK
туже атворизацию, наличие файла и прочее
DK
а в котролере уже юзать createFrom...
EG
А вообще это нормальная практика в модели вызывать фасады? по сути это почти глобальная переменная.
Фасады это зло!!! Но иногда без них не куда
AZ
Фасады это зло!!! Но иногда без них не куда
Согласен! Но как обосновонно обяснить что их нельзя использовать и они только для крайних случаев? И тем более их не нужно вызывать в модели…
F
Слабые аргументы, человек мне не поверит) Для меня этот код просто ужасен, но мне не хватает опыта объяснить почему, и человек мне отвечает, раз в ларавель есть такое то это ок и этот код прямо очень хорош.
Ему уже ничего не поможет
AP
с архитектурной точки зрения код говно
AP
модель зависит от других классов
AP
нарушает солид, гыгы
EG
Согласен! Но как обосновонно обяснить что их нельзя использовать и они только для крайних случаев? И тем более их не нужно вызывать в модели…
Пусть попробует на это метод написать тест, и все встанет на свои места
OS
Добрый день, может кто-то объяснить этот код нормальный или плохой. Если плохой накидайте несколько аршументов.
Сам метод находится в Модели
Сам метод находится в Модели
думаю это не настоящий МВС . Модель только нужно занимация с базами, валидация это не его работа. Валидация для контроллера.
DK
думаю это не настоящий МВС . Модель только нужно занимация с базами, валидация это не его работа. Валидация для контроллера.
а в случае ларавель это реквесты :D