Everlution\Navigation\FilteredContainer

Complexity

Total Complexity

4

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getItems()	0	9	2
A	get()	0	4	1

<?php

declare(strict_types=1);

namespace Everlution\Navigation;

use Everlution\Navigation\Filter\ChainFilter;
use Everlution\Navigation\Item\ItemInterface;

/**
 * Class FilteredContainer.
 *
 * @author Ivan Barlog <ivan.barlog@everlution.sk>
 */
class FilteredContainer implements ContainerInterface
{
    /** @var FilteredContainerInterface */
    private $container;
    /** @var ItemInterface[] */
    private $items;

    public function __construct(FilteredContainerInterface $container)
    {
        $this->container = $container;
    }

    /**
     * @return ItemInterface[]
     */
    public function getItems(): array
    {
        if (!$this->items) {

The expression $this->items of type Everlution\Navigation\Item\ItemInterface[] 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.

            $filter = new ChainFilter($this->container->getFilters());
            $this->items = $filter->filterItems($this->container)->getItems();
        }

        return $this->items;
    }

    public function get(string $name): ItemInterface
    {
        return $this->items[$name];
    }
}