Inspector\Analysis\Analyzer

Complexity

Total Complexity

4

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	analyze()	0	15	2
A	getIssues()	0	7	1

<?php

namespace Inspector\Analysis;

use PhpParser\Parser;
use Inspector\Analysis\Result\Issue;
use Inspector\Filesystem\SourceIterator;
use Inspector\Analysis\Result\AnalyzedFile;
use Inspector\Foundation\MessageProviderAwareTrait;
use Inspector\Foundation\MessageProviderAwareInterface;

/**
 * @author Kabir Baidhya
 */
class Analyzer implements MessageProviderAwareInterface
{

    use MessageProviderAwareTrait;

    /**
     * @var Parser
     */
    protected $parser;

    /**
     * @var FlawDetector
     */
    protected $flawDetector;

    /**
     * @param Parser $parser
     * @param FlawDetector $flawDetector
     */
    public function __construct(Parser $parser, FlawDetector $flawDetector)
    {
        $this->parser = $parser;
        $this->flawDetector = $flawDetector;
    }

    /**
     *
     * @param SourceIterator $source
     * @param array $options
     * @return array
     */
    public function analyze(SourceIterator $source, array $options)

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

    {
        $result = [];
        foreach ($source as $filename => $code) {
            $ast = $this->parser->parse($code);

            $issues = $this->getIssues($ast, [
                'filename' => $filename
            ]);

            $result[$filename] = new AnalyzedFile($filename, $issues);
        }

        return $result;
    }

    /**
     * @param $ast
     * @param array $data
     * @return Issue[]
     */
    protected function getIssues($ast, array $data)

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

    {
        $exceptions = $this->flawDetector->analyze($ast);
        $issues = $this->messageProvider->translateExceptions($exceptions);

        return $issues;
    }

}