Ray\Aop\CodeVisitor

Complexity

Total Complexity

10

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Importance

Changes	5
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	enterNode()	0	21	4
A	enterNodeClass()	0	14	3
A	validateClass()	0	7	3

<?php

declare(strict_types=1);

namespace Ray\Aop;

use PhpParser\Node;
use PhpParser\Node\Stmt\Class_;
use PhpParser\Node\Stmt\ClassMethod;
use PhpParser\Node\Stmt\Declare_;
use PhpParser\Node\Stmt\Namespace_;
use PhpParser\Node\Stmt\Use_;
use PhpParser\NodeVisitorAbstract;
use Ray\Aop\Exception\MultipleClassInOneFileException;

final class CodeVisitor extends NodeVisitorAbstract
{
    /** @var ?Namespace_ */
    public $namespace;

    /** @var Declare_[] */
    public $declare = [];

    /** @var Use_[] */
    public $use = [];

    /** @var Class_|null */
    public $class;

    /** @var ClassMethod[] */
    public $classMethod = [];

    /**
     * @return null
     */
    public function enterNode(Node $node)
    {
        if ($node instanceof Declare_) {
            $this->declare[] = $node;

            return null;
        }

        if ($node instanceof Use_) {
            $this->use[] = $node;

            return null;
        }

        if ($node instanceof Namespace_) {
            $this->namespace = $node;

            return null;
        }

        return $this->enterNodeClass($node);

Are you sure the usage of $this->enterNodeClass($node) targeting Ray\Aop\CodeVisitor::enterNodeClass() seems to always return null.

    }

    private function validateClass(Class_ $class): void
    {
        $isClassAlreadyDeclared = $this->class instanceof Class_;
        if ($isClassAlreadyDeclared) {
            $name = $class->name instanceof Node\Identifier ? $class->name->name : '';

            throw new MultipleClassInOneFileException($name);
        }
    }

    /**
     * @return null
     */
    private function enterNodeClass(Node $node)
    {
        if ($node instanceof Class_) {
            $this->validateClass($node);
            $this->class = $node;

            return null;
        }

        if ($node instanceof ClassMethod) {
            $this->classMethod[] = $node;
        }

        return null;
    }
}