LocateASTFromFiles::__invoke() - Code Metrics - Inspection of "[WIP] raise phpstan level" - maglnet/ComposerRequireChecker - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Pull Request — master (#153)

by Matthias

created 2019-10-25 15:32 UTC

LocateASTFromFiles::__invoke() A

↳ Parent: LocateASTFromFiles

Complexity

Conditions	3
Paths	3

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	3.0416

Importance

Changes	3
Bugs	0	Features	1

Metric	Value
eloc	5
c	3
b	0
f	1
dl	0
loc	8
ccs	5
cts	6
cp	0.8333
rs	10
cc	3
nc	3
nop	1
crap	3.0416

<?php

namespace ComposerRequireChecker\ASTLocator;

use PhpParser\ErrorHandler;
use PhpParser\Parser;
use Traversable;

final class LocateASTFromFiles
{
    /**
     * @var Parser
     */
    private $parser;

    /**
     * @var ErrorHandler|null
     */
    private $errorHandler;

    public function __construct(Parser $parser, ?ErrorHandler $errorHandler)
    {
        $this->parser = $parser;
        $this->errorHandler = $errorHandler;
    }

    /**
     * @param Traversable|string[] $files
     *
     * @return Traversable|array[] a series of AST roots, one for each given file
     */
    public function __invoke(Traversable $files): Traversable
    {
        foreach ($files as $file) {
            $fileContent = file_get_contents($file);
            if ($fileContent === false) {
                throw new \InvalidArgumentException('could not load file [' . $file . ']');
            }
            yield $this->parser->parse($fileContent, $this->errorHandler);
        }
    }
}


1		<?php
2
3		namespace ComposerRequireChecker\ASTLocator;
4
5		use PhpParser\ErrorHandler;
6		use PhpParser\Parser;
7		use Traversable;
8
9		final class LocateASTFromFiles
10		{
11		/**
12		* @var Parser
13		*/
14		private $parser;
15
16		/**
17		* @var ErrorHandler\|null
18		*/
19		private $errorHandler;
20
21	6	public function __construct(Parser $parser, ?ErrorHandler $errorHandler)
22		{
23	6	$this->parser = $parser;
24	6	$this->errorHandler = $errorHandler;
25	6	}
26
27		/**
28		* @param Traversable\|string[] $files
29		*
30		* @return Traversable\|array[] a series of AST roots, one for each given file
31		*/
32	6	public function __invoke(Traversable $files): Traversable
33		{
34	6	foreach ($files as $file) {
35	6	$fileContent = file_get_contents($file);
36	6	if ($fileContent === false) {
37		throw new \InvalidArgumentException('could not load file [' . $file . ']');
38		}
39	6	yield $this->parser->parse($fileContent, $this->errorHandler);
40		}
41	5	}
42		}
43

maglnet / ComposerRequireChecker

Pull Request — master (#153)

LocateASTFromFiles::__invoke() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like