Del\Repository\CountryRepository

Complexity

Total Complexity

9

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

61.53%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	findAllCountries()	0	10	2
A	findCountryByIsoCode()	0	8	2
A	findCountryBy()	0	10	3
A	createFromArray()	0	11	1

<?php

namespace Del\Repository;

use Del\Exception\CountryException;
use Del\Entity\Country;
use Del\Factory\CountryList;

class CountryRepository
{
    /** @var Country[] $countries */
    private $countries;

    /**
     * CountryRepository constructor.
     */
    public function __construct()
    {
        $list = new CountryList();
        $this->countries = $list->getCountries();
    }

    /**
     * @return Country[]
     */
    public function findAllCountries(): array
    {
        $countries = [];

        foreach ($this->countries as $country) {
            $countries[] = $this->createFromArray($country);

$country is of type object<Del\Entity\Country>, but the function expects a array.

        }

        return $countries;
    }

    /**
     * @param string $id
     * @return Country
     * @throws CountryException
     */
    public function findCountryByIsoCode(string $id): Country
    {
        if (isset($this->countries[$id])) {
            return $this->createFromArray($this->countries[$id]);

$this->countries[$id] is of type object<Del\Entity\Country>, but the function expects a array.

        }

        throw new CountryException(CountryException::ERROR_NOT_FOUND);
    }

    /**
     * @param string $id

There is no parameter named $id. Was it maybe removed?

     * @return Country
     * @throws CountryException
     */
    public function findCountryBy(string $key, string $value): Country
    {
        foreach ($this->countries as $country) {
            if ($country[$key] === $value) {
                return $this->createFromArray($this->countries[$id]);

The variable $id does not exist. Did you forget to declare it?

$this->countries[$id] is of type object<Del\Entity\Country>, but the function expects a array.

            }
        }

        throw new CountryException(CountryException::ERROR_NOT_FOUND);
    }

    /**
     * @param array $data
     * @return Country
     */
    public function createFromArray(array $data): Country
    {
        $country = new Country();
        $country->setId($data['id']);
        $country->setIso($data['iso']);
        $country->setName($data['name']);
        $country->setNumCode($data['numcode']);
        $country->setFlag($data['flag']);

        return $country;
    }
}