Gielfeldt\Iterators\RepeatIterator

Complexity

Total Complexity

11

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	2
A	count()	0	7	2
A	getIterationCount()	0	4	1
A	getCurrentIteration()	0	4	1
A	rewind()	0	5	1
A	next()	0	8	2
A	valid()	0	4	2

<?php

namespace Gielfeldt\Iterators;

class RepeatIterator extends TraversableIterator implements \Countable
{
    protected $count;
    protected $currentIteration = 0;
    protected $innerCount;

    public function __construct(\Traversable $iterator, int $count)
    {
        if ($count < 0) {
            throw new \InvalidArgumentException('Count cannot be less than 0.');
        }

        $this->count = $count;
        parent::__construct($iterator);
    }

    public function count()
    {
        if (!isset($this->innerCount)) {
            $this->innerCount = count(new CountableIterator($this->getInnerIterator()));
        }
        return $this->innerCount * $this->count;
    }

    public function getIterationCount()
    {
        return $this->count;
    }

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

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

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

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

            $this->currentIteration++;
            parent::rewind();

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

        }
    }

    public function valid()
    {
        return $this->currentIteration < $this->count && parent::valid();
    }
}