ComposerRequireChecker\NodeVisitor\DefinedSymbolCollector

Complexity

Total Complexity

17

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	beforeTraverse()	0	6	1
A	getDefinedSymbols()	0	4	1
A	enterNode()	0	8	1
A	recordClassDefinition()	0	6	2
A	recordInterfaceDefinition()	0	6	2
A	recordTraitDefinition()	0	6	2
A	recordFunctionDefinition()	0	6	2
A	recordConstDefinition()	0	8	3
A	recordDefinitionOf()	0	13	2

<?php

namespace ComposerRequireChecker\NodeVisitor;

use PhpParser\Node;
use PhpParser\NodeVisitorAbstract;

final class DefinedSymbolCollector extends NodeVisitorAbstract
{
    /**
     * @var mixed[]
     */
    private $definedSymbols = [];

    public function __construct()
    {
    }

    /**
     * {@inheritDoc}
     */
    public function beforeTraverse(array $nodes)
    {
        $this->definedSymbols = [];

        return parent::beforeTraverse($nodes);
    }

    /**
     * @return string[]
     */
    public function getDefinedSymbols() : array
    {
        return array_keys($this->definedSymbols);
    }

    /**
     * {@inheritDoc}
     */
    public function enterNode(Node $node)
    {
        $this->recordClassDefinition($node);
        $this->recordInterfaceDefinition($node);
        $this->recordTraitDefinition($node);
        $this->recordFunctionDefinition($node);
        $this->recordConstDefinition($node);
    }

    private function recordClassDefinition(Node $node)
    {
        if ($node instanceof Node\Stmt\Class_) {
            $this->recordDefinitionOf($node);
        }
    }

    private function recordInterfaceDefinition(Node $node)
    {
        if ($node instanceof Node\Stmt\Interface_) {
            $this->recordDefinitionOf($node);
        }
    }

    private function recordTraitDefinition(Node $node)
    {
        if ($node instanceof Node\Stmt\Trait_) {
            $this->recordDefinitionOf($node);
        }
    }

    private function recordFunctionDefinition(Node $node)
    {
        if ($node instanceof Node\Stmt\Function_) {
            $this->recordDefinitionOf($node);
        }
    }

    private function recordConstDefinition(Node $node)
    {
        if ($node instanceof Node\Stmt\Const_) {
            foreach ($node->consts as $const) {
                $this->recordDefinitionOf($const);
            }
        }
    }

    /**
     * @param Node $node
     *
     * @return void
     */
    private function recordDefinitionOf(Node $node)
    {
        if (! isset($node->namespacedName)) {

Accessing namespacedName on the interface PhpParser\Node suggest that you code against a concrete implementation. How about adding an instanceof check?

            throw new \UnexpectedValueException(sprintf(
                'Given node of type "%s" (defined at line %s)does not have an assigned "namespacedName" property: '
                . 'did you pass it through a name resolver visitor?',
                get_class($node),
                $node->getLine()
            ));
        }

        $this->definedSymbols[(string) $node->namespacedName] = $node->namespacedName;

Accessing namespacedName on the interface PhpParser\Node suggest that you code against a concrete implementation. How about adding an instanceof check?

    }
}