AbstractList::offsetSet() - Code Metrics - Inspection of "Merge branch 'feature/small-improvements'" - spaark/composite-utils - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 79f24a...4b5497 )

by Emily

created 2017-03-19 14:34 UTC

AbstractList::offsetSet() A

↳ Parent: AbstractList

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2.032

Importance

Changes

Metric	Value
dl	0
loc	11
ccs	4
cts	5
cp	0.8
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	2
crap	2.032

<?php
/**
 * This file is part of the Composite Utils package.
 *
 * (c) Emily Shepherd <[email protected]>
 *
 * For the full copyright and license information, please view the
 * LICENSE.md file that was distributed with this source code.
 *
 * @package spaark/composite-utils
 * @author Emily Shepherd <[email protected]>
 * @license MIT
 */

namespace Spaark\CompositeUtils\Model\Collection\ListCollection;

use Spaark\CompositeUtils\Model\Collection\AbstractCollection;

/**
 * Represents an abstract collection which acts as a list of items
 */
abstract class AbstractList
    extends AbstractCollection

    implements ArrayList

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

    /**
     * {@inheritDoc}
     */
    public function set(int $index, $value)
    {
        $this->splice($offset, 0, [$value]);

    }

    /**
     * {@inheritDoc}
     */
    public function offsetUnset($index)
    {
        $this->remove($index);
    }

    /**
     * {@inheritDoc}
     */
    public function offsetExists($index)
    {
        return $index < $this->size();
    }

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

    /**
     * {@inheritDoc}
     */
    public function remove(int $index)
    {
        $this->splice($index, 1);
    }
}



1		<?php
2		/**
3		* This file is part of the Composite Utils package.
4		*
5		* (c) Emily Shepherd <[email protected]>
6		*
7		* For the full copyright and license information, please view the
8		* LICENSE.md file that was distributed with this source code.
9		*
10		* @package spaark/composite-utils
11		* @author Emily Shepherd <[email protected]>
12		* @license MIT
13		*/
14
15		namespace Spaark\CompositeUtils\Model\Collection\ListCollection;
16
17		use Spaark\CompositeUtils\Model\Collection\AbstractCollection;
18
19		/**
20		* Represents an abstract collection which acts as a list of items
21		*/
22		abstract class AbstractList
23		extends AbstractCollection
		0 ignored issues – show Coding Style introduced 2017-02-16 18:58 UTC by Report Bug Copy Issue Report The extends keyword must be on the same line as the class name Loading history... Coding Style introduced 2017-02-16 18:58 UTC by Report Bug Copy Issue Report Expected 0 spaces between "AbstractCollection" and comma; 1 found Loading history...
24		implements ArrayList
		0 ignored issues – show Coding Style introduced 2017-03-19 14:36 UTC by Report Bug Copy Issue Report The implements keyword must be on the same line as the class name Loading history...
25		{
26		/**
27		* {@inheritDoc}
28		*/
29	18	public function offsetSet($offset, $value)
30		{
31	18	if ($offset === null)
32		{
33	18	$this->add($value);
34		}
35		else
36		{
37		$this->set($offset, $value);
38		}
39	18	}
40
41		/**
42		* {@inheritDoc}
43		*/
44		public function set(int $index, $value)
45		{
46		$this->splice($offset, 0, [$value]);
		0 ignored issues – show Bug introduced 2017-03-19 14:36 UTC by Report Bug Copy Issue Report The variable `$offset` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
47		}
48
49		/**
50		* {@inheritDoc}
51		*/
52		public function offsetUnset($index)
53		{
54		$this->remove($index);
55		}
56
57		/**
58		* {@inheritDoc}
59		*/
60		public function offsetExists($index)
61		{
62		return $index < $this->size();
63		}
64
65		/**
66		* {@inheritDoc}
67		*/
68	15	public function offsetGet($index)
69		{
70	15	return $this->get($index);
71		}
72
73		/**
74		* {@inheritDoc}
75		*/
76	2	public function remove(int $index)
77		{
78	2	$this->splice($index, 1);
79	2	}
80		}
81
82

spaark / composite-utils

Push — master ( 79f24a...4b5497 )

AbstractList::offsetSet() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like