Spaark\CompositeUtils\Model\Collection\FixedList

Complexity

Total Complexity

12

Size/Duplication

Total Lines	111
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

83.87%

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	get()	0	4	1
A	set()	0	4	1
A	remove()	0	4	1
A	getIterator()	0	4	1
A	size()	0	4	1
A	add()	0	4	1
A	splice()	0	14	2
A	resize()	0	4	1
A	getCurrentPosition()	0	4	1
A	resizeToFull()	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;

use IteratorIterator;
use SplFixedArray;

/**
 * Represents an List stored in a PHP array
 */
class FixedList extends AbstractList
{
    /**
     * @var ValueType[]
     */
    protected $data;

    /**
     * @var int
     */
    protected $pointer = 0;

    public function __construct(int $size = 0)
    {
        $this->data = new SplFixedArray($size);

It seems like new \SplFixedArray($size) of type object<SplFixedArray> is incompatible with the declared type array<integer,object<Spaark\CompositeUtils\Model\Collection\ValueType>> of property $data.

    }

    /**
     * {@inheritDoc}
     */
    public function add($item)
    {
        $this->data[$this->pointer++] = $item;
    }

    /**
     * {@inheritDoc}
     */
    public function get(int $index)
    {
        return $this->data[$index];
    }

    /**
     * {@inheritDoc}
     */
    public function splice
    (
        int $offset,
        ?int $length = null,
        array $replacement = []
    )
    {
        $end = $offset + $length;

        for ($i = $offset, $j = 0; $i < $end; $i++, $j++)
        {
            $this->data[$i] = $replacement[$j] ?? null;
        }
    }

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

    /**
     * {@inheritDoc}
     */
    public function remove(int $item)
    {
        $this->set($item, null);

null is of type null, but the function expects a object<Spaark\CompositeUtils\Model\Collection\ValueType>.

    }

    /**
     * {@inheritDoc}
     */
    public function getIterator()
    {
        return new IteratorIterator($this->data);
    }

    /**
     * {@inheritDoc}
     */
    public function size() : int
    {
        return count($this->data);
    }

    /**
     * Resizes the FixedList, throwing away any unused elements
     *
     * @param int $size The new size
     */
    public function resize(int $size)
    {
        $this->data->setSize($size);

The method setSize cannot be called on $this->data (of type array<integer,object<Spaark\CompositeUtils\Model\Collection\ValueType>>).

    }

    /**
     * Returns the current pointer position
     *
     * @return int
     */
    public function getCurrentPosition() : int
    {
        return $this->pointer;
    }

    /**
     * Resizes to the current pointer
     */
    public function resizeToFull()
    {
        $this->resize($this->getCurrentPosition());
    }
}