itertools\CurrentCachedIterator

Complexity

Total Complexity

10

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	invalidateCache()	0	5	1
A	rewind()	0	5	1
A	uncachedCurrent()	0	4	1
A	current()	0	8	2
A	next()	0	5	1
A	uncachedValid()	0	4	1
A	valid()	0	8	2

<?php

namespace itertools;

use IteratorIterator;


class CurrentCachedIterator extends IteratorIterator
{
	protected $cachedCurrent;
	protected $currentUpToDate;
	protected $cachedValid;
	protected $validUpToDate;

	public function __construct($innerIterator)
	{
		parent::__construct(IterUtil::asIterator($innerIterator));
		$this->currentUpToDate = false;
		$this->validUpToDate = false;
	}

	public function invalidateCache()
	{
		$this->currentUpToDate = false;
		$this->validUpToDate = false;
	}

    public function rewind()
	{
		$this->invalidateCache();
		return parent::rewind();
    }

	public function uncachedCurrent()
	{
		return parent::current();

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

	}

    public function current()
	{
		if(!$this->currentUpToDate) {
			$this->cachedCurrent = $this->uncachedCurrent();
			$this->currentUpToDate = true;
		}
        return $this->cachedCurrent;
    }

    public function next()
	{
		$this->invalidateCache();
		return parent::next();
    }

	public function uncachedValid()
	{
		return parent::valid();

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

	}

    public function valid()
	{
		if($this->validUpToDate) {
			return $this->cachedValid;
		}
		$this->validUpToDate = true;
        return $this->cachedValid = $this->uncachedValid();
    }
}