EndorphinStudio\Detector\Detector

Complexity

Total Complexity

13

Size/Duplication

Total Lines	100
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	getDataProvider()	0	4	1
A	getResultObject()	0	4	1
A	setDataProvider()	0	4	1
A	getUserAgent()	0	4	1
A	__construct()	0	17	3
A	analyze()	0	14	3
A	detect()	0	4	1
A	getPatternList()	0	4	2

<?php
/**
 * @author Serhii Nekhaienko <sergey.nekhaenko@gmail.com>
 * @license GPL
 * @copyright Serhii Nekhaienko &copy 2018
 * @version 4.0.0
 * @project browser-detector
 */

namespace EndorphinStudio\Detector;

use EndorphinStudio\Detector\Data\Result;
use EndorphinStudio\Detector\Exception\StorageException;
use EndorphinStudio\Detector\Storage\StorageInterface;
use Twig\Parser;

class Detector
{
    /**
     * @var StorageInterface
     */
    private $dataProvider;

    /**
     * @return StorageInterface
     */
    public function getDataProvider(): StorageInterface
    {
        return $this->dataProvider;
    }

    /**
     * @return Result
     */
    public function getResultObject(): Result
    {
        return $this->resultObject;
    }

    /**
     * @var Result
     */
    private $resultObject;

    /**
     * @param StorageInterface $dataProvider
     */
    public function setDataProvider(StorageInterface $dataProvider)
    {
        $this->dataProvider = $dataProvider;
    }

    /**
     * @return mixed
     */
    public function getUserAgent()
    {
        return $this->ua;
    }

    private $ua;

    /**
     * @var array
     */
    private $detectors;

    /**
     * Detector constructor.
     * @param string $dataProvider
     * @param string $format
     * @throws StorageException
     */
    public function __construct(string $dataProvider = '\\EndorphinStudio\\Detector\\Storage\\YamlStorage', string $format = 'yaml')
    {
        $dataDirectory = sprintf('%s/var/%s', dirname(__DIR__), $format);
        $dataProvider = new $dataProvider();
        $dataProvider->setDataDirectory($dataDirectory);
        $this->setDataProvider($dataProvider);
        $this->detectors = [];
        $this->resultObject = new Result();
        $check = ['device', 'os', 'browser', 'robots'];
        foreach ($check as $detectionnType) {
            $className = sprintf('\\EndorphinStudio\\Detector\\Detection\\%s', ucfirst(sprintf('%sDetector', $detectionnType)));
            if(class_exists($className)) {
                $this->detectors[$detectionnType] = new $className();
                $this->detectors[$detectionnType]->init($this);
            }
        }
    }

    public function analyze(string $ua = 'ua')
    {
        $this->ua = $ua === 'ua' ? $_SERVER['HTTP_USER_AGENT'] : $ua;
        $config = $this->getDataProvider()->getConfig();

$config is not used, you could remove the assignment.

        foreach ($this->detectors as $detectionType => $detector) {
            $detector->detect($ua);
        }
        echo 'test';
//        $this->resultObject = new Result();

54% of this comment could be valid code. Did you maybe forget this after debugging?

//        foreach ($check as $type) {
//            $this->detect($config, $type);
//        }
//        var_dump($this->resultObject);
    }

    private function detect($config, $type)

This method is not used, and could be removed.

The parameter $config is not used and could be removed.

The parameter $type is not used and could be removed.

    {

    }

    public function getPatternList($list, $type)
    {
        return array_key_exists($type, $list) ? $list[$type] : [];
    }
}