Size: a a a

2020 November 27

MW

Maxyc Webber in Laravel UA
Так ты проверяй это до создания обьекта
источник

РИ

Роман Игумнов... in Laravel UA
Т.е в контроллере?
источник

MW

Maxyc Webber in Laravel UA
Вообще это возможно холиварный вопрос. Но я считаю, что до создания объекта это надо делать
источник

MW

Maxyc Webber in Laravel UA
Роман Игумнов
Т.е в контроллере?
Ты ж объект вроде в апп сервис провайдере создаешь
источник

РИ

Роман Игумнов... in Laravel UA
точно. Сори
источник

MW

Maxyc Webber in Laravel UA
А вот контекст, который приходит уже от пользователя в контроллере проверяй и отдавай сервису если все ок
источник

РИ

Роман Игумнов... in Laravel UA
Итак. Вот что у меня вышло в сервис провайдере. Возмонжо кому-то будет полезно
источник

РИ

Роман Игумнов... in Laravel UA
public function register()
   {
       $this->app->bind(PayPartsInterface::class, function() {

           if(empty(config('payparts.store_id'))){
               throw new \InvalidArgumentException('store_id is  empty');
           }
           if(empty(config('payparts.password'))){
               throw new \InvalidArgumentException('password is empty');
           }

           $service = new PayPartsService(
               config('payparts.store_id'),
               config('payparts.password'),
           );

           return $service;
       });
   }
источник

MR

Maxim Rebitskiy in Laravel UA
Роман Игумнов
Ну а вдруг кто-то завтра случайно сотрет ключ из env
Я бы проверял такие данные в классе который и требует данные. Так же я могу передать туда и просто мануально пустые данные.
источник

РИ

Роман Игумнов... in Laravel UA
Maxim Rebitskiy
Я бы проверял такие данные в классе который и требует данные. Так же я могу передать туда и просто мануально пустые данные.
А зачем?
источник

РИ

Роман Игумнов... in Laravel UA
Не вижу смысла
источник

MW

Maxyc Webber in Laravel UA
Ну вот да, тут холивар)
источник

РИ

Роман Игумнов... in Laravel UA
Может стоит уже готовые данные передавать
источник

MW

Maxyc Webber in Laravel UA
С опытом поймается как правильней
источник

MR

Maxim Rebitskiy in Laravel UA
Зачем что именно?)
источник

РИ

Роман Игумнов... in Laravel UA
Ты получается уже на 100% доверяешь данным из сервис провайдера
источник

РИ

Роман Игумнов... in Laravel UA
Maxim Rebitskiy
Зачем что именно?)
Проверку делать в классе который требует данные. Например если я верно понял ты про сервис класс
источник

MR

Maxim Rebitskiy in Laravel UA
Роман Игумнов
Ты получается уже на 100% доверяешь данным из сервис провайдера
А если не через сервис провайдер которому доверяю передаю?
источник

РИ

Роман Игумнов... in Laravel UA
Maxim Rebitskiy
А если не через сервис провайдер которому доверяю передаю?
Ну в данном контексте мы про сервис провайдер говорим
источник

MR

Maxim Rebitskiy in Laravel UA
Роман Игумнов
Ну в данном контексте мы про сервис провайдер говорим
Но мы же рассуждаем как правильно, и как может быть. А не как есть
источник