PHPSemVerCheckerGit\SourceFileProcessor

Complexity

Total Complexity

6

Size/Duplication

Total Lines	80
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	13	3
A	processFileList()	0	9	1
A	scanFileList()	0	9	2

<?php

namespace PHPSemVerCheckerGit;


use PHPSemVerChecker\Finder\Finder;
use PHPSemVerChecker\Scanner\Scanner;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Helper\ProgressBar;
use PHPSemVerCheckerGit\Filter\SourceFilter;
use Gitter\Repository;

class SourceFileProcessor
{
    /**
     * @var Repository
     */
    private $repository;
    /**
     * @var OutputInterface
     */
    private $output;
    /**
     * @var Finder
     */
    private $finder;
    /**
     * @var SourceFilter
     */
    private $filter;
    /**
     * @var string
     */
    private $directory;
    /**
     * @var string[]
     */
    private $modifiedFiles;

    /**
     * SourceFileProcessor constructor.
     * @param SourceFilter &$filter
     * @param Repository $repository
     * @param OutputInterface $output
     * @param Finder $finder
     * @param string $directory
     * @param string[] $modifiedFiles
     */
    public function __construct(SourceFilter $filter, Repository $repository, OutputInterface $output, Finder $finder, $directory, array $modifiedFiles)
    {
        $this->repository = $repository;
        $this->output = $output;
        $this->finder = $finder;
        $this->filter = $filter;
        $this->directory = $directory;
        foreach($modifiedFiles as $file) {
            if(substr($file, -4) === '.php') {
                $this->modifiedFiles[] = $file;
            }
        }
    }

    /**
     * @param string $commitIdentifier
     * @param $include
     * @param $exclude
     * @return ProcessedFileList
     */
    public function processFileList($commitIdentifier, $include, $exclude)
    {
        $scanner = new Scanner();
        $this->repository->checkout($commitIdentifier . ' --');
        $unfiltered = $this->finder->findFromString($this->directory, $include, $exclude);
        $source = $this->filter->filter($source, $this->modifiedFiles);

The variable $source seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

        $this->scanFileList($scanner, $source);
        return new ProcessedFileList($unfiltered, $source, $scanner);
    }

    /**
     * @param Scanner $scanner
     * @param array $files
     */
    private function scanFileList(Scanner $scanner, array $files)
    {
        $progress = new ProgressBar($this->output, count($files));
        foreach ($files as $file) {
            $scanner->scan($file);
            $progress->advance();
        }
        $progress->clear();
    }
}