Dictionary::addAll() - Code Metrics - Inspection of "Fixed Set that now only allows unique values #13" - italolelis/collections - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 801ae3...f2d109 )

by Ítalo

created 2016-06-23 14:23 UTC

Dictionary::addAll() A

↳ Parent: Dictionary

Complexity

Conditions	3
Paths	3

Size

Total Lines	11
Code Lines	6

Duplication

Lines	11
Ratio	100 %

Code Coverage

Tests	4
CRAP Score	3

Importance

Changes	4
Bugs	0	Features	0

Metric	Value
c	4
b	0
f	0
dl	11
loc	11
ccs	4
cts	4
cp	1
rs	9.4285
cc	3
eloc	6
nc	3
nop	1
crap	3

<?php

// Copyright (c) italolelis. All rights reserved. See License.txt in the project root for license information.
namespace Collections;

use Collections\Exception\KeyException;
use Collections\Iterator\MapIterator;
use Collections\Traits\GuardTrait;
use Collections\Traits\StrictKeyedIterableTrait;

/**
 * Represents a collection of keys and values.
 */
class Dictionary extends AbstractConstCollectionArray implements MapInterface, \ArrayAccess
{
    use GuardTrait,
        StrictKeyedIterableTrait;

    public function at($k)
    {
        return $this[$k];
    }

    public function set($key, $value)
    {
        $this->container[$key] = $value;

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function get($index)

    {
        if ($this->containsKey($index) === false) {
            throw new \OutOfBoundsException('No element at position ' . $index);
        }

        return $this->container[$index];
    }

    /**
     * {@inheritdoc}
     */
    public function tryGet($index, $default = null)
    {
        if ($this->containsKey($index) === false) {
            return $default;
        }

        return $this->get($index);
    }
    
    /**
     * {@inheritdoc}
     */
    public function add($key, $value)
    {
        if ($this->containsKey($key)) {
            throw new KeyException('The key ' . $key . ' already exists!');
        }

        $this->set($key, $value);

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function addAll($items)

    {
        $this->validateTraversable($items);

        foreach ($items as $key => $value) {
            if (is_array($value)) {
                $value = new static($value);
            }
            $this->add($key, $value);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function setAll($items)

    {
        $this->validateTraversable($items);

        foreach ($items as $key => $item) {
            if (is_array($item)) {
                $item = new static($item);
            }
            $this->set($key, $item);
        }

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function containsKey($key)
    {
        return array_key_exists($key, $this->container);
    }

    /**
     * {@inheritdoc}
     */
    public function contains($element)
    {
        return in_array($element, $this->container, true);
    }

    /**
     * {@inheritdoc}
     */
    public function remove($element)
    {
        $key = array_search($element, $this->container);

        if (false === $key) {
            throw new \OutOfBoundsException('No element found in the collection ');
        }

        $this->removeKey($key);

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function removeKey($key)

    {
        if ($this->containsKey($key) === false) {
            throw new \OutOfBoundsException('No element at position ' . $key);
        }

        unset($this->container[$key]);

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function offsetExists($offset)
    {
        return isset($this->container[$offset]) || array_key_exists($offset, $this->container);
    }

    /**
     * {@inheritdoc}
     */
    public function offsetGet($offset)
    {
        return $this->get($offset);
    }

    /**
     * {@inheritdoc}
     */
    public function offsetSet($offset, $value)
    {
        if (is_null($offset)) {
            $this->add($offset, $value);
        } else {
            $this->set($offset, $value);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function offsetUnset($offset)
    {
        $this->removeKey($offset);
    }

    /**
     * Gets the collection's iterator
     * @return MapIterator
     */
    public function getIterator()
    {
        return new MapIterator($this->container);
    }

    /**
     * {@inheritDoc}
     * @return $this
     */
    public function concat($iterable)
    {
        $this->validateTraversable($iterable);

        $this->setAll($this->concatRecurse($this, $iterable));

        return $this;
    }
}


1			<?php
2
3			// Copyright (c) italolelis. All rights reserved. See License.txt in the project root for license information.
4			namespace Collections;
5
6			use Collections\Exception\KeyException;
7			use Collections\Iterator\MapIterator;
8			use Collections\Traits\GuardTrait;
9			use Collections\Traits\StrictKeyedIterableTrait;
10
11			/**
12			* Represents a collection of keys and values.
13			*/
14			class Dictionary extends AbstractConstCollectionArray implements MapInterface, \ArrayAccess
15			{
16			use GuardTrait,
17			StrictKeyedIterableTrait;
18
19			public function at($k)
20			{
21			return $this[$k];
22			}
23
24			public function set($key, $value)
25			{
26	22		$this->container[$key] = $value;
27
28	22		return $this;
29			}
30	22
31			/**
32			* {@inheritdoc}
33			*/
34		View Code Duplication	public function get($index)
			0 ignored issues – show Duplication introduced 2016-05-08 11:10 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
35			{
36	8		if ($this->containsKey($index) === false) {
37			throw new \OutOfBoundsException('No element at position ' . $index);
38	8		}
39	1
40			return $this->container[$index];
41			}
42	7
43			/**
44			* {@inheritdoc}
45			*/
46			public function tryGet($index, $default = null)
47			{
48	12		if ($this->containsKey($index) === false) {
49			return $default;
50	12		}
51	1
52			return $this->get($index);
53	12		}
54
55	12		/**
56			* {@inheritdoc}
57			*/
58			public function add($key, $value)
59			{
60			if ($this->containsKey($key)) {
61	4		throw new KeyException('The key ' . $key . ' already exists!');
62			}
63	4
64	1		$this->set($key, $value);
65
66			return $this;
67	3		}
68	3
69	1		/**
70	1		* {@inheritdoc}
71	3		*/
72	3	View Code Duplication	public function addAll($items)
			0 ignored issues – show Duplication introduced 2016-05-08 11:10 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
73	3		{
74			$this->validateTraversable($items);
75
76			foreach ($items as $key => $value) {
77			if (is_array($value)) {
78	17		$value = new static($value);
79			}
80	17		$this->add($key, $value);
81			}
82			}
83
84			/**
85			* {@inheritdoc}
86			*/
87		View Code Duplication	public function setAll($items)
			0 ignored issues – show Duplication introduced 2016-06-23 14:27 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
88			{
89			$this->validateTraversable($items);
90
91			foreach ($items as $key => $item) {
92			if (is_array($item)) {
93			$item = new static($item);
94	1		}
95			$this->set($key, $item);
96	1		}
97	1
98			return $this;
99			}
100
101			/**
102			* {@inheritdoc}
103			*/
104			public function containsKey($key)
105	2		{
106			return array_key_exists($key, $this->container);
107	2		}
108	1
109			/**
110	1		* {@inheritdoc}
111			*/
112			public function contains($element)
113			{
114			return in_array($element, $this->container, true);
115			}
116	1
117			/**
118	1		* {@inheritdoc}
119			*/
120			public function remove($element)
121			{
122			$key = array_search($element, $this->container);
123
124	5		if (false === $key) {
125			throw new \OutOfBoundsException('No element found in the collection ');
126	5		}
127
128			$this->removeKey($key);
129
130			return $this;
131			}
132	10
133			/**
134	10		* {@inheritdoc}
135			*/
136		View Code Duplication	public function removeKey($key)
			0 ignored issues – show Duplication introduced 2016-06-10 09:11 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
137	10		{
138			if ($this->containsKey($key) === false) {
139	10		throw new \OutOfBoundsException('No element at position ' . $key);
140			}
141
142			unset($this->container[$key]);
143
144	1		return $this;
145			}
146	1
147	1		/**
148			* {@inheritdoc}
149			*/
150			public function offsetExists($offset)
151			{
152			return isset($this->container[$offset]) \|\| array_key_exists($offset, $this->container);
153	10		}
154
155	10		/**
156			* {@inheritdoc}
157			*/
158			public function offsetGet($offset)
159			{
160			return $this->get($offset);
161			}
162
163			/**
164			* {@inheritdoc}
165			*/
166			public function offsetSet($offset, $value)
167			{
168			if (is_null($offset)) {
169			$this->add($offset, $value);
170			} else {
171			$this->set($offset, $value);
172			}
173			}
174
175			/**
176			* {@inheritdoc}
177			*/
178			public function offsetUnset($offset)
179			{
180			$this->removeKey($offset);
181			}
182
183			/**
184			* Gets the collection's iterator
185			* @return MapIterator
186			*/
187			public function getIterator()
188			{
189			return new MapIterator($this->container);
190			}
191
192			/**
193			* {@inheritDoc}
194			* @return $this
195			*/
196			public function concat($iterable)
197			{
198			$this->validateTraversable($iterable);
199
200			$this->setAll($this->concatRecurse($this, $iterable));
201
202			return $this;
203			}
204			}
205

italolelis / collections

Push — master ( 801ae3...f2d109 )

Dictionary::addAll() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like