Arcanedev\Currencies\Entities\CurrencyCollection

Complexity

Total Complexity

10

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	load()	0	12	4
A	add()	0	9	1
A	addOne()	0	6	1
A	findOrFails()	0	10	2
A	loadMany()	0	8	2

<?php namespace Arcanedev\Currencies\Entities;

use Arcanedev\Currencies\Exceptions\CurrencyNotFoundException;
use Arcanedev\Support\Collection;
use Arcanedev\Currencies\Contracts\Entities\Currency as CurrencyContract;

/**
 * Class     CurrencyCollection
 *
 * @package  Arcanedev\Currencies\Entities
 * @author   ARCANEDEV <arcanedev.maroc@gmail.com>
 */
class CurrencyCollection extends Collection
{
    /* ------------------------------------------------------------------------------------------------
     |  Main Functions
     | ------------------------------------------------------------------------------------------------
     */
    /**
     * @param  array  $currencies
     * @param  bool   $includeNonIso
     *
     * @return self
     */
    public function load(array $currencies, $includeNonIso = false)
    {
        $this->reset();

        foreach ($currencies as $group => $items) {
            if ($group !== 'iso' && ! $includeNonIso) continue;

            $this->loadMany($items, ($group === 'iso'));
        }

        return $this;
    }

    /**
     * Add a currency to the collection.
     *
     * @param  string  $key
     * @param  array   $attributes
     *
     * @return self
     */
    public function add($key, array $attributes)
    {
        $key        = strtoupper($key);
        $attributes = compact('key') + $attributes;

        $this->addOne(Currency::make($key, $attributes));

        return $this;
    }

    /**
     * Add a Currency object to the collection.
     *
     * @param  \Arcanedev\Currencies\Contracts\Entities\Currency  $currency
     *
     * @return self
     */
    public function addOne(CurrencyContract $currency)
    {
        $this->put($currency->key, $currency);

Accessing key on the interface Arcanedev\Currencies\Contracts\Entities\Currency suggest that you code against a concrete implementation. How about adding an instanceof check?

        return $this;
    }

    /**
     * Check if the currency exists.
     *
     * @param  string  $iso
     *
     * @return \Arcanedev\Currencies\Entities\Currency
     *
     * @throws \Arcanedev\Currencies\Exceptions\CurrencyNotFoundException
     */
    public function findOrFails($iso)
    {
        $iso = strtoupper($iso);

        if ( ! $this->has($iso)) {
            throw new CurrencyNotFoundException("The Currency with the ISO code [$iso] not found");
        }

        return $this->get($iso);
    }

    /* ------------------------------------------------------------------------------------------------
     |  Other Functions
     | ------------------------------------------------------------------------------------------------
     */
    /**
     * Add many currencies.
     *
     * @param  array  $items
     * @param  bool   $isIso
     */
    protected function loadMany($items, $isIso = false)
    {
        foreach ($items as $key => $attributes) {
            $attributes = ['is_iso' => $isIso] + $attributes;

            $this->add($key, $attributes);
        }
    }
}