Mbh\Iterator\ConcatIterator

Complexity

Total Complexity

11

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	count()	0	3	1
B	__construct()	0	24	6
A	getIteratorByIndex()	0	12	3
A	toArray()	0	3	1

<?php namespace Mbh\Iterator;

/**
 * MBHFramework
 *
 * @link      https://github.com/MBHFramework/mbh-framework
 * @copyright Copyright (c) 2017 Ulises Jeremias Cornejo Fandos
 * @license   https://github.com/MBHFramework/mbh-framework/blob/master/LICENSE (MIT License)
 */

use ArrayAccess;
use AppendIterator;
use JsonSerializable;
use Countable;
use Iterator;
use RuntimeException;
use InvalidArgumentException;

/**
 * Iterator to allow multiple iterators to be concatenated
 */

class ConcatIterator extends AppendIterator implements Countable, Iterator
{
    const INVALID_INDEX = 'Index invalid or out of range';

    /** @var int $count Fast-lookup count for full set of iterators */
    public $count = 0;

    /**
     * Build an iterator over multiple iterators
     * Unlike a LimitIterator, the $end defines the last index, not the count
     *
     * @param Iterator $iterator,... Concat iterators in order
     */
    public function __construct(...$args)
    {
        parent::__construct();
        foreach ($args as $i => $iterator) {
            if (
                $iterator instanceof ArrayAccess &&
                $iterator instanceof Countable
            ) {
                // Unroll other ConcatIterators, so we avoid deep iterator stacks
                if ($iterator instanceof self) {
                    foreach ($iterator->getArrayIterator() as $innerIt) {

The method getArrayIterator() does not exist on Countable. It seems like you code against a sub-type of Countable such as Mbh\Iterator\ConcatIterator.

The method getArrayIterator() does not exist on ArrayAccess.

                        $this->append($innerIt);
                    }
                } else {
                    $this->append($iterator);

$iterator of type ArrayAccess&Countable is incompatible with the type Iterator expected by parameter $iterator of AppendIterator::append().

                }

                $this->count += count($iterator);
            } else {
                throw new InvalidArgumentException(
                    'Argument ' . $i .
                    ' passed to ' . __METHOD__ .
                    ' must be of type ArrayAccess, Countable, and Traversable. ' .
                    gettype($iterator) . ' given.'
                );
            }
        }
    }

    /**
     * Countable
     */
    public function count(): int
    {
        return $this->count;
    }

    public function toArray(): array
    {
        return iterator_to_array($this, false);
    }

    /**
     * Find which of the inner iterators an index corresponds to
     *
     * @param int $index
     * @return [ArrayAccess, int] The iterator and interior index
     */

The doc comment [ArrayAccess, int] at position 0 could not be parsed: Unknown type name '[' at position 0 in [ArrayAccess, int].

    protected function getIteratorByIndex($index = 0)
    {
        $runningCount = 0;
        foreach ($this->getArrayIterator() as $innerIt) {
            $count = count($innerIt);
            if ($index < $runningCount + $count) {
                return [$innerIt, $index - $runningCount];
            }
            $runningCount += $count;
        }

        return null;
    }
}