RemoveNullNode::__construct() - Code Metrics - Inspection of "Let Modifiers use a customizable traverser." - loophp/phptree - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( f2e6bb...d5f140 )

by Pol

created 2020-02-26 20:32 UTC

RemoveNullNode::__construct() A

↳ Parent: RemoveNullNode

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

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

<?php

declare(strict_types=1);

namespace loophp\phptree\Modifier;

use loophp\phptree\Node\NodeInterface;
use loophp\phptree\Traverser\PostOrder;
use loophp\phptree\Traverser\TraverserInterface;

/**
 * Class RemoveNullNode.
 */
class RemoveNullNode implements ModifierInterface
{
    /**
     * @var \loophp\phptree\Traverser\PreOrder|\loophp\phptree\Traverser\TraverserInterface
     */
    private $traverser;

    /**
     * RemoveNullNode constructor.
     *
     * @param \loophp\phptree\Traverser\TraverserInterface|null $traverser
     */
    public function __construct(?TraverserInterface $traverser = null)
    {
        $this->traverser = $traverser ?? new PostOrder();
    }

    /**
     * {@inheritdoc}
     */
    public function modify(NodeInterface $tree): NodeInterface
    {
        /** @var \loophp\phptree\Node\ValueNodeInterface $item */
        foreach ($this->traverser->traverse($tree) as $item) {
            if (null === $parent = $item->getParent()) {
                continue;
            }

            if (false === $item->isLeaf()) {
                continue;
            }

            if (null !== $item->getValue()) {
                continue;
            }

            $parent->remove($item);
            $this->modify($parent);
        }

        return $tree;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace loophp\phptree\Modifier;
6
7		use loophp\phptree\Node\NodeInterface;
8		use loophp\phptree\Traverser\PostOrder;
9		use loophp\phptree\Traverser\TraverserInterface;
10
11		/**
12		* Class RemoveNullNode.
13		*/
14		class RemoveNullNode implements ModifierInterface
15		{
16		/**
17		* @var \loophp\phptree\Traverser\PreOrder\|\loophp\phptree\Traverser\TraverserInterface
18		*/
19		private $traverser;
20
21		/**
22		* RemoveNullNode constructor.
23		*
24		* @param \loophp\phptree\Traverser\TraverserInterface\|null $traverser
25		*/
26	10	public function __construct(?TraverserInterface $traverser = null)
27		{
28	10	$this->traverser = $traverser ?? new PostOrder();
29	10	}
30
31		/**
32		* {@inheritdoc}
33		*/
34	7	public function modify(NodeInterface $tree): NodeInterface
35		{
36		/** @var \loophp\phptree\Node\ValueNodeInterface $item */
37	7	foreach ($this->traverser->traverse($tree) as $item) {
38	7	if (null === $parent = $item->getParent()) {
39	7	continue;
40		}
41
42	7	if (false === $item->isLeaf()) {
43	4	continue;
44		}
45
46	7	if (null !== $item->getValue()) {
47	7	continue;
48		}
49
50	2	$parent->remove($item);
51	2	$this->modify($parent);
52		}
53
54	7	return $tree;
55		}
56		}
57

loophp / phptree

Push — master ( f2e6bb...d5f140 )

RemoveNullNode::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like