Knp\FriendlyContexts\Table\NodeCollection

Complexity

Total Complexity

16

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	addNode()	0	5	1
C	atPosition()	0	23	13
A	search()	0	6	1
A	getNodes()	0	8	1

<?php

namespace Knp\FriendlyContexts\Table;

use Knp\FriendlyContexts\Table\Node;

class NodeCollection
{
    private $nodes = [];

    public function getNodes()
    {
        return array_reduce(
            $this->nodes,
            function ($previous, $line) {
                return array_merge($previous, $line);
            },
            []
        );
    }

    public function addNode(Node $node, $line, $column)
    {
        $this->nodes[$line][$column] = $node;

        return $this;
    }

    public function atPosition($line = null, $column = null)
    {
        switch (true) {
            case null === $line && null === $column:
                return $this->getNodes();
            case null !== $line && null === $column:
                return array_key_exists($line, $this->nodes)
                    ? $this->nodes[$line]
                    : []
                ;
            case null === $line && null !== $column:
                $nodes = [];
                foreach ($this->nodes as $i => $nodes) {
                    if (array_key_exists($column, $nodes)) {
                        $nodes[$i] = $nodes[$column];
                    }
                }
                return $nodes;
            case null !== $line && null !== $column:
                $nodes = $this->atPosition($line);
                return array_key_exists($column, $nodes)

It seems like $nodes can also be of type null; however, parameter $search of array_key_exists() does only seem to accept array, maybe add an additional type check?

                    ? $nodes[$column]
                    : null
                ;
        }
    }

    public function search($content)
    {
        return array_filter(
            $this->getNodes(),
            function ($e) use ($content) {
                return $e->getContent() === $content;
            }
        );
    }
}