Mbh\Tree\Builder

Complexity

Total Complexity

6

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	2
A	popNode()	0	3	1
A	emptyStack()	0	4	1
A	pushNode()	0	4	1
A	getNodeAt()	0	3	1

<?php namespace Mbh\Tree;

/**
 * MBHFramework
 *
 * @link      https://github.com/MBHFramework/mbh-framework
 * @copyright Copyright (c) 2017 Ulises Jeremias Cornejo Fandos
 * @license   https://github.com/MBHFramework/mbh-framework/blob/master/LICENSE (MIT License)
 */

use Mbh\Tree\Interfaces\Node as NodeInterface;
use Mbh\Tree\Interfaces\Builder as BuilderInterface;

class Builder
{
    use Traits\Builder;

    /**
     * @var NodeInterface[]
     */
    protected $nodeStack = [];

    /**
     * @param NodeInterface $node
     */
    public function __construct(NodeInterface $node = null)
    {
        $this->setNode($node ?: $this->nodeInstanceByValue());

It seems like $node ?: $this->nodeInstanceByValue() can also be of type Mbh\Tree\Traits\Node; however, parameter $node of Mbh\Tree\Builder::setNode() does only seem to accept Mbh\Tree\Interfaces\Node, maybe add an additional type check?

    }

    protected function getNodeAt(int $index): NodeInterface
    {
        return $this->nodeStack($index);

The method nodeStack() does not exist on Mbh\Tree\Builder.

    }

    protected function emptyStack()
    {
        $this->nodeStack = [];
        return $this;
    }

    protected function pushNode(NodeInterface $node)
    {
        array_push($this->nodeStack, $node);
        return $this;
    }

    protected function popNode()
    {
        return array_pop($this->nodeStack);
    }
}