DetectorVisitor::enterNode() - Code Metrics - Inspection of "Check for named constants" - povils/phpmnd - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#86)

by Scott

created 2019-03-07 13:21 UTC

DetectorVisitor::enterNode() B

↳ Parent: DetectorVisitor

Complexity

Conditions	11
Paths	17

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	42
rs	7.3166
c	0
b	0
f	0
cc	11
nc	17
nop	1

How to fix Complexity

<?php

namespace Povils\PHPMND\Visitor;

use PhpParser\Node;
use PhpParser\Node\Const_;
use PhpParser\Node\Expr\UnaryMinus;
use PhpParser\Node\Expr\UnaryPlus;
use PhpParser\Node\Scalar;
use PhpParser\Node\Scalar\DNumber;
use PhpParser\Node\Scalar\LNumber;
use PhpParser\Node\Scalar\String_;
use PhpParser\NodeTraverser;
use PhpParser\NodeVisitorAbstract;
use Povils\PHPMND\Console\Option;
use Povils\PHPMND\Extension\ArrayAwareExtension;
use Povils\PHPMND\Extension\Extension;
use Povils\PHPMND\Extension\FunctionAwareExtension;
use Povils\PHPMND\FileReport;

/**
 * Class DetectorVisitor
 *
 * @package Povils\PHPMND
 */
class DetectorVisitor extends NodeVisitorAbstract
{
    /**
     * @var FileReport
     */
    private $fileReport;

    /**
     * @var Option
     */
    private $option;

    public function __construct(FileReport $fileReport, Option $option)
    {
        $this->fileReport = $fileReport;
        $this->option = $option;
    }

    public function enterNode(Node $node): ?int
    {
        if ($this->isIgnoreableConst($node)) {
            if ($this->checkNameContainsLanguage(
                $node->name->name,
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
                $node->value->value ?? 0
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
            )) {
                $this->fileReport->addEntry($node->getLine(), $node->value->value);
            }

            return NodeTraverser::DONT_TRAVERSE_CHILDREN;
        }

        /** @var LNumber|DNumber|String_ $scalar */
        $scalar = $node;
        if ($this->hasSign($scalar)) {
            $node = $scalar->getAttribute('parent');
            if ($this->isMinus($node) && isset($scalar->value)) {
                $scalar->value = -$scalar->value;
            }
        }

        if ($this->isNumber($scalar) || $this->isString($scalar)) {
            if ($this->checkNameContainsLanguage(
                $scalar->getAttribute('parent')->var->name ?? '',
                $scalar->value
            )) {
                $this->fileReport->addEntry($node->getLine(), $scalar->value);
            }

            foreach ($this->option->getExtensions() as $extension) {
                $extension->setOption($this->option);
                if ($extension->extend($node)) {
                    $this->fileReport->addEntry($scalar->getLine(), $scalar->value);

                    return null;
                }
            }
        }

        return null;
    }

    private function isIgnoreableConst(Node $node): bool
    {
        return $node instanceof Const_ &&
            ($this->isNumber($node->value) || $this->isString($node->value));
    }

    private function isNumber(Node $node): bool
    {
        $isNumber = (
            $node instanceof LNumber ||
            $node instanceof DNumber ||
            $this->isValidNumeric($node)
        );

        return $isNumber && false === $this->ignoreNumber($node);
    }

    private function isString(Node $node): bool
    {
        return $this->option->includeStrings() && $node instanceof String_ && false === $this->ignoreString($node);
    }

    private function ignoreNumber(Node $node): bool
    {
        return in_array($node->value, $this->option->getIgnoreNumbers(), true);
    }

    private function ignoreString(Node $node): bool
    {
        return in_array($node->value, $this->option->getIgnoreStrings(), true);
    }

    private function hasSign(Node $node): bool
    {
        return $node->getAttribute('parent') instanceof UnaryMinus
            || $node->getAttribute('parent') instanceof UnaryPlus;
    }

    private function isMinus(Node $node): bool
    {
        return $node instanceof UnaryMinus;
    }

    private function isValidNumeric(Node $node): bool
    {
        return $this->option->includeNumericStrings() &&
        isset($node->value) &&
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
        is_numeric($node->value) &&
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
        false === $this->ignoreString($node);
    }

    /**
     * @param string $name
     * @param string|int $value
     * @return bool
     */
    private function checkNameContainsLanguage(string $name, $value): bool
    {
        foreach ($this->option->checkNaming() as $language) {
            $generatedNumbers = $language->parse($value);

            $regex = '/(?<name>';
            foreach ($generatedNumbers as $word) {
                $regex .= "(?:{$word}[\s_-]*)?";
            }

            $regex .= ')/i';
            preg_match($regex, $name, $matches);

            if (strlen($matches['name']) > 0) {
                return true;
            }
        }

        return false;
    }
}


1			<?php
2
3			namespace Povils\PHPMND\Visitor;
4
5			use PhpParser\Node;
6			use PhpParser\Node\Const_;
7			use PhpParser\Node\Expr\UnaryMinus;
8			use PhpParser\Node\Expr\UnaryPlus;
9			use PhpParser\Node\Scalar;
10			use PhpParser\Node\Scalar\DNumber;
11			use PhpParser\Node\Scalar\LNumber;
12			use PhpParser\Node\Scalar\String_;
13			use PhpParser\NodeTraverser;
14			use PhpParser\NodeVisitorAbstract;
15			use Povils\PHPMND\Console\Option;
16			use Povils\PHPMND\Extension\ArrayAwareExtension;
17			use Povils\PHPMND\Extension\Extension;
18			use Povils\PHPMND\Extension\FunctionAwareExtension;
19			use Povils\PHPMND\FileReport;
20
21			/**
22			* Class DetectorVisitor
23			*
24			* @package Povils\PHPMND
25			*/
26			class DetectorVisitor extends NodeVisitorAbstract
27			{
28			/**
29			* @var FileReport
30			*/
31			private $fileReport;
32
33			/**
34			* @var Option
35			*/
36			private $option;
37
38			public function __construct(FileReport $fileReport, Option $option)
39			{
40			$this->fileReport = $fileReport;
41			$this->option = $option;
42			}
43
44			public function enterNode(Node $node): ?int
45			{
46			if ($this->isIgnoreableConst($node)) {
47			if ($this->checkNameContainsLanguage(
48			$node->name->name,
			0 ignored issues – show Bug introduced 2019-01-28 06:45 UTC by Report Bug Copy Issue Report Accessing `name` on the interface `PhpParser\Node` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
49			$node->value->value ?? 0
			0 ignored issues – show Bug introduced 2019-01-28 06:45 UTC by Report Bug Copy Issue Report Accessing `value` on the interface `PhpParser\Node` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
50			)) {
51			$this->fileReport->addEntry($node->getLine(), $node->value->value);
52			}
53
54			return NodeTraverser::DONT_TRAVERSE_CHILDREN;
55			}
56
57			/** @var LNumber\|DNumber\|String_ $scalar */
58			$scalar = $node;
59			if ($this->hasSign($scalar)) {
60			$node = $scalar->getAttribute('parent');
61			if ($this->isMinus($node) && isset($scalar->value)) {
62			$scalar->value = -$scalar->value;
63			}
64			}
65
66			if ($this->isNumber($scalar) \|\| $this->isString($scalar)) {
67			if ($this->checkNameContainsLanguage(
68			$scalar->getAttribute('parent')->var->name ?? '',
69			$scalar->value
70			)) {
71			$this->fileReport->addEntry($node->getLine(), $scalar->value);
72			}
73
74			foreach ($this->option->getExtensions() as $extension) {
75			$extension->setOption($this->option);
76			if ($extension->extend($node)) {
77			$this->fileReport->addEntry($scalar->getLine(), $scalar->value);
78
79			return null;
80			}
81			}
82			}
83
84			return null;
85			}
86
87			private function isIgnoreableConst(Node $node): bool
88			{
89			return $node instanceof Const_ &&
90			($this->isNumber($node->value) \|\| $this->isString($node->value));
91			}
92
93			private function isNumber(Node $node): bool
94			{
95			$isNumber = (
96			$node instanceof LNumber \|\|
97			$node instanceof DNumber \|\|
98			$this->isValidNumeric($node)
99			);
100
101			return $isNumber && false === $this->ignoreNumber($node);
102			}
103
104			private function isString(Node $node): bool
105			{
106			return $this->option->includeStrings() && $node instanceof String_ && false === $this->ignoreString($node);
107			}
108
109			private function ignoreNumber(Node $node): bool
110			{
111			return in_array($node->value, $this->option->getIgnoreNumbers(), true);
112			}
113
114			private function ignoreString(Node $node): bool
115			{
116			return in_array($node->value, $this->option->getIgnoreStrings(), true);
117			}
118
119			private function hasSign(Node $node): bool
120			{
121			return $node->getAttribute('parent') instanceof UnaryMinus
122			\|\| $node->getAttribute('parent') instanceof UnaryPlus;
123			}
124
125			private function isMinus(Node $node): bool
126			{
127			return $node instanceof UnaryMinus;
128			}
129
130			private function isValidNumeric(Node $node): bool
131			{
132			return $this->option->includeNumericStrings() &&
133			isset($node->value) &&
			0 ignored issues – show Bug introduced 2018-02-26 21:05 UTC by Report Bug Copy Issue Report Accessing `value` on the interface `PhpParser\Node` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
134			is_numeric($node->value) &&
			0 ignored issues – show Bug introduced 2018-03-17 19:09 UTC by Report Bug Copy Issue Report Accessing `value` on the interface `PhpParser\Node` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
135			false === $this->ignoreString($node);
136			}
137
138			/**
139			* @param string $name
140			* @param string\|int $value
141			* @return bool
142			*/
143			private function checkNameContainsLanguage(string $name, $value): bool
144			{
145			foreach ($this->option->checkNaming() as $language) {
146			$generatedNumbers = $language->parse($value);
147
148			$regex = '/(?<name>';
149			foreach ($generatedNumbers as $word) {
150			$regex .= "(?:{$word}[\s_-]*)?";
151			}
152
153			$regex .= ')/i';
154			preg_match($regex, $name, $matches);
155
156			if (strlen($matches['name']) > 0) {
157			return true;
158			}
159			}
160
161			return false;
162			}
163			}
164

povils / phpmnd

Pull Request — master (#86)

DetectorVisitor::enterNode() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like