Infinitypaul\NairaExchangeRates\Response

Complexity

Total Complexity

5

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
A	getStatusCode()	0	4	1
A	getTimestamp()	0	4	1
A	getData()	0	4	1
A	toJSON()	0	8	1

<?php


namespace Infinitypaul\NairaExchangeRates;


class Response
{
    # The actual Guzzle response:
    private $response;

    # Core response:
    private $headers;
    private $bodyRaw;
    private $body;

    # Properties:
    private $statusCode;
    private $timestamp;

    function __construct( \GuzzleHttp\Psr7\Response $response = null )

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->response = $response;
        $this->headers    = $response->getHeaders();

It seems like $response is not always an object, but can also be of type null. Maybe add an additional type check?

        $this->bodyRaw    = (string) $response->getBody();
        $this->body       = json_decode( $this->bodyRaw );

        # Set our properties:
        $this->statusCode   = $response->getStatusCode();
        $this->timestamp    = date('c');
    }

    /****************************/
    /*                          */
    /*         GETTERS          */
    /*                          */
    /****************************/

    # Get the status code:
    public function getStatusCode()
    {
        return (int) $this->statusCode;
    }

    #ÊGet the timestamp of the request:
    public function getTimestamp()
    {
        return $this->timestamp;
    }

    #ÊGet the timestamp of the request:
    public function getData()
    {
        return $this->body;
    }

    #ÊConvert the response to JSON:
    public function toJSON()
    {
        return json_encode([
            'statusCode'   => $this->getStatusCode(),
            'timestamp'    => $this->getTimestamp(),
            'body' => $this->getData()
        ]);
    }
}