Size: a a a

2020 March 05

F

Fingerprint in Laravel Pro
Oybek Sayfullayev
думаю это не настоящий МВС . Модель только нужно занимация с базами, валидация это не его  работа. Валидация для контроллера.
Валидация для Request
источник

DK

Denis 🕸 Khomusyak in Laravel Pro
нужно больше ифов
источник

OS

Oybek Sayfullayev in Laravel Pro
Fingerprint
Валидация для Request
да понятно извиняюсь, я тоже новичок
источник

DK

Denis 🕸 Khomusyak in Laravel Pro
по логике кода, это как трейт функция, у модели может быть поле хранящее файл разного названия
источник

ВБ

Виктор Бурцев... in Laravel Pro
Egor Gruzdev
Пусть попробует на это метод написать тест, и все встанет на свои места
А что помешает на этот код тесты набросать?)
источник

AZ

Alexander Zaplitniy in Laravel Pro
Вот написал некое объяснение почему его подход плох. Скажите своем мнение.

Итак мы имеем модель InternalFile(лист бумаги) что она умеет в нашем варианте: - модель умеет себя сохранить в нужное ей место (лист бумаги решает где ему храниться лучше в папке или в тумбочке под столом); - модель знает как получить свой файлик (лист бумаги кричит человеку я в тумбочке спаси меня) - Модель еще и права проверяет(лист бумаги спрашивает слушай если ты Сережа то я тебя покажу свой текст, а если какой-то Вася то я не разрешу тебе себя читать.)  В моей голове текущий подход к написанию логики как-то не складывается…
источник

AZ

Alexander Zaplitniy in Laravel Pro
норм или фигня?
источник

s

sheykvit in Laravel Pro
Egor Gruzdev
Фасады это зло!!! Но иногда без них не куда
Почему фасады это зло?
источник

EG

Egor Gruzdev in Laravel Pro
sheykvit
Почему фасады это зло?
Зло, т.к. их используют и к мету и не месту, также фасад == global с точки зрения вызова, а также формирование неявной зависимости от класса, который завернут в фасад
источник

AP

Alexander Pavlenko 🌚... in Laravel Pro
ну в контексте лары
источник

AP

Alexander Pavlenko 🌚... in Laravel Pro
ларавел сама неявная зависимость)))
источник

M

Michael in Laravel Pro
Не понимаю ненависти к фасадам..
источник

AP

Alexander Pavlenko 🌚... in Laravel Pro
потому хз, фасад как паттерн иногда полезен, а вконтексте самой лары то это вполне норма
источник

AP

Alexander Pavlenko 🌚... in Laravel Pro
главное не увлекаться
источник

AZ

Alexander Zaplitniy in Laravel Pro
Alexander Pavlenko 🌚
главное не увлекаться
Самим laravel 😂
источник

M

Michael in Laravel Pro
Окей, делать свои фасады и юзать их - наверное не ок. Просто, ну, зачем, если во фреймворке есть DI. Но почему нельзя юзать встроенные ларавеловские фасады ?
источник

EG

Egor Gruzdev in Laravel Pro
Michael
Окей, делать свои фасады и юзать их - наверное не ок. Просто, ну, зачем, если во фреймворке есть DI. Но почему нельзя юзать встроенные ларавеловские фасады ?
Потому что, все что обернуто в фасады, можно также подтянуть через DI
источник

s

sheykvit in Laravel Pro
Michael
Окей, делать свои фасады и юзать их - наверное не ок. Просто, ну, зачем, если во фреймворке есть DI. Но почему нельзя юзать встроенные ларавеловские фасады ?
Почему нельзя? Можно. Сама Лара рекомендует.
источник

EG

Egor Gruzdev in Laravel Pro
sheykvit
Почему нельзя? Можно. Сама Лара рекомендует.
Лара использует фасады не дальше контролера (то что я видел в доках), все сервис классы должны  получать зависимость явно и через DI, а приведенный выше пример их использует не к месту.
источник

M

Michael in Laravel Pro
А зачем лучше подключать обёрнутое в фасады при помощи DI ? Мы же не собираемся тестировать это..
источник