а написать обработчик, понятный клиенту?
нуок)) вот контроллер: publi function makePayment(Data $data): Response { return $this->paymentManager->makePayment($data); } ты предлагаешь чтобы метод созадния платежа возвращал тексты ошибок?
да, это очеивдно будет
а теперь представь немного усложненную логику. контроллер вызывает сервис, который вызывает сервис, который вызывает сервис, который вызывает метод сервиса в котором какая-то сущность не найдена. получается, все твои сервисы должны будут возвращать в результате строку, которая должна дойти до контроллера и тогда он ее покажет клиенту?
и еще одна проблема - $this->paymentManager->makePayment($data) должен вернуть Payment, а он вдруг будет строки с ошибками возвращать( где очевидность?
сложна.... надо делать проще, на каждом сервисе своя ответсвенность и свои логи, что сломалось. идем в логи и через 1 минуту всё ясно
нет, он вернуть должен ?Payment
это понятно, что в логах ты все поймешь, а что с читаемостью кода?
нет, он же должен отдать контроллеру строку с ошибкой, как ты ранее говорил
обработать на месте логику, а не тащить исключение через сервис-сервис-сервис
Сильно зависит от бизнес логики. Думаю тут лучше вернуть статус объект (где внутри платежка) или кастомное исключение
обработать нормально через хендлер какой-нить и будет нормально
для метода makePayment вернуть нулл вместо результата - это пиздец как стремно
В хэндлере отловить исключение и потом вернуть 200ую?
да согласен, просто в упрощенных сущностях примеры привожу
я не говорил именно про код, можно и 40* но с номральным текстом, что "чувак у тебя ИД юзера корявой дай нормальный", кто как договорился
Обсуждают сегодня