Jmw\Collection\Lists\ReverseListIterator

Complexity

Total Complexity

6

Size/Duplication

Total Lines	76
Duplicated Lines	31.58 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	next()	12	12	2
A	hasNext()	0	4	1
A	remove()	12	12	2

<?php
namespace Jmw\Collection\Lists;

use Jmw\Collection\IteratorInterface;
use Jmw\Collection\Exception\NoSuchElementException;
use Jmw\Collection\Exception\IllegalStateException;
/**
 * A ReverseListIterator starts at the end of a list and works its way to the beginning
 * @author john
 *
 */
class ReverseListIterator implements IteratorInterface
{

	/**
	 * 
	 * @var ListInterface
	 */
	protected $list;

	/**
	 * 
	 * @var int
	 */
	protected $index;
	
	/**
	 * 
	 * @var boolean
	 */
	protected $canRemove;

	/**
	 * Construct a ReverseListIterator
	 * @param ListInterface $list
	 */
	public function __construct(ListInterface $list)
	{
		$this->index = $list->size();
		$this->list = $list;
	}

	/**
	 * Returns true if this list iterator has more elements when traversing the list in the backward direction.
	 * @return boolean
	 */
	public function next()

This method seems to be duplicated in your project.

	{
		if ($this->hasNext())
		{
			$this->canRemove = true;
			return $this->list->get(--$this->index);
		}
		else
		{
			throw new NoSuchElementException("Could not find element at index {$this->index}");
		}
	}

	/**
	 * Returns the next element in the list and advances the cursor position.
	 * @return multitype
	 * @throws NoSuchElementException
	 */
	public function hasNext()
	{
		return $this->index > 0;
	}

	/**
	 * Removes from the list the last element that was returned by next()
	 * @return void
	 * @throws IllegalStateException
	 */
	public function remove()

This method seems to be duplicated in your project.

	{
		if($this->canRemove)
		{
			$this->list->removeAt($this->index);
			$this->canRemove = false;
		}
		else
		{
			throw new IllegalStateException("Cannot remove element unless next has been called");
		}
	}
}