Bigwhoop\Trumpet\CodeParsing\Php\PhpCodeParser

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

86.67%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	parse()	0	15	2
A	createVisitor()	0	4	1

<?php declare(strict_types=1);

namespace Bigwhoop\Trumpet\CodeParsing\Php;

use Bigwhoop\Trumpet\CodeParsing\ParserException;
use PhpParser\Error;
use PhpParser\ParserAbstract as Parser;
use PhpParser\NodeTraverserInterface as NodeTraverser;
use PhpParser\PrettyPrinterAbstract as Printer;

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

    /** @var NodeTraverser */
    private $nodeTraverser;

    /** @var Printer */
    private $printer;

    public function __construct(Parser $parser, NodeTraverser $nodeTraverser, Printer $printer)
    {
        $this->parser = $parser;
        $this->nodeTraverser = $nodeTraverser;
        $this->printer = $printer;
    }

    public function parse(string $code): ParserResult
    {
        try {
            $stmts = $this->parser->parse($code);

            $visitor = $this->createVisitor();
            $this->nodeTraverser->addVisitor($visitor);

            $this->nodeTraverser->traverse($stmts);

It seems like $stmts defined by $this->parser->parse($code) on line 32 can also be of type null; however, PhpParser\NodeTraverserInterface::traverse() does only seem to accept array<integer,object<PhpParser\Node>>, maybe add an additional type check?

            return $visitor->getResult();
        } catch (Error $e) {
            throw new ParserException("Failed to parse PHP file: ".$e->getMessage(), $e->getCode(), $e);
        }
    }
    
    private function createVisitor(): NodeVisitor
    {
        return new NodeVisitor($this->printer);
    }
}