R3bers\BittrexApi\BittrexClient

Complexity

Total Complexity

19

Size/Duplication

Total Lines	180
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

100%

Importance

Changes

0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getCurrentMinuteCount()	0	4	1
A	setCredential()	0	5	1
A	public()	0	6	1
A	countAPI()	0	11	2
A	getPublicClient()	0	4	2
A	createPublicClient()	0	11	1
A	market()	0	5	1
A	getPrivateClient()	0	4	2
A	createPrivateClient()	0	18	3
A	getKey()	0	4	1
A	getSecret()	0	4	1
A	account()	0	5	1
A	getPrevMinuteCount()	0	4	1

<?php

declare(strict_types=1);

namespace R3bers\BittrexApi;

use GuzzleHttp\Client;
use GuzzleHttp\HandlerStack;
use R3bers\BittrexApi\Api\Account;
use R3bers\BittrexApi\Api\Market;
use R3bers\BittrexApi\Api\PublicApi;
use R3bers\BittrexApi\Exception\InvalidCredentialException;
use R3bers\BittrexApi\Middleware\Authentication;

class BittrexClient
{
    /**
     * Main URL to Bittrex Exchange
     */
    private const BASE_URI = 'https://api.bittrex.com';

    /** @var Client */
    private $publicClient;

    /** @var Client */
    private $privateClient;

    /** @var string */
    private $key = '';

    /** @var string */
    private $secret = '';

    /** @var string */
    private $currentMinuteCount = 0;
    /**
     * @var int
     */
    private $lastAPITime;
    /**
     * @var int
     */
    private $prevMinuteCount = 0;

    /**
     * BittrexClient constructor.
     */
    public function __construct()
    {
        $this->lastAPITime = time();
    }

    /**
     * @return int
     */
    public function getCurrentMinuteCount(): int
    {
        return $this->currentMinuteCount;
    }

    /**
     * @param string $key
     * @param string $secret
     */
    public function setCredential(string $key, string $secret): void
    {
        $this->key = $key;
        $this->secret = $secret;
    }

    /**
     * @return PublicApi
     */
    public function public(): PublicApi
    {
        $this->countAPI();
        return new PublicApi($this->getPublicClient());

    }

    /**
     * Counting in minute API
     */
    private function countAPI(): void
    {
        $currentTime = time();
        if (intdiv($this->lastAPITime, 60) < intdiv($currentTime, 60)) {
            $this->prevMinuteCount = $this->currentMinuteCount;

The property $prevMinuteCount was declared of type integer, but $this->currentMinuteCount is of type string. Maybe add a type cast?

            $this->currentMinuteCount = 1;

The property $currentMinuteCount was declared of type string, but 1 is of type integer. Maybe add a type cast?

        } else {
            $this->currentMinuteCount++;
            $this->lastAPITime = $currentTime;
        }
    }

    /**
     * @return Client
     */
    private function getPublicClient(): Client
    {
        return $this->publicClient ?: $this->createPublicClient();
    }

    /**
     * @return Client
     */
    private function createPublicClient(): Client
    {
        return new Client([
            'headers' => [
                'User-Agent' => 'r3bers/bittrex-api/1.3',
                'Accept' => 'application/json',
                'Content-Type' => 'application/json'
            ],
            'base_uri' => self::BASE_URI
        ]);
    }

    /**
     * @return Market
     * @throws InvalidCredentialException
     */
    public function market(): Market
    {
        $this->countAPI();
        return new Market($this->getPrivateClient());
    }

    /**
     * @return Client
     * @throws InvalidCredentialException
     */
    private function getPrivateClient(): Client
    {
        return $this->privateClient ?: $this->createPrivateClient();
    }

    /**
     * @return Client
     * @throws InvalidCredentialException
     */
    private function createPrivateClient(): Client
    {
        if (empty($this->key) || empty($this->secret)) {
            throw new InvalidCredentialException('Key and secret must be set for authenticated API');
        }
        $stack = HandlerStack::create();
        $stack->push(new Authentication($this->getKey(), $this->getSecret()));

        return new Client([
            'headers' => [
                'User-Agent' => 'r3bers/bittrex-api/1.3',
                'Accept' => 'application/json',
                'Content-Type' => 'application/json'
            ],
            'handler' => $stack,
            'base_uri' => self::BASE_URI
        ]);
    }

    /**
     * @return string
     */
    public function getKey(): string
    {
        return $this->key;
    }

    /**
     * @return string
     */
    public function getSecret(): string
    {
        return $this->secret;
    }

    /**
     * @return Account
     * @throws InvalidCredentialException
     */
    public function account(): Account
    {
        $this->countAPI();
        return new Account($this->getPrivateClient());
    }

    /**
     * @return int
     */
    public function getPrevMinuteCount(): int
    {
        return $this->prevMinuteCount;
    }
}