FileParser::parserFile() - Code Metrics - Inspection of "[ControlFlowGraph] Initial" - ovr/phpsa - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#293)

by Дмитрий

created 2017-01-14 13:52 UTC

FileParser::parserFile() B

↳ Parent: FileParser

Complexity

Conditions	4
Paths	17

Size

Total Lines	24
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	4.3035

Importance

Changes

Metric	Value
cc	4
eloc	15
nc	17
nop	2
dl	0
loc	24
ccs	11
cts	15
cp	0.7332
crap	4.3035
rs	8.6845
c	0
b	0
f	0

<?php
/**
 * @author Patsura Dmitry https://github.com/ovr <[email protected]>
 */

namespace PHPSA\Definition;

use PHPSA\Compiler;
use PHPSA\Context;
use RuntimeException;

class FileParser
{
    /**
     * @var \PhpParser\Parser
     */
    protected $parser;

    /**
     * @var \PhpParser\NodeTraverser
     */
    protected $nodeTraverser;

    /**
     * @var Compiler
     */
    protected $compiler;

    /**
     * @var \PHPSA\Compiler\DefinitionVisitor
     */
    protected $definitionVisitor;

    /**
     * @param \PhpParser\Parser $parser
     * @param Compiler $compiler
     */
    public function __construct(\PhpParser\Parser $parser, Compiler $compiler)
    {
        $this->nodeTraverser = new \PhpParser\NodeTraverser();
        $this->nodeTraverser->addVisitor(new \PhpParser\NodeVisitor\NameResolver);
        $this->nodeTraverser->addVisitor(
            $this->definitionVisitor = new \PHPSA\Compiler\DefinitionVisitor($compiler)
        );

        $this->parser = $parser;
        $this->compiler = $compiler;
    }

    /**
     * @param string $filepath
     * @param Context $context
     * @throws RuntimeException when filepath is not readable
     */
    public function parserFile($filepath, Context $context)
    {
        $context->setFilepath($filepath);

        try {
            if (!is_readable($filepath)) {
                throw new RuntimeException('File ' . $filepath . ' is not readable');
            }

            $context->debug('<comment>Precompile: ' . $filepath . '.</comment>');

            $code = file_get_contents($filepath);
            $astTree = $this->parser->parse($code);

            $this->definitionVisitor->setFilePath($filepath);
            $this->nodeTraverser->traverse($astTree);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

            $context->clear();
        } catch (\PhpParser\Error $e) {
            $context->syntaxError($e, $filepath);
        } catch (\Exception $e) {
            $context->output->writeln("<error>{$e->getMessage()}</error>");
        }
    }
}


1		<?php
2		/**
3		* @author Patsura Dmitry https://github.com/ovr <[email protected]>
4		*/
5
6		namespace PHPSA\Definition;
7
8		use PHPSA\Compiler;
9		use PHPSA\Context;
10		use RuntimeException;
11
12		class FileParser
13		{
14		/**
15		* @var \PhpParser\Parser
16		*/
17		protected $parser;
18
19		/**
20		* @var \PhpParser\NodeTraverser
21		*/
22		protected $nodeTraverser;
23
24		/**
25		* @var Compiler
26		*/
27		protected $compiler;
28
29		/**
30		* @var \PHPSA\Compiler\DefinitionVisitor
31		*/
32		protected $definitionVisitor;
33
34		/**
35		* @param \PhpParser\Parser $parser
36		* @param Compiler $compiler
37		*/
38	51	public function __construct(\PhpParser\Parser $parser, Compiler $compiler)
39		{
40	51	$this->nodeTraverser = new \PhpParser\NodeTraverser();
41	51	$this->nodeTraverser->addVisitor(new \PhpParser\NodeVisitor\NameResolver);
42	51	$this->nodeTraverser->addVisitor(
43	51	$this->definitionVisitor = new \PHPSA\Compiler\DefinitionVisitor($compiler)
44	51	);
45
46	51	$this->parser = $parser;
47	51	$this->compiler = $compiler;
48	51	}
49
50		/**
51		* @param string $filepath
52		* @param Context $context
53		* @throws RuntimeException when filepath is not readable
54		*/
55	51	public function parserFile($filepath, Context $context)
56		{
57	51	$context->setFilepath($filepath);
58
59		try {
60	51	if (!is_readable($filepath)) {
61		throw new RuntimeException('File ' . $filepath . ' is not readable');
62		}
63
64	51	$context->debug('<comment>Precompile: ' . $filepath . '.</comment>');
65
66	51	$code = file_get_contents($filepath);
67	51	$astTree = $this->parser->parse($code);
68
69	51	$this->definitionVisitor->setFilePath($filepath);
70	51	$this->nodeTraverser->traverse($astTree);
		0 ignored issues – show Bug introduced 2016-05-21 07:27 UTC by Report Bug Copy Issue Report It seems like `$astTree` defined by `$this->parser->parse($code)` on line `67` can also be of type `null`; however, `PhpParser\NodeTraverser::traverse()` does only seem to accept `array<integer,object<PhpParser\Node>>`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
71
72	51	$context->clear();
73	51	} catch (\PhpParser\Error $e) {
74		$context->syntaxError($e, $filepath);
75		} catch (\Exception $e) {
76		$context->output->writeln("<error>{$e->getMessage()}</error>");
77		}
78	51	}
79		}
80

ovr / phpsa

Pull Request — master (#293)

FileParser::parserFile() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like