Sarnado\Converter\Collections\CryptoRatesCollection

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	filterByPair()	0	5	2
A	check()	0	13	4
A	__construct()	0	9	2

<?php


namespace Sarnado\Converter\Collections;

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

/**
 * Class CryptoRatesCollection
 * @package Sarnado\Converter\Collections
 */
class CryptoRatesCollection extends Collection
{
    /**
     * CryptoRatesCollection 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 CryptoRateObject');
        }
    }

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

    public function filterByPair(string $crypto, string $fiat): CryptoRatesCollection
    {
        return $this->filter(function (CryptoRateObject $rate) use ($crypto, $fiat)
        {

Opening brace should be on the same line as the declaration

            return $rate->getCrypto() === $crypto && $rate->getFiat() === $fiat;
        });
    }
}