Railt\Compiler\Iterator\LookaheadIterator

Complexity

Total Complexity

11

Size/Duplication

Total Lines	142
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	current()	0	4	1
A	key()	0	4	1
A	rewind()	0	7	1
A	getInnerIterator()	0	4	1
A	next()	0	14	2
A	valid()	0	4	1
A	hasNext()	0	4	1
A	getNext()	0	4	1
A	getNextKey()	0	4	1

<?php
/**
 * This file is part of Railt package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
declare(strict_types=1);

namespace Railt\Compiler\Iterator;

/**
 * Class LookaheadIterator
 */
class LookaheadIterator extends \IteratorIterator
{
    /**
     * Current iterator.
     *
     * @var \Iterator
     */
    protected $_iterator;

    /**
     * Current key.
     *
     * @var mixed
     */
    protected $_key = 0;

    /**
     * Current value.
     *
     * @var mixed
     */
    protected $_current;

    /**
     * Whether the current element is valid or not.
     *
     * @var bool
     */
    protected $_valid = false;

    /**
     * LookaheadIterator constructor.
     * @param \Traversable $iterator
     */
    public function __construct(\Traversable $iterator)
    {
        $this->_iterator = $iterator;

$iterator is of type object<Traversable>, but the property $_iterator was declared to be of type object<Iterator>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    /**
     * Return the current element.
     *
     * @return mixed
     */
    public function current()
    {
        return $this->_current;
    }

    /**
     * Return the key of the current element.
     *
     * @return mixed
     */
    public function key()
    {
        return $this->_key;
    }

    /**
     * Rewind the iterator to the first element.
     *
     * @return void
     */
    public function rewind()
    {
        $out = $this->getInnerIterator()->rewind();

Are you sure the assignment to $out is correct as $this->getInnerIterator()->rewind() (which targets Iterator::rewind()) seems to always return null.

        $this->next();

        return $out;
    }

    /**
     * Get inner iterator.
     *
     * @return \Iterator
     */
    public function getInnerIterator()
    {
        return $this->_iterator;
    }

    /**
     * Move forward to next element.
     *
     * @return void
     */
    public function next()
    {
        $innerIterator = $this->getInnerIterator();
        $this->_valid  = $innerIterator->valid();

        if (false === $this->_valid) {
            return;
        }

        $this->_key     = $innerIterator->key();
        $this->_current = $innerIterator->current();

        return $innerIterator->next();
    }

    /**
     * Check if current position is valid.
     *
     * @return bool
     */
    public function valid()
    {
        return $this->_valid;
    }

    /**
     * Check whether there is a next element.
     *
     * @return bool
     */
    public function hasNext()
    {
        return $this->getInnerIterator()->valid();
    }

    /**
     * Get next value.
     *
     * @return mixed
     */
    public function getNext()
    {
        return $this->getInnerIterator()->current();
    }

    /**
     * Get next key.
     *
     * @return mixed
     */
    public function getNextKey()
    {
        return $this->getInnerIterator()->key();
    }
}