Tarantool\Mapper\Client

Complexity

Total Complexity

8

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

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

<?php

namespace Tarantool\Mapper;

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 = [];

    public function connect()
    {
        $start = microtime(1);
        $result = parent::connect();

Are you sure the assignment to $result is correct as parent::connect() (which targets Tarantool\Client\Client::connect()) seems to always return null.

        $this->log($start, Connection::class);
        return $result;
    }

    public function sendRequest(Request $request)
    {
        $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;
    }

}