Gielfeldt\Iterators\InfiniteIterator

Complexity

Total Complexity

7

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	2
A	getCurrentIteration()	0	4	1
A	rewind()	0	5	1
A	valid()	0	9	3

<?php

namespace Gielfeldt\Iterators;

class InfiniteIterator extends IteratorIterator
{
    private $endCondition;
    private $currentIteration;

    public function __construct(\Traversable $iterator, callable $endCondition = null)
    {
        $this->endCondition = $endCondition ? \Closure::fromCallable($endCondition) : null;
        parent::__construct($iterator);
    }

    public function getCurrentIteration()
    {
        return $this->currentIteration;
    }

    public function rewind()
    {
        $this->currentIteration = 0;
        parent::rewind();
    }

    public function valid()
    {
        if (!parent::valid()) {
            parent::rewind();

It seems like you call parent on a different method (rewind() instead of valid()). Are you sure this is correct? If so, you might want to change this to $this->rewind().

            $this->currentIteration++;
        }

        return $this->endCondition ? !($this->endCondition)($this) : parent::valid();
    }

}