Spaark\CompositeUtils\Model\Collection\ListCollection\AbstractList

Complexity

Total Complexity

7

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

50%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	offsetSet()	0	11	2
A	set()	0	4	1
A	offsetUnset()	0	4	1
A	offsetExists()	0	4	1
A	offsetGet()	0	4	1
A	remove()	0	4	1

<?php
/**
 * This file is part of the Composite Utils package.
 *
 * (c) Emily Shepherd <emily@emilyshepherd.me>
 *
 * 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 <emily@emilyshepherd.me>
 * @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

The extends keyword must be on the same line as the class name

Expected 0 spaces between "AbstractCollection" and comma; 1 found

    implements ArrayList

The implements keyword must be on the same line as the class name

{
    /**
     * {@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]);

The variable $offset does not exist. Did you forget to declare it?

    }

    /**
     * {@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);
    }
}