UnreachableVisitor::enterBlock() - Code Metrics - Inspection of "[ControlFlowGraph] Initial" - ovr/phpsa - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#293)

by Дмитрий

created 2017-01-14 13:54 UTC

UnreachableVisitor::enterBlock() B

↳ Parent: UnreachableVisitor

Complexity

Conditions	6
Paths	5

Size

Total Lines	17
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes

Metric	Value
cc	6
eloc	9
nc	5
nop	1
dl	0
loc	17
ccs	0
cts	13
cp	0
crap	42
rs	8.8571
c	0
b	0
f	0

<?php
/**
 * @author Patsura Dmitry https://github.com/ovr <[email protected]>
 */

namespace PHPSA\ControlFlow\Visitor;

use PHPSA\ControlFlow\Block;
use PHPSA\ControlFlow\Node\AbstractNode;

/**
 *
 * function test($a) {
 *     return $a;
 *
 *     $a = 1; // Unreachable node!
 * }
 *
 */
class UnreachableVisitor extends AbstractVisitor
{
    /**
     * @param Block $block
     */
    public function enterBlock(Block $block)
    {
        $childrens = $block->getChildrens();
        if ($childrens) {

            $childrensCount = count($childrens);
            if ($childrensCount <= 1) {
                return;
            }

            foreach ($childrens as $index => $children) {
                // Check that exit node is not the latest
                if ($children->willExit() && ($index + 1) != $childrensCount) {
                    echo 'Unreacheable block ' . $block->getId() . PHP_EOL;
                }
            }
        }
    }

    /**
     * @param Block $block
     */
    public function leaveBlock(Block $block)
    {
    }

    /**
     * @param AbstractNode $block
     */
    public function enterNode(AbstractNode $block)
    {
    }
}


1			<?php
2			/**
3			* @author Patsura Dmitry https://github.com/ovr <[email protected]>
4			*/
5
6			namespace PHPSA\ControlFlow\Visitor;
7
8			use PHPSA\ControlFlow\Block;
9			use PHPSA\ControlFlow\Node\AbstractNode;
10
11			/**
12			*
13			* function test($a) {
14			* return $a;
15			*
16			* $a = 1; // Unreachable node!
17			* }
18			*
19			*/
20			class UnreachableVisitor extends AbstractVisitor
21			{
22			/**
23			* @param Block $block
24			*/
25			public function enterBlock(Block $block)
26			{
27			$childrens = $block->getChildrens();
28			if ($childrens) {
			0 ignored issues – show Bug Best Practice introduced 2017-01-02 09:12 UTC by Report Bug Copy Issue Report The expression `$childrens` of type `PHPSA\ControlFlow\Node\AbstractNode[]` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
29			$childrensCount = count($childrens);
30			if ($childrensCount <= 1) {
31			return;
32			}
33
34			foreach ($childrens as $index => $children) {
35			// Check that exit node is not the latest
36			if ($children->willExit() && ($index + 1) != $childrensCount) {
37			echo 'Unreacheable block ' . $block->getId() . PHP_EOL;
38			}
39			}
40			}
41			}
42
43			/**
44			* @param Block $block
45			*/
46			public function leaveBlock(Block $block)
47			{
48			}
49
50			/**
51			* @param AbstractNode $block
52			*/
53			public function enterNode(AbstractNode $block)
54			{
55			}
56			}
57

ovr / phpsa

Pull Request — master (#293)

UnreachableVisitor::enterBlock() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like