LaravelCryptoStats\LaravelCryptoStatsFactory::getInstance()

Complexity

Conditions	5
Paths	7

Size

Total Lines	23
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function getInstance($currency)
    {
        $connectors = $this->connectors;
        if ($connectors) {

The expression $connectors of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $supported_currencies = [];
            $instance = [];

            foreach ($connectors as $connector) {
                $connector->setCurrency($currency);
                $supported_currencies = array_merge($supported_currencies, $connector->supported_currencies);

                if (in_array($currency, $connector->supported_currencies)) {
                    $instance = $connector;
                }
            }

            if ($instance) {
                return $instance;
            }

            $supported_currencies = implode(', ', $supported_currencies);

            throw new Exception('"'.$currency.'" cryptocurrency is not supported now! Currently available values: '.$supported_currencies);
        }