Graze\DataStructure\Container\ImmutableContainer

Complexity

Total Complexity

7

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	2
A	set()	0	7	1
A	remove()	0	11	2
A	setParameter()	0	4	1
A	removeParameter()	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\Container;

class ImmutableContainer extends Container
{
    /**
     * @param array $params
     */
    public function __construct(array $params = [])
    {
        parent::__construct([]);

        foreach ($params as $key => $value) {
            $this->setParameter($key, $value);
        }
    }

    /**
     * @param string $key
     * @param mixed  $value
     *
     * @return ContainerInterface
     */
    public function set($key, $value)
    {
        $cont = clone $this;
        $cont->setParameter($key, $value);

        return $cont;
    }

    /**
     * @param string $key
     *
     * @return ContainerInterface
     */
    public function remove($key)
    {
        if ($this->has($key)) {
            $cont = clone $this;
            $cont->removeParameter($key);

            return $cont;
        }

        return $this;
    }

    /**
     * @param string $key
     * @param mixed  $value
     *
     * @return ContainerInterface
     */
    protected function setParameter($key, $value)
    {
        return parent::set($key, $value);

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

    }

    /**
     * @param string $key
     *
     * @return ContainerInterface
     */
    protected function removeParameter($key)
    {
        return parent::remove($key);

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

    }
}