PHPSA\ControlFlow\BlockTraverser::traverse()

Complexity

Conditions	6
Paths	8

Size

Total Lines	23
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes

0

<?php
    public function traverse(ControlFlowGraph $cfg)
    {
        $rootBlock = $cfg->getRoot();

        /** @var AbstractVisitor $visitor */
        foreach ($this->visitors as $visitor) {
            $visitor->enterBlock($rootBlock);
        }

        $children = $rootBlock->getChildren();
        if ($children) {

The expression $children 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.

            foreach ($children as $child) {
                foreach ($this->visitors as $visitor) {
                    $visitor->enterNode($child);
                }
            }
        }

        /** @var AbstractVisitor $visitor */
        foreach ($this->visitors as $visitor) {
            $visitor->leaveBlock($rootBlock);
        }
    }
}