Knp\FriendlyContexts\Table\NodesBuilder

Complexity

Total Complexity

3

Size/Duplication

Total Lines	17
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	build()	0	15	3

<?php

namespace Knp\FriendlyContexts\Table;

use Knp\FriendlyContexts\Table\Node;
use Knp\FriendlyContexts\Table\NodeCollection;

class NodesBuilder
{
    public function build(array $table)
    {
        $collection = new NodeCollection;
        foreach ($table as $line => $columns) {
            foreach ($columns as $column => $content) {
                $collection->addNode($node = new Node, $line, $column);
                $node->setContent($content);
                $node->setTop($collection->atPosition($line - 1, $column));

It seems like $collection->atPosition($line - 1, $column) can also be of type array; however, parameter $top of Knp\FriendlyContexts\Table\Node::setTop() does only seem to accept Knp\FriendlyContexts\Table\Node|null, maybe add an additional type check?

                $node->setBottom($collection->atPosition($line + 1, $column));

It seems like $collection->atPosition($line + 1, $column) can also be of type array; however, parameter $bottom of Knp\FriendlyContexts\Table\Node::setBottom() does only seem to accept Knp\FriendlyContexts\Table\Node|null, maybe add an additional type check?

                $node->setLeft($collection->atPosition($line, $column - 1));

It seems like $collection->atPosition($line, $column - 1) can also be of type array; however, parameter $left of Knp\FriendlyContexts\Table\Node::setLeft() does only seem to accept Knp\FriendlyContexts\Table\Node|null, maybe add an additional type check?

                $node->setLeft($collection->atPosition($line, $column + 1));
            }
        }

        return $collection;
    }
}