itertools\SliceIterator

Complexity

Total Complexity

9

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	key()	0	8	2
A	next()	0	6	1
A	valid()	0	14	4
A	rewind()	0	6	1

<?php

namespace itertools;

use IteratorIterator;


/**
 * Iterator equivalent of [array_slice](http://be1.php.net/manual/en/function.array-slice.php).
 * Example:
 *     $lines = new SliceIterator(new FileLineIterator('file.txt'), 0, 1000); // will iterate the first 1000 lines of the file
 */
class SliceIterator extends IteratorIterator
{
	const PRESERVE_KEYS = true;
	const DONT_PRESERVE_KEYS = false;

	protected $offset;
	protected $length;
	protected $preserveKeys;
	protected $innerIterationCount;
	protected $outerIterationCount;

	public function __construct($iterable, $offset, $length = INF, $preserveKeys = false)
	{
		parent::__construct(IterUtil::asTraversable($iterable));
		$this->offset = $offset;
		$this->length = $length;
		$this->preserveKeys = $preserveKeys;
	}

	public function key()
	{
		if($this->preserveKeys) {
			return parent::key();
		} else {
			return $this->outerIterationCount;
		}
	}

	public function next()
	{
		$this->innerIterationCount += 1;
		$this->outerIterationCount += 1;
		parent::next();
	}

	public function valid()
	{
		while($this->innerIterationCount < $this->offset) {
			if(!parent::valid()) {
				return false;
			}
			$this->innerIterationCount += 1;
			parent::next();

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

		}
		if($this->innerIterationCount >= $this->offset + $this->length) {
			return false;
		}
		return parent::valid();
	}

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