mrcnpdlk\Xmdb\Omdb

Complexity

Total Complexity

11

Size/Duplication

Total Lines	125
Duplicated Lines	57.6 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
B	getByImdbId()	36	36	5
B	getByTitle()	36	36	5

<?php
/**
 * Created by Marcin.
 * Date: 09.12.2017
 * Time: 22:46
 */

namespace mrcnpdlk\Xmdb;


use Campo\UserAgent;
use Curl\Curl;
use HttpLib\Http;
use mrcnpdlk\Xmdb\Model\Omdb\Title;

class Omdb
{
    /**
     * @var \mrcnpdlk\Xmdb\Client
     */
    private $oClient;
    /**
     * @var \Psr\Log\LoggerInterface
     */
    private $oLog;
    /**
     * @var \mrcnpdlk\Psr16Cache\Adapter
     */
    private $oAdapter;
    /**
     * @var string
     */
    private $sToken;
    /**
     * @var string
     */
    private $url;

    /**
     * Omdb constructor.
     *
     * @param \mrcnpdlk\Xmdb\Client $oClient
     *
     * @throws \mrcnpdlk\Xmdb\Exception
     */
    public function __construct(Client $oClient)
    {
        $this->oClient  = $oClient;
        $this->oLog     = $oClient->getLogger();
        $this->oAdapter = $oClient->getAdapter();
        $this->sToken   = $oClient->getOmdbToken();
        $this->url      = 'http://www.omdbapi.com/?r=json&apikey=' . $oClient->getOmdbToken();
    }

    /**
     * @param string $imdbId
     *
     * @return Title
     * @throws \mrcnpdlk\Xmdb\Exception
     */
    public function getByImdbId(string $imdbId)

This method seems to be duplicated in your project.

    {
        try {
            $oResp = $this->oClient->getAdapter()->useCache(
                function () use ($imdbId) {
                    $oCurl = new Curl();
                    $oCurl->setUserAgent(UserAgent::random());
                    $oCurl->setHeader('Accept-Language', $this->oClient->getLang());
                    $params = [
                        'i'    => $imdbId,
                        'plot' => 'full',
                        'r'    => 'json',
                    ];
                    $oCurl->get($this->url . '&' . http_build_query($params));

                    if ($oCurl->error) {
                        throw new \RuntimeException('Curl Error! ' . ($oCurl->httpStatusCode ? Http::message($oCurl->httpStatusCode) : 'Unknown code'),
                            $oCurl->error_code);
                    }
                    if ($oCurl->response->Response !== 'True') {
                        throw new \RuntimeException($oCurl->response->Error);
                    }

                    return $oCurl->response;
                },
                [__METHOD__, $imdbId, $this->oClient->getLang()],
                3600 * 2)
            ;


            return Title::create($oResp);

        } catch (\Exception $e) {
            throw new Exception($e->getMessage(), 1, $e);
        }
    }

    /**
     * @param string $title
     *
     * @return Title
     * @throws \mrcnpdlk\Xmdb\Exception
     */
    public function getByTitle(string $title)

This method seems to be duplicated in your project.

    {
        try {
            $oResp = $this->oClient->getAdapter()->useCache(
                function () use ($title) {
                    $oCurl = new Curl();
                    $oCurl->setUserAgent(UserAgent::random());
                    $oCurl->setHeader('Accept-Language', $this->oClient->getLang());
                    $params = [
                        't'    => $title,
                        'plot' => 'full',
                        'r'    => 'json',
                    ];
                    $oCurl->get($this->url . '&' . http_build_query($params));

                    if ($oCurl->error) {
                        throw new \RuntimeException('Curl Error! ' . ($oCurl->httpStatusCode ? Http::message($oCurl->httpStatusCode) : 'Unknown code'),
                            $oCurl->error_code);
                    }
                    if ($oCurl->response->Response !== 'True') {
                        throw new \RuntimeException($oCurl->response->Error);
                    }

                    return $oCurl->response;
                },
                [__METHOD__, $title, $this->oClient->getLang()],
                3600 * 2)
            ;


            return Title::create($oResp);

        } catch (\Exception $e) {
            throw new Exception($e->getMessage(), 1, $e);
        }
    }
}