Crossjoin\Browscap\Parser\Sqlite\Adapter\AdapterFactory

Complexity

Total Complexity

12

Size/Duplication

Total Lines	101
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getInstance()	18	18	4
A	setDefaultAdapterClasses()	4	4	1
A	setAdapterClasses()	18	18	3
B	getInstanceByClassName()	23	23	4

<?php
declare(strict_types=1);

namespace Crossjoin\Browscap\Parser\Sqlite\Adapter;

use Crossjoin\Browscap\Exception\BrowscapException;
use Crossjoin\Browscap\Exception\InvalidArgumentException;
use Crossjoin\Browscap\Exception\ParserConditionNotSatisfiedException;
use Crossjoin\Browscap\Exception\UnexpectedValueException;

/**
 * Class AdapterFactory
 *
 * @package Crossjoin\Browscap\Parser\Sqlite\Adapter
 * @author Christoph Ziegenberg <ziegenberg@crossjoin.com>
 * @link https://github.com/crossjoin/browscap
 */
class AdapterFactory
{
    /**
     * @var array
     */
    protected static $defaultAdapterClasses = [
        '\Crossjoin\Browscap\Parser\Sqlite\Adapter\Pdo',
        '\Crossjoin\Browscap\Parser\Sqlite\Adapter\Sqlite3',
    ];

    /**
     * @var array|null
     */
    protected static $adapterClasses;

    /**
     * @param string $fileName
     *
     * @return AdapterInterface
     * @throws ParserConditionNotSatisfiedException
     * @throws UnexpectedValueException
     */
    public static function getInstance(string $fileName) : AdapterInterface
    {
        // Initialize adapter classes
        if (static::$adapterClasses === null) {
            self::setDefaultAdapterClasses();
        }
        
        foreach (static::$adapterClasses as $className) {

The expression static::$adapterClasses of type null|array is not guaranteed to be traversable. How about adding an additional type check?

            $instance = static::getInstanceByClassName($className, $fileName);
            if ($instance !== null) {
                return $instance;
            }
        }

        throw new ParserConditionNotSatisfiedException(
            "No Sqlite extension found. Either 'pdo_sqlite' or 'sqlite3' extension required."
        );
    }
    
    public static function setDefaultAdapterClasses()
    {
        self::setAdapterClasses(self::$defaultAdapterClasses);
    }

    /**
     * @param array $fullyQualifiedClassNames
     *
     * @throws InvalidArgumentException
     */
    public static function setAdapterClasses(array $fullyQualifiedClassNames)
    {
        $rememberClasses = self::$adapterClasses;

        self::$adapterClasses = [];
        foreach ($fullyQualifiedClassNames as $className) {
            if (is_string($className)) {
                self::$adapterClasses[] = $className;
            } else {
                // Reset to previous value on error
                self::$adapterClasses = $rememberClasses;

                throw new InvalidArgumentException(
                    "A value in the class name array is of type '" . gettype($className) . "'. String expected."
                );
            }
        }
    }

    /**
     * @param string $className
     * @param string $fileName
     *
     * @return AdapterInterface|null
     * @throws UnexpectedValueException
     */
    protected static function getInstanceByClassName(string $className, string $fileName)
    {
        if (class_exists($className)) {
            $interface = '\Crossjoin\Browscap\Parser\Sqlite\Adapter\AdapterFactoryInterface';
            $interfaces = class_implements($className);
            if (array_key_exists(ltrim($interface, '\\'), $interfaces)) {
                try {
                    return new $className($fileName);
                } catch (BrowscapException $e) {
                    // Ignore exception, because we just return NULL on failure
                }
            } else {
                throw new UnexpectedValueException(
                    "Class '$className' has to implement the interface '$interface'.",

As per coding-style, please use concatenation or sprintf for the variable $className instead of interpolation.

As per coding-style, please use concatenation or sprintf for the variable $interface instead of interpolation.

                    1459000689
                );
            }
        } else {
            throw new UnexpectedValueException("Class '$className' doesn't exist.", 1459000690);

As per coding-style, please use concatenation or sprintf for the variable $className instead of interpolation.

        }

        return null;
    }
}