ReversedIterator - Code Metrics - zicht/itertools - Measure and Improve Code Quality continuously with Scrutinizer

ReversedIterator A
last analyzed 2021-03-29 13:12 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

Metric	Value
dl	0
loc	34
ccs	12
cts	12
cp	1
rs	10
c	0
b	0
f	0
wmc	4
lcom	0
cbo	1

3 Methods

Rating	Name	Size	Complexity
A	key()	5	1
A	__construct()	8	2
A	current()	5	1

<?php
/**
 * @copyright Zicht Online <http://zicht.nl>
 */

namespace Zicht\Itertools\lib;

use Zicht\Itertools\lib\Interfaces\FiniteIterableInterface;
use Zicht\Itertools\lib\Traits\FiniteIterableTrait;

class ReversedIterator extends \IteratorIterator implements FiniteIterableInterface
{
    use FiniteIterableTrait;

    /**
     * @param \Iterator $iterable
     */
    public function __construct(\Iterator $iterable)
    {
        $data = [];
        foreach ($iterable as $key => $value) {
            $data [] = [$key, $value];
        }
        parent::__construct(new \ArrayIterator(array_reverse($data)));
    }

    /**
     * {@inheritDoc}
     */
    public function key()
    {
        list($key, $value) = parent::current();
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

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

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
        return $key;
    }

    /**
     * {@inheritDoc}
     */
    public function current()
    {
        list($key, $value) = parent::current();
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;
        return $value;
    }
}


1		<?php
2		/**
3		* @copyright Zicht Online <http://zicht.nl>
4		*/
5
6		namespace Zicht\Itertools\lib;
7
8		use Zicht\Itertools\lib\Interfaces\FiniteIterableInterface;
9		use Zicht\Itertools\lib\Traits\FiniteIterableTrait;
10
11		class ReversedIterator extends \IteratorIterator implements FiniteIterableInterface
12		{
13		use FiniteIterableTrait;
14
15		/**
16		* @param \Iterator $iterable
17		*/
18	4	public function __construct(\Iterator $iterable)
19		{
20	4	$data = [];
21	4	foreach ($iterable as $key => $value) {
22	3	$data [] = [$key, $value];
23		}
24	4	parent::__construct(new \ArrayIterator(array_reverse($data)));
25	4	}
26
27		/**
28		* {@inheritDoc}
29		*/
30	2	public function key()
31		{
32	2	list($key, $value) = parent::current();
		0 ignored issues – show Comprehensibility Bug introduced 2017-01-12 12:49 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`current()` instead of `key()`). Are you sure this is correct? If so, you might want to change this to `$this->current()`. 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... Unused Code introduced 2017-01-12 12:49 UTC by Report Bug Copy Issue Report The assignment to `$value` is unused. Consider omitting it like so `list($first,,$third)`. This checks looks for assignemnts to variables using the `list(...)` function, where not all assigned variables are subsequently used. Consider the following code example. <?php function returnThreeValues() { return array('a', 'b', 'c'); } list($a, $b, $c) = returnThreeValues(); print $a . " - " . $c; Only the variables `$a` and `$c` are used. There was no need to assign `$b`. Instead, the list call could have been. list($a,, $c) = returnThreeValues(); Loading history...
33	2	return $key;
34		}
35
36		/**
37		* {@inheritDoc}
38		*/
39	2	public function current()
40		{
41	2	list($key, $value) = parent::current();
		0 ignored issues – show Unused Code introduced 2017-01-12 12:49 UTC by Report Bug Copy Issue Report The assignment to `$key` is unused. Consider omitting it like so `list($first,,$third)`. This checks looks for assignemnts to variables using the `list(...)` function, where not all assigned variables are subsequently used. Consider the following code example. <?php function returnThreeValues() { return array('a', 'b', 'c'); } list($a, $b, $c) = returnThreeValues(); print $a . " - " . $c; Only the variables `$a` and `$c` are used. There was no need to assign `$b`. Instead, the list call could have been. list($a,, $c) = returnThreeValues(); Loading history...
42	2	return $value;
43		}
44		}
45

zicht / itertools

ReversedIterator A last analyzed 2021-03-29 13:12 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

3 Methods

Duplication Side-by-Side

Filter issues like

ReversedIterator A
last analyzed 2021-03-29 13:12 UTC