Sarnado\Converter\Collections\FiatRatesCollection

Complexity

Total Complexity

8

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	2
A	filterByPair()	0	5	2
A	check()	0	13	4

<?php


namespace Sarnado\Converter\Collections;


use Sarnado\Converter\Objects\CryptoRateObject;
use Sarnado\Converter\Objects\FiatRateObject;
use Tightenco\Collect\Support\Collection;

/**
 * Class FiatRatesCollection
 * @package Sarnado\Converter\Collections
 */
class FiatRatesCollection extends Collection
{
    /**
     * FiatRatesCollection constructor.
     * @param array $data
     */
    public function __construct(array $data)
    {
        if ($this->check($data))
        {
            parent::__construct($data);
        }
        else

Expected 1 space(s) after ELSE keyword; newline found

        {
            throw new \InvalidArgumentException('Item from array must be instance of FiatRateObject');
        }
    }

    /**
     * @param array $data
     * @return bool
     */
    private function check(array $data): bool
    {
        if (!empty($data))
        {
            foreach ($data as $item)
            {
                if (!($item instanceof FiatRateObject))
                {
                    return false;
                }
            }
        }
        return true;
    }

    public function filterByPair(string $from, string $to): FiatRatesCollection
    {
        return $this->filter(function (FiatRateObject $rate) use ($from, $to)
        {

Opening brace should be on the same line as the declaration

            return $rate->getFrom() === $from && $rate->getTo() === $to;
        });
    }
}