Как лучше реализовать sender?

Question

[kaxa3201]

Написал вот такой класс

class JsonRpcClient implements ClientContract
{
    private $client;
    public function __construct(ClientInterface $client)
    {
        $this->client = $client;
    }
    public function sendRequest($url, $methodName, $params, $headers)
    {
        try {
            $response = $this->client->post($url, [
                RequestOptions::JSON => [
                    'jsonrpc' => '2.0',
                    'method'  => $methodName,
                    'params'  => ($params),
                    'id'      => Uuid::uuid4(),
                ],

                RequestOptions::HEADERS => $headers,
            ]);

            $responseData = json_decode($response->getBody()->getContents(), true);

            if (empty($responseData['result'])) {
                return null;
            }

            return $responseData['result'];

        } catch (ClientException $exception) {
            return $exception->getMessage();
        }
    }

Меня смущает rerunt null, правильно ли так отлавливать пустой ответ или это нужно отлавливать уже в клиентском коде? Вообщем, как лучше сделать? буду благодарен за разумный совет

Answer 1

[Denis]

Как вариант бросьте эксепшен (лучше создать свой).
А в клиентском коде (или лучше создайте декоратор) и перенесите туда try...catch уже ловите его и обрабатывайте как надо.
ну если лень заморачиваться или оставьте как есть и в return $exception->getMessage(); вернется сообщение пустого респонса

Comments

[kaxa3201]

Denis вот тут допустим уберу трай кейтч, и напишу
if (isset($responseData['error'])) {
throw new Exceptions(тут сообщение об ошибке) }
стоит ли мне тут так писать об ошибке?или можно и нужно конкретную ошибку поймать? Правильно ли так будет?

[Denis]

SendRequest должен вернуть либо результат либо ошибку почему этот результат не получен. Тип ошибки обозначен соответсвующим классом exception и соответвующим сообщением а далее вы обрабатываете результат или в catch исключение

[kaxa3201]

Denis, подскажите какое сообщение тут будет уместно? client error ? или ещё как то?

[kaxa3201]

Denis ,

public function sendRequest($url, $methodName, $params, $headers)
    {
        $response = $this->client->post($url, [
            RequestOptions::JSON => [
                'jsonrpc' => '2.0',
                'method'  => $methodName,
                'params'  => $params,
                'id'      => Uuid::uuid4(),
            ],

            RequestOptions::HEADERS => $headers,
        ]);
        $responseData = json_decode($response->getBody()->getContents(), true);

        if (isset($responseData['error'])) {
            throw new \Exception('Json-rpc client error');
        }

        return $responseData;
    }

так нормально будет?

[Denis]

Да, но тут нюанс, что Вы теряете оригинальное сообщение об ошибке хранящееся где то $responseData['error'], и когда вы будет дебажить логи для отладки приложения у Вам будет доступно Json-rpc client error вместо реальной причины отказа API
если Вы хотите для конечного пользователя замаскировать ошибку (может быть потому что там содержаться системные данные которые лучше не публиковать), то необходимо в Вашу систему прокидывать логгер в который будет помещаться оригинальные респонсы ошибок и тут два варианта
1) Если не хотите заморачиваться. Вернуть try ... catch в этот код и прокинуть туда логгер и сделать

<?php
try {
            $response = $this->client->post($url, [
                RequestOptions::JSON => [
                    'jsonrpc' => '2.0',
                    'method'  => $methodName,
                    'params'  => ($params),
                    'id'      => Uuid::uuid4(),
                ],

                RequestOptions::HEADERS => $headers,
            ]);

            $responseData = json_decode($response->getBody()->getContents(), true);

            if (isset($responseData['error'])) {
                 throw new \Exception($responseData['error']['message']);
            }

            return $responseData['result'];

        } catch ( \Exception $exception) {
            $logger->error($exception->getMessage());
            throw new ClientException('Json-rpc client error');
        }

2) более правильный но немного замороченней: оставить Ваш последний вариант
только вместо

<?php
throw new \Exception('Json-rpc client error');

напишите

<?php
throw new \ClientException($responseData['error']['message']);

и оббернуть его (см паттерн Декоратор) другим классом и в этот класс прокинуть логгер

[kaxa3201]

Denis, а ClientException это guzzle ? Почему не оставить просто Exceptions? Спасибо вам огромное)

[Denis]

ClientException нет, в данном случаее я имел ввиду Ваш собственный класс (можно обычный класс отнаследованный не важно Exception, RuntimeException, DomainException и т. д.) с помощью которого Вы в дальнейшем разделять серверные ошибки api от Ваших собственных (например не валидные данные) и ошибки Вашего модуля и ошибки приложения в целом