SliceIterator::valid() - Code Metrics - SuRaMoN/itertools - Measure and Improve Code Quality continuously with Scrutinizer

SliceIterator::valid() A
last analyzed 2020-05-28 15:38 UTC

↳ Parent: SliceIterator

Complexity

Conditions	4
Paths	5

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	14
rs	9.7998
c	0
b	0
f	0
cc	4
nc	5
nop	0

<?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();
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
		}
		if($this->innerIterationCount >= $this->offset + $this->length) {
			return false;
		}
		return parent::valid();
	}

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



1			<?php
2
3			namespace itertools;
4
5			use IteratorIterator;
6
7
8			/**
9			* Iterator equivalent of [array_slice](http://be1.php.net/manual/en/function.array-slice.php).
10			* Example:
11			* $lines = new SliceIterator(new FileLineIterator('file.txt'), 0, 1000); // will iterate the first 1000 lines of the file
12			*/
13			class SliceIterator extends IteratorIterator
14			{
15			const PRESERVE_KEYS = true;
16			const DONT_PRESERVE_KEYS = false;
17
18			protected $offset;
19			protected $length;
20			protected $preserveKeys;
21			protected $innerIterationCount;
22			protected $outerIterationCount;
23
24			public function __construct($iterable, $offset, $length = INF, $preserveKeys = false)
25			{
26			parent::__construct(IterUtil::asTraversable($iterable));
27			$this->offset = $offset;
28			$this->length = $length;
29			$this->preserveKeys = $preserveKeys;
30			}
31
32			public function key()
33			{
34			if($this->preserveKeys) {
35			return parent::key();
36			} else {
37			return $this->outerIterationCount;
38			}
39			}
40
41			public function next()
42			{
43			$this->innerIterationCount += 1;
44			$this->outerIterationCount += 1;
45			parent::next();
46			}
47
48			public function valid()
49			{
50			while($this->innerIterationCount < $this->offset) {
51			if(!parent::valid()) {
52			return false;
53			}
54			$this->innerIterationCount += 1;
55			parent::next();
			0 ignored issues – show Comprehensibility Bug introduced 2015-11-27 15:17 UTC by Report Bug Copy Issue Report 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()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
56			}
57			if($this->innerIterationCount >= $this->offset + $this->length) {
58			return false;
59			}
60			return parent::valid();
61			}
62
63			public function rewind()
64			{
65			$this->innerIterationCount = 0;
66			$this->outerIterationCount = 0;
67			parent::rewind();
68			}
69			}
70
71

SuRaMoN / itertools

SliceIterator::valid() A last analyzed 2020-05-28 15:38 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

SliceIterator::valid() A
last analyzed 2020-05-28 15:38 UTC