Spiral\Cycle\Promise\Traverser

Complexity

Total Complexity

7

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	traverseFilename()	0	4	1
A	traverseClonedNodes()	0	4	1
A	parseNodes()	0	4	1
A	cloneNodes()	0	4	1
A	makeTraverser()	0	9	2

<?php
declare(strict_types=1);

namespace Cycle\ORM\Promise;

use PhpParser\NodeTraverser;
use PhpParser\NodeVisitor;
use PhpParser\Parser;
use PhpParser\ParserFactory;

class Traverser
{
    /** @var Parser */
    private $parser;

    public function __construct(Parser $parser = null)
    {
        $this->parser = $parser ?? (new ParserFactory())->create(ParserFactory::ONLY_PHP7);
    }

    public function traverseFilename(string $filename, NodeVisitor ...$visitors)
    {
        return $this->makeTraverser(...$visitors)->traverse($this->parseNodes($filename));

It seems like $this->parseNodes($filename) targeting Cycle\ORM\Promise\Traverser::parseNodes() 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?

    }

    public function traverseClonedNodes(array $nodes, NodeVisitor ...$visitors)
    {
        return $this->makeTraverser(...$visitors)->traverse($this->cloneNodes($nodes));
    }

    private function parseNodes(string $filename)
    {
        return $this->parser->parse(file_get_contents($filename));
    }

    private function cloneNodes(array $nodes)
    {
        return $this->makeTraverser(new NodeVisitor\CloningVisitor())->traverse($nodes);
    }

    private function makeTraverser(NodeVisitor ...$visitors): NodeTraverser
    {
        $traverser = new NodeTraverser();
        foreach ($visitors as $visitor) {
            $traverser->addVisitor($visitor);
        }

        return $traverser;
    }
}