Блог

Я осуществляю платежи для своего веб-сайта, используя API внешней службы (т.Е. служба поставщика платежей).

Допустим, пользователь нажимает «КУПИТЬ», а затем мы переходим к моему контроллеру, который говорит что-то вроде:

 public function buyFunction() {

    $result = $this->ExternalService->pay();

    if ($result->success == true) {
        return 'We are happy';
    }
}
 

Я также создал вышеупомянутый externalService , который имеет pay() метод:

 class ExternalService {

    public function pay() {
        response = //Do stuff with Guzzle to call the API to make the payment

        return response;
    }
}
 

Теперь иногда что-то идет не так.
Допустим, API возвращает ошибку, что означает, что он выдает исключение GuzzleException — как мне с этим справиться?

В идеале, если есть ошибка, я хотел бы зарегистрировать ее и перенаправить пользователя на страницу и сообщить ему, что что-то пошло не так.

Что я пробовал

1. Я пытался использовать оператор try / catch в функции pay() и использовать abort(500) , но это не позволяет мне перенаправлять на нужную страницу.
2. Я пытался использовать оператор try / catch в функции pay() и использовать return redirect('/mypage') , но это просто возвращает объект перенаправления контроллеру, который затем завершается с ошибкой при попытке вызова result->success
3. Я попытался использовать номер 2, но также добавил блок try / catch в метод контроллера, но ничего не изменилось.

В конце концов, я нашел два решения. В обоих случаях я использую блок try / catch внутри метода pay(). Затем я либо return 0; проверяю контроллер if (result == 0) , либо использую abort( redirect('/mypage') ); внутри блока try / catch метода pay().

Как правильно с этим справиться? Как использовать блоки try / catch?

По моему опыту, избегайте обработки исключений, пропускайте их и обрабатывайте их соответствующим образом с помощью try catches. Это наиболее прагматичный подход. В качестве альтернативы вы в конечном итоге проверите правильность результата в странных местах, например. if ($result) {...} . Просто предположим, что все прошло хорошо, за исключением случаев, когда генерируется исключение. Бонус: никогда не делайте перехват покемонов Exception $e , если вам это специально не нужно!

 class ExternalService {

    public function pay() {
        try {
            response = $client->get(...);
        } catch (BadResponseException $exception) {
            Log::warning('This should not happen check payment api: ' . $exception->getMessage());
            
            throw new PaymentException('Payment did not go through');
        } 

        return response;
    }
}
 

Предполагая, что у вас есть собственное исключение.

 class PaymentException extends HttpException
{
    public function __construct(?Exception $previous = null)
    {
        parent::__construct(Response::HTTP_BAD_REQUEST, 'Unexpected error processing the payment', $previous);
    }
}
 

Это позволяет обрабатывать поток в контроллере, где имело бы смысл обрабатывать перенаправление. Иногда, если исключение является очень неотъемлемым или общим для веб-приложения, оно также может обрабатываться обработчиком исключений.

 class PaymentController {
    public function pay(PaymentService $service) {
        try {
            $payment = $service->buyFunction();
        } catch (PaymentException $exception) {
            return redirect()->route('app.payment.error');
        } 
        
        return view('app.payment.success', compact('payment'));
    }
}