Graze\DataStructure\Collection\ImmutableCollection

Complexity

Total Complexity

7

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	2
A	add()	0	7	1
A	filter()	0	4	1
A	sort()	0	7	1
A	sortOn()	0	4	1
A	addItem()	0	4	1

<?php
/*
 * This file is part of Graze DataStructure
 *
 * Copyright (c) 2014 Nature Delivered Ltd. <http://graze.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @see  http://github.com/graze/data-structure/blob/master/LICENSE
 * @link http://github.com/graze/data-structure
 */

namespace Graze\DataStructure\Collection;

use Graze\Sort;

class ImmutableCollection extends Collection
{
    /**
     * @param mixed[] $items
     */
    public function __construct(array $items = [])
    {
        parent::__construct([]);

        foreach ($items as $item) {
            $this->addItem($item);
        }
    }

    /**
     * @param mixed $value
     *
     * @return ImmutableCollection
     */
    public function add($value)
    {
        $items = $this->items;
        $items[] = $value;

        return new self($items);
    }

    /**
     * @param callable $fn
     *
     * @return ImmutableCollection
     */
    public function filter(callable $fn)
    {
        return new self(array_filter($this->items, $fn));
    }

    /**
     * @param callable $fn
     *
     * @return ImmutableCollection
     */
    public function sort(callable $fn)
    {
        $items = $this->items;
        usort($items, $fn);

        return new self($items);
    }

    /**
     * @param callable $fn
     * @param int      $order
     *
     * @return ImmutableCollection
     */
    public function sortOn(callable $fn, $order = Sort\ASC)
    {
        return new self(Sort\comparison($this->items, $fn, $order));
    }

    /**
     * @param mixed $value
     */
    protected function addItem($value)
    {
        parent::add($value);

It seems like you call parent on a different method (add() instead of addItem()). Are you sure this is correct? If so, you might want to change this to $this->add().

    }
}