ParseInlineFqnNodeVisitor::__construct() - Code Metrics - jerowork/class-dependencies-parser - Measure and Improve Code Quality continuously with Scrutinizer

ParseInlineFqnNodeVisitor::__construct() A
last analyzed 2023-02-23 18:22 UTC

↳ Parent: ParseInlineFqnNodeVisitor

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	0

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	1
CRAP Score	1

Importance

Changes

Metric	Value
eloc	0
dl	0
loc	5
ccs	1
cts	1
cp	1
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	3
crap	1

<?php

declare(strict_types=1);

namespace Jerowork\ClassDependenciesParser\PhpParser\NodeVisitor;

use Jerowork\ClassDependenciesParser\ClassDependencies;
use Jerowork\ClassDependenciesParser\PhpParser\NodeVisitor\InlineFqnParser\Decliner\InlineFqnDecliner;
use Jerowork\ClassDependenciesParser\PhpParser\NodeVisitor\InlineFqnParser\Processor\InlineFqnProcessor;
use PhpParser\Node;
use PhpParser\Node\Name;
use PhpParser\NodeVisitorAbstract;

final class ParseInlineFqnNodeVisitor extends NodeVisitorAbstract
{
    /**
     * @param iterable<InlineFqnDecliner> $decliners
     * @param iterable<InlineFqnProcessor> $processors
     */
    public function __construct(
        private readonly ClassDependencies $classDependencies,
        private readonly iterable $decliners,
        private readonly iterable $processors,
    ) {
    }

    public function enterNode(Node $node): int|Node|null
    {
        if (!$node instanceof Name) {
            return parent::enterNode($node);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        }

        /** @var Node $parent */
        $parent = $node->getAttribute('parent');

        foreach ($this->decliners as $decliner) {
            if ($decliner->shouldDecline($parent, $node, $this->classDependencies)) {
                return parent::enterNode($node);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
            }
        }

        foreach ($this->processors as $processor) {
            if (!$processor->shouldProcess($parent, $node, $this->classDependencies)) {
                continue;
            }

            $fqn = $processor->process($parent, $node, $this->classDependencies);

            if ($fqn !== null) {
                $this->classDependencies->addInlineFqn($fqn);
            }

            break;
        }

        return parent::enterNode($node);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Jerowork\ClassDependenciesParser\PhpParser\NodeVisitor;
6
7		use Jerowork\ClassDependenciesParser\ClassDependencies;
8		use Jerowork\ClassDependenciesParser\PhpParser\NodeVisitor\InlineFqnParser\Decliner\InlineFqnDecliner;
9		use Jerowork\ClassDependenciesParser\PhpParser\NodeVisitor\InlineFqnParser\Processor\InlineFqnProcessor;
10		use PhpParser\Node;
11		use PhpParser\Node\Name;
12		use PhpParser\NodeVisitorAbstract;
13
14		final class ParseInlineFqnNodeVisitor extends NodeVisitorAbstract
15		{
16		/**
17		* @param iterable<InlineFqnDecliner> $decliners
18		* @param iterable<InlineFqnProcessor> $processors
19		*/
20	4	public function __construct(
21		private readonly ClassDependencies $classDependencies,
22		private readonly iterable $decliners,
23		private readonly iterable $processors,
24		) {
25	4	}
26
27	4	public function enterNode(Node $node): int\|Node\|null
28		{
29	4	if (!$node instanceof Name) {
30	4	return parent::enterNode($node);
		0 ignored issues – show Bug introduced 2023-02-23 18:06 UTC by Report Bug Copy Issue Report Are you sure the usage of `parent::enterNode($node)` targeting `PhpParser\NodeVisitorAbstract::enterNode()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
31		}
32
33		/** @var Node $parent */
34	4	$parent = $node->getAttribute('parent');
35
36	4	foreach ($this->decliners as $decliner) {
37	4	if ($decliner->shouldDecline($parent, $node, $this->classDependencies)) {
38	4	return parent::enterNode($node);
		0 ignored issues – show Bug introduced 2023-02-23 18:06 UTC by Report Bug Copy Issue Report Are you sure the usage of `parent::enterNode($node)` targeting `PhpParser\NodeVisitorAbstract::enterNode()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
39		}
40		}
41
42	4	foreach ($this->processors as $processor) {
43	4	if (!$processor->shouldProcess($parent, $node, $this->classDependencies)) {
44	4	continue;
45		}
46
47	4	$fqn = $processor->process($parent, $node, $this->classDependencies);
48
49	4	if ($fqn !== null) {
50	4	$this->classDependencies->addInlineFqn($fqn);
51		}
52
53	4	break;
54		}
55
56	4	return parent::enterNode($node);
		0 ignored issues – show Bug introduced 2023-02-23 18:06 UTC by Report Bug Copy Issue Report Are you sure the usage of `parent::enterNode($node)` targeting `PhpParser\NodeVisitorAbstract::enterNode()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
57		}
58		}
59

jerowork / class-dependencies-parser

ParseInlineFqnNodeVisitor::__construct() A last analyzed 2023-02-23 18:22 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

ParseInlineFqnNodeVisitor::__construct() A
last analyzed 2023-02-23 18:22 UTC