Benoth\StaticReflection\Parser\DebugNodeVisitor

Complexity

Total Complexity

8

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	2
Bugs	1	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	enterNode()	0	4	4
A	leaveNode()	0	4	1
A	beforeTraverse()	0	5	1
A	afterTraverse()	0	5	1
A	debug()	0	4	1

<?php

namespace Benoth\StaticReflection\Parser;

use PhpParser\Node as NodeInterface;
use PhpParser\NodeVisitorAbstract;

/**
 * @codeCoverageIgnore
 */
class DebugNodeVisitor extends NodeVisitorAbstract
{
    protected $level = 0;

    public function enterNode(NodeInterface $node)
    {
        $this->debug(str_repeat('  ', ++$this->level).$node->getType().(property_exists($node, 'name') ? ' '.(method_exists($node->name, 'toString') ? $node->name->toString() : $node->name) : '').(method_exists($node, 'getLine') ? ' L:'.$node->getLine() : '').PHP_EOL);

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

    }

    public function leaveNode(NodeInterface $node)
    {
        --$this->level;
    }

    public function beforeTraverse(array $nodes)
    {
        $this->level = 0;
        $this->debug('BeforeTraverse'.PHP_EOL);
    }

    public function afterTraverse(array $nodes)
    {
        $this->level = 0;
        $this->debug('AfterTraverse'.PHP_EOL);
    }

    /**
     * @param string $text
     */
    protected function debug($text)
    {
        echo $text;
    }
}