IG
artisan make:request StoreProductRequest
потом вынести в prepareForValidation
потом вынести в prepareForValidation
а ну у меня есть это
public function store(StoreProduct $request)
Size: a a a
2020 May 13
IG
ИЛ
Просто ларавельщики не в курсе, что их Model – это не модель и их Facade – это не фасад. У них всё меняет смысл.
Наша модель как раз обычная модель AR, а фасад - да, не фасад
RK
public function store(StoreProduct $request)
Кстати правильнее всеже дописать StoreProductRequest без реквест выглядит как эвент
ИЛ
Наша модель как раз обычная модель AR, а фасад - да, не фасад
RK
Холи вар
IG
Кстати правильнее всеже дописать StoreProductRequest без реквест выглядит как эвент
да блин, че так сокращать то
RK
artisan make:request StoreProductRequest
потом вынести в prepareForValidation
потом вынести в prepareForValidation
Ну такое) слаг не относится не каким образом к валидатору
IG
хотя, минималистический код
DE
Наша модель как раз обычная модель AR, а фасад - да, не фасад
Как раз понятия "модель AR" нет. Так что и модель не модель.
O
Ну такое) слаг не относится не каким образом к валидатору
а зачем он в реквесте тогда?
AH
Вообще это красиво и правильно
Как бы не так))
1. Док-блок с 13-й по 30-ю строки можно вынести во внешний файл, автоматом сгенерированный через
2. 34-я строка не по PSR-12;
3. Метод
4. Методу
5. Метод
6.
7. Методы
8.
В общем, это яркий пример как НЕ НАДО делать. Ни в коем случае нельзя мешать МОДЕЛЬ с БИЗНЕС-ЛОГИКОЙ, для реализации которой существуют сервисы.
1. Док-блок с 13-й по 30-ю строки можно вынести во внешний файл, автоматом сгенерированный через
ide-helper командой php artisan ide-helper:models -R -N;2. 34-я строка не по PSR-12;
3. Метод
rolesList можно в конфиг вынести;4. Методу
register вообще нечего делать в модели - это задача сервиса. Еще б валидатор туда впихнул...;5. Метод
registerByNetwork та же песня. Вдобавок, с какого перепуга модель юзера создает записи для релейшена?6.
new...7. Методы
isWait, isActive, isModerator, isAdmin, isPhoneVerified и isPhoneAuthEnabled можно превратить в атрибуты. В таком виде они могут быть извлечены вместе с моделью там, где это необходимо. Например, в SPA;8.
verify, changeRole, changeRole, unverifyPhone, requestPhoneVerification, verifyPhone, enablePhoneAuth, disablePhoneAuth, addToFavorites, removeFromFavorites, - невооруженным глазом видны участок бизнес-логики... В МОДЕЛИ...;В общем, это яркий пример как НЕ НАДО делать. Ни в коем случае нельзя мешать МОДЕЛЬ с БИЗНЕС-ЛОГИКОЙ, для реализации которой существуют сервисы.
ML
Который раз становлюсь наблюдателем того что в группе Новичков не совсем по уровню тему обсуждаются
ML
причем те же люди....
RK
а зачем он в реквесте тогда?
Да потому что он код не нормально оформил , там вообще по другому надо сделать, кароч забей это уже обсудили)
O
Да потому что он код не нормально оформил , там вообще по другому надо сделать, кароч забей это уже обсудили)
ок)
ML
Заходишь такой и тебе в лицо - "Фасады это не фасады!"
ИЛ
Как бы не так))
1. Док-блок с 13-й по 30-ю строки можно вынести во внешний файл, автоматом сгенерированный через
2. 34-я строка не по PSR-12;
3. Метод
4. Методу
5. Метод
6.
7. Методы
8.
В общем, это яркий пример как НЕ НАДО делать. Ни в коем случае нельзя мешать МОДЕЛЬ с БИЗНЕС-ЛОГИКОЙ, для реализации которой существуют сервисы.
1. Док-блок с 13-й по 30-ю строки можно вынести во внешний файл, автоматом сгенерированный через
ide-helper командой php artisan ide-helper:models -R -N;2. 34-я строка не по PSR-12;
3. Метод
rolesList можно в конфиг вынести;4. Методу
register вообще нечего делать в модели - это задача сервиса. Еще б валидатор туда впихнул...;5. Метод
registerByNetwork та же песня. Вдобавок, с какого перепуга модель юзера создает записи для релейшена?6.
new...7. Методы
isWait, isActive, isModerator, isAdmin, isPhoneVerified и isPhoneAuthEnabled можно превратить в атрибуты. В таком виде они могут быть извлечены вместе с моделью там, где это необходимо. Например, в SPA;8.
verify, changeRole, changeRole, unverifyPhone, requestPhoneVerification, verifyPhone, enablePhoneAuth, disablePhoneAuth, addToFavorites, removeFromFavorites, - невооруженным глазом видны участок бизнес-логики... В МОДЕЛИ...;В общем, это яркий пример как НЕ НАДО делать. Ни в коем случае нельзя мешать МОДЕЛЬ с БИЗНЕС-ЛОГИКОЙ, для реализации которой существуют сервисы.
Даже так, ещё лучше
ИЛ
Заходишь такой и тебе в лицо - "Фасады это не фасады!"
Так наши фасады != паттерн "Фасад"