Bazinga\Bundle\GeocoderBundle\DataCollector\ProfilingProvider

Complexity

Total Complexity

9

Size/Duplication

Total Lines	90
Duplicated Lines	28.89 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	geocodeQuery()	13	13	1
A	reverseQuery()	13	13	1
A	logQuery()	0	18	3
A	getQueries()	0	4	1
A	__call()	0	4	1
A	getName()	0	4	1

<?php

namespace Bazinga\Bundle\GeocoderBundle\DataCollector;

use Geocoder\Collection;
use Geocoder\Exception\LogicException;
use Geocoder\Location;
use Geocoder\Provider\Provider;
use Geocoder\Query\GeocodeQuery;
use Geocoder\Query\ReverseQuery;

/**
 * @author Tobias Nyholm <tobias.nyholm@gmail.com>
 */
class ProfilingProvider implements Provider
{
    /**
     * @var Provider
     */
    private $realProvider;

    /**
     * @var array
     */
    private $queries = [];

    /**
     * @param Provider       $realProvider
     */
    public function __construct(Provider $realProvider)
    {
        $this->realProvider = $realProvider;
    }

    public function geocodeQuery(GeocodeQuery $query): Collection

This method seems to be duplicated in your project.

    {
        $startTime = microtime(true);
        try {
            $result = $this->realProvider->geocodeQuery($query);
        } finally {
            $duration = (microtime(true) - $startTime) * 1000;

            $this->logQuery($query, $duration, $result);
        }

        return $result;
    }

    public function reverseQuery(ReverseQuery $query): Collection

This method seems to be duplicated in your project.

    {
        $startTime = microtime(true);
        try {
            $result = $this->realProvider->reverseQuery($query);
        } finally {
            $duration = (microtime(true) - $startTime) * 1000;

            $this->logQuery($query, $duration, $result);
        }

        return $result;
    }

    /**
     * @param GeocodeQuery|ReverseQuery     $query
     * @param float      $duration      geocoding duration
     * @param Collection $result
     */
    private function logQuery($query, float $duration, Collection $result = null)
    {
        if ($query instanceof GeocodeQuery) {
            $queryString = $query->getText();
        } elseif ($query instanceof ReverseQuery) {
            $queryString = sprintf('(%s, %s)', $query->getCoordinates()->getLongitude(),  $query->getCoordinates()->getLongitude());
        } else {
            throw new LogicException('First parameter to ProfilingProvider::logQuery must be a query');
        }

        $this->queries[] = array(
            'query' => $query,
            'queryString' => $queryString,
            'duration' => $duration,
            'providerName' => $this->getName(),
            'result' => $result,
        );
    }

    /**
     * @return array
     */
    public function getQueries(): array
    {
        return $this->queries;
    }

    public function __call($method, $args)
    {
        return call_user_func_array([$this->realProvider, $method], $args);
    }

    public function getName(): string
    {
        return $this->realProvider->getName();
    }
}