sarnado / converter

src/Collections/CryptoRatesCollection.php

<?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;
        });
    }
}