Geocoder\Provider\Cache\ProviderCache

Complexity

Total Complexity

8

Size/Duplication

Total Lines	87
Duplicated Lines	27.59 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	geocodeQuery()	12	12	2
A	reverseQuery()	12	12	2
A	getName()	0	4	1
A	__call()	0	4	1
A	getCacheKey()	0	5	1

<?php

declare(strict_types=1);

/*
 * This file is part of the Geocoder package.
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @license    MIT License
 */

namespace Geocoder\Provider\Cache;

use Geocoder\Collection;
use Geocoder\Query\GeocodeQuery;
use Geocoder\Query\ReverseQuery;
use Geocoder\Provider\Provider;
use Psr\SimpleCache\CacheInterface;

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

    /**
     * @var CacheInterface
     */
    protected $cache;

    /**
     * How log a result is going to be cached.
     *
     * @var int|null
     */
    protected $lifetime;

    /**
     * @param Provider       $realProvider
     * @param CacheInterface $cache
     * @param int            $lifetime
     */
    final public function __construct(Provider $realProvider, CacheInterface $cache, int $lifetime = null)

Unnecessary FINAL modifier in FINAL class

    {
        $this->realProvider = $realProvider;
        $this->cache = $cache;
        $this->lifetime = $lifetime;
    }

    /**
     * {@inheritdoc}
     */
    final public function geocodeQuery(GeocodeQuery $query): Collection

This method seems to be duplicated in your project.

Unnecessary FINAL modifier in FINAL class

    {
        $cacheKey = $this->getCacheKey($query);
        if (null !== $result = $this->cache->get($cacheKey)) {
            return $result;
        }

        $result = $this->realProvider->geocodeQuery($query);
        $this->cache->set($cacheKey, $result, $this->lifetime);

        return $result;
    }

    /**
     * {@inheritdoc}
     */
    final public function reverseQuery(ReverseQuery $query): Collection

This method seems to be duplicated in your project.

Unnecessary FINAL modifier in FINAL class

    {
        $cacheKey = $this->getCacheKey($query);
        if (null !== $result = $this->cache->get($cacheKey)) {
            return $result;
        }

        $result = $this->realProvider->reverseQuery($query);
        $this->cache->set($cacheKey, $result, $this->lifetime);

        return $result;
    }

    /**
     * {@inheritdoc}
     */
    public function getName(): string
    {
        return sprintf('%s (cache)', $this->realProvider->getName());
    }

    final public function __call($method, $args)

Unnecessary FINAL modifier in FINAL class

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

    /**
     * @param GeocodeQuery|ReverseQuery $query
     *
     * @return string
     */
    protected function getCacheKey($query): string
    {
        // Include the major version number of the geocoder to avoid issues unserializing.
        return 'v4'.sha1((string) $query);
    }
}