Geo\Service\AbstractClient

Complexity

Total Complexity

11

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	1
A	setupClient()	0	4	1
A	preQuery()	0	5	1
B	query()	0	24	5
A	queryOne()	0	7	2
A	processResult()	0	4	1

<?php
/**
 * YAWIK
 *
 * @filesource
 * @license MIT
 * @copyright  2013 - 2017 Cross Solution <http://cross-solution.de>
 */
  
/** */
namespace Geo\Service;

use Zend\Http\Client;
use Zend\Json\Json;

/**
 * ${CARET}
 * 
 * @author Mathias Gelhausen <gelhausen@cross-solution.de>
 * @todo write test 
 */
class AbstractClient
{
    /**
     *
     *
     * @var \Zend\Http\Client
     */
    protected $client;

    /**
     *
     *
     * @var \Zend\Cache\Storage\Adapter\AbstractAdapter
     */
    protected $cache;


    public function __construct($uri, $cache = false)
    {
        $this->client = $this->setupClient($uri);
        $this->cache = $cache;

It seems like $cache of type boolean is incompatible with the declared type object<Zend\Cache\Storage\Adapter\AbstractAdapter> of property $cache.

//        $namespace = explode('\\', get_class($this));

49% of this comment could be valid code. Did you maybe forget this after debugging?

//        $namespace = array_pop($namespace);
//        //$this->cache->setOptions(['namespace' => $namespace]);
    }

    protected function setupClient($uri)
    {
        return new Client($uri);
    }

    protected function preQuery($term, array $params)
    {
        $params['q'] = $term;
        $this->client->setParameterGet($params);
    }

    public function query($term, array $params = [])
    {
        $cacheId = md5($term);

        if ($this->cache && ($result = $this->cache->getItem($cacheId))) {
            return $result;
        }

        $this->preQuery($term, $params);

        $response = $this->client->send();
        if ($response->getStatusCode() !== 200) {
            throw new \RuntimeException('Query failed, because ' . $response->getReasonPhrase());
        }

        $result = $response->getBody();
        $result = $this->processResult($result);

        if ($this->cache) {
            $this->cache->setItem($cacheId, $result);
        }

        return $result;
    }

    public function queryOne($term)
    {
        $result = $this->query($term);
        $result = Json::decode($result, Json::TYPE_ARRAY);

        return isset($result[0]) ? $result[0] : false;
    }

    protected function processResult($result)
    {
        return $result;
    }

}