Bigwhoop\Trumpet\CodeParsing\Php\NodeVisitor

Complexity

Total Complexity

10

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

90.91%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	2
A	getResult()	0	4	1
A	enterNode()	0	12	4
A	leaveNode()	0	6	2
A	getSource()	0	4	1

<?php declare(strict_types=1);

namespace Bigwhoop\Trumpet\CodeParsing\Php;

use PhpParser\Node;
use PhpParser\NodeVisitorAbstract;
use PhpParser\PrettyPrinterAbstract as Printer;

final class NodeVisitor extends NodeVisitorAbstract
{
    /** @var ParserResult */
    private $result;

    /** @var PhpClass|null */
    private $currentClass;

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

    /**
     * @param Printer      $printer
     * @param ParserResult $result
     */
    public function __construct(Printer $printer, ParserResult $result = null)
    {
        $this->printer = $printer;
        $this->result = $result ?: new ParserResult();
    }

    public function getResult(): ParserResult
    {
        return $this->result;
    }

    /**
     * {@inheritdoc}
     */
    public function enterNode(Node $node)
    {
        if ($node instanceof Node\Stmt\Class_) {
            $fullyQualifiedName = $node->namespacedName->toString();

The property namespacedName does not seem to exist in PhpParser\Node\Stmt\Class_.

            $this->currentClass = new PhpClass($fullyQualifiedName, [], $this->getSource($node));
        } elseif ($node instanceof Node\Stmt\ClassMethod) {
            $this->currentClass->addMethod(new PhpMethod($node->name, $node->isStatic(), $this->getSource($node)));
        } elseif ($node instanceof Node\Stmt\Function_) {
            $fullyQualifiedName = $node->namespacedName->toString();

The property namespacedName does not seem to exist in PhpParser\Node\Stmt\Function_.

            $this->result->addFunction(new PhpFunction($fullyQualifiedName, $this->getSource($node)));
        }
    }

    /**
     * {@inheritdoc}
     */
    public function leaveNode(Node $node)
    {
        if ($node instanceof Node\Stmt\Class_) {
            $this->result->addClass($this->currentClass);

It seems like $this->currentClass can be null; however, addClass() does not accept null, maybe add an additional type check?

        }
    }

    private function getSource(Node $node): string
    {
        return $this->printer->prettyPrint([$node]);
    }
}