Tarantool\Mapper\Client

Complexity

Total Complexity

13

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	connect()	0	6	1
A	disableRequest()	0	7	2
A	resetDisabled()	0	4	1
A	sendRequest()	0	10	2
A	log()	0	6	2
A	getTimeSummary()	0	8	2
A	setLogging()	0	4	1
A	getLog()	0	4	1
A	resetLog()	0	4	1

<?php

namespace Tarantool\Mapper;

use Exception;
use Tarantool\Client\Connection\Connection;
use Tarantool\Client\Client as TarantoolClient;
use Tarantool\Client\Request\Request;

class Client extends TarantoolClient
{
    private $logging = false;
    private $log = [];

    private $disabledRequests = [];

    public function connect()
    {
        $start = microtime(1);
        parent::connect();
        $this->log($start, Connection::class);
    }

    public function disableRequest($class)
    {
        if (!is_subclass_of($class, Request::class)) {

Due to PHP Bug #53727, is_subclass_of might return inconsistent results on some PHP versions if \Tarantool\Client\Request\Request::class can be an interface. If so, you could instead use ReflectionClass::implementsInterface.

            throw new Exception("Invalid request type: " . $class);
        }
        $this->disabledRequests[] = $class;
    }

    public function resetDisabled()
    {
        $this->disabledRequests = [];
    }

    public function sendRequest(Request $request)
    {
        if (in_array(get_class($request), $this->disabledRequests)) {
            throw new Exception(get_class($request) . ' is disabled');
        }
        $start = microtime(1);
        $response = parent::sendRequest($request);
        $this->log($start, get_class($request), $request->getBody(), $response->getData());

It seems like $response->getData() targeting Tarantool\Client\Response::getData() can also be of type null; however, Tarantool\Mapper\Client::log() does only seem to accept array, maybe add an additional type check?

        return $response;
    }

    private function log($start, $class, $request = [], $response = [])
    {
        if ($this->logging) {
            $this->log[] = [microtime(1) - $start, $class, $request, $response];
        }
    }

    public function getTimeSummary()
    {
        $summary = 0;
        foreach ($this->log as $request) {
            $summary += $request[0];
        }
        return $summary;
    }

    public function setLogging($logging)
    {
        $this->logging = $logging;
    }

    public function getLog()
    {
        return $this->log;
    }

    public function resetLog()
    {
        $this->log = [];
    }
}