AliAlharthi\SaudiAddress\Api\Districts

Complexity

Total Complexity

15

Size/Duplication

Total Lines	165
Duplicated Lines	18.79 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	all()	15	24	5
A	get()	0	6	1
A	getId()	8	8	1
A	byId()	0	4	1
A	id()	0	4	1
A	getName()	8	8	1
A	byName()	0	4	1
A	named()	0	4	1
A	districtName()	0	4	1
A	check()	0	6	2

<?php

/*
 * Part of the Saudi Address API PHP package.
 *
 * NOTICE OF LICENSE
 *
 * Licensed under the MIT.
 *
 * This source file is subject to the MIT License that is
 * bundled with this package in the LICENSE file.
 *
 * @package    Saudi Address
 * @version    1.3
 * @author     Ali Alharthi
 * @license    MIT
 * @copyright  (c) 2020, Ali Alharthi
 * @link       https://aalharthi.sa
 */

namespace AliAlharthi\SaudiAddress\Api;

class Districts extends Api
{
    /**
     * The response array.
     *
     * @var array|null
     */
    protected $response = null;

    /**
     * The cache directory.
     *
     * @var string|null
     */
    protected $cacheDir = __DIR__ . '/cache/';

    /**
     * The cache file name.
     *
     * @var string|null
     */
    protected $file = __DIR__ . '/cache/' . 'districts_';

    /**
     * Returns a list of all the districts in a city.
     *
     * @param   int     $cityId
     * @param   string  $lang
     * @return  Districts
     */
    public function all($cityId = 1, $lang = 'A')
    {
        $cache = $this->file . $cityId . '_' . strtolower($lang) .  '.data';

        $this->response = $this->cacheValue($cache);

        if ($this->response == null) {

This code seems to be duplicated across your project.

            $response = $this->_get(
                'v3.1/lookup/districts',
                $lang,
                [
                    'cityid' => $cityId
                ]
            );
            if ($this->config->getCache()) {
                (!file_exists($this->cacheDir)) ?
                    mkdir($this->cacheDir, 0755, false) : ((file_exists($cache)) ? unlink($cache) : touch($cache));
                file_put_contents($cache, serialize($response));
            }
            $this->response = $response;
        }

        return $this;
    }

    /**
     * Returns a the response.
     *
     * @return  array
     */
    public function get()
    {
        $this->check();

        return $this->response['Districts'];
    }

    /**
     * Returns a specific district from a city by ID.
     *
     * @param   int     $districtId
     * @return  array
     */
    public function getId(int $districtId)

This method seems to be duplicated in your project.

    {
        $this->check();

        $key = array_search($districtId, array_column($this->response['Districts'], 'Id'));

        return $this->response['Districts'][$key];
    }

    /**
     * Returns a specific district from a city by ID.
     *
     * @param   int     $districtId
     * @return  array
     */
    public function byId(int $districtId)
    {
        return $this->getId($districtId);
    }

    /**
     * Returns a specific district from a city by ID.
     *
     * @param   int     $districtId
     * @return  array
     */
    public function id(int $districtId)
    {
        return $this->getId($districtId);
    }

    /**
     * Returns a certian district from a city by name.
     *
     * @param   string  $districtName
     * @return  array
     */
    public function getName($districtName)

This method seems to be duplicated in your project.

    {
        $this->check();

        $key = array_search($districtName, array_column($this->response['Districts'], 'Name'));

        return $this->response[$key];
    }

    /**
     * Returns a certian district from a city by name.
     *
     * @param   string  $districtName
     * @return  array
     */
    public function byName($districtName)
    {
        return $this->getName($districtName);
    }

    /**
     * Returns a certian district from a city by name.
     *
     * @param   string  $districtName
     * @return  array
     */
    public function named($districtName)
    {
        return $this->getName($districtName);
    }

    /**
     * Returns a certian district from a city by name.
     *
     * @param   string  $districtName
     * @return  array
     */
    public function districtName($districtName)
    {
        return $this->getName($districtName);
    }

    /**
     * Check if all() method was called first.
     *
     * @return  void
     * @throws  \BadMethodCallException
     */
    protected function check()
    {
        if ($this->response == null) {
            throw new \BadMethodCallException("You need to call all() method first.");
        }
    }
}