Pairs::next() - Code Metrics - letournel/path-finder - Measure and Improve Code Quality continuously with Scrutinizer

Pairs::next() A
last analyzed 2017-07-23 14:48 UTC

↳ Parent: Pairs

Complexity

Conditions	3
Paths	4

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	16
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	9
nc	4
nop	0

<?php

namespace Letournel\PathFinder\Core;

class Pairs implements \Iterator
{
    private
        $hasNext,
class A {
    var $property;
}
        $firstIndex,
        $secondIndex,
        $maxIndex,
        $keys,
        $values;
    
    public function __construct(array $items)
    {
        if(count($items) < 2)
        {
            throw new \RuntimeException('Pairs must be initialized with at least two items');
        }
        $this->hasNext = true;
        $this->firstIndex = 0;
        $this->secondIndex = 1;
        $this->maxIndex = count($items);
        
        $this->keys = array();
        $this->values = array();
        foreach($items as $key => $value)
        {
            $this->keys[] = $key;
            $this->values[] = $value;
        }
    }
    
    public function rewind()
    {
        $this->firstIndex = 0;
        $this->secondIndex = 1;
        $this->hasNext = true;
    }
    
    public function current()
    {
        return array($this->values[$this->firstIndex], $this->values[$this->secondIndex]);
    }
    
    public function key()
    {
        return (int) (($this->maxIndex - ($this->firstIndex + 3) / 2) * $this->firstIndex + $this->secondIndex - 1);
    }
    
    public function next()
    {
        $this->secondIndex++;
        if($this->secondIndex >= $this->maxIndex)
        {
            $this->firstIndex++;
            $this->secondIndex = $this->firstIndex + 1;
        }
        if($this->secondIndex >= $this->maxIndex)
        {
            $this->hasNext = false;
            return null;
        }
        
        return $this->current();
    }
    
    public function valid()
    {
        return $this->hasNext;
    }
}


1			<?php
2
3			namespace Letournel\PathFinder\Core;
4
5			class Pairs implements \Iterator
6			{
7			private
8			$hasNext,
			0 ignored issues – show Coding Style introduced 2015-12-08 23:25 UTC by Report Bug Copy Issue Report It is generally advisable to only define one property per statement. Only declaring a single property per statement allows you to later on add doc comments more easily. It is also recommended by PSR2, so it is a common style that many people expect. Loading history... Coding Style introduced 2015-12-08 23:25 UTC by Report Bug Copy Issue Report The visibility should be declared for property `$hasNext`. The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using class A { var $property; } the property is implicitly global. To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2. Loading history...
9			$firstIndex,
10			$secondIndex,
11			$maxIndex,
12			$keys,
13			$values;
14
15			public function __construct(array $items)
16			{
17			if(count($items) < 2)
18			{
19			throw new \RuntimeException('Pairs must be initialized with at least two items');
20			}
21			$this->hasNext = true;
22			$this->firstIndex = 0;
23			$this->secondIndex = 1;
24			$this->maxIndex = count($items);
25
26			$this->keys = array();
27			$this->values = array();
28			foreach($items as $key => $value)
29			{
30			$this->keys[] = $key;
31			$this->values[] = $value;
32			}
33			}
34
35			public function rewind()
36			{
37			$this->firstIndex = 0;
38			$this->secondIndex = 1;
39			$this->hasNext = true;
40			}
41
42			public function current()
43			{
44			return array($this->values[$this->firstIndex], $this->values[$this->secondIndex]);
45			}
46
47			public function key()
48			{
49			return (int) (($this->maxIndex - ($this->firstIndex + 3) / 2) * $this->firstIndex + $this->secondIndex - 1);
50			}
51
52			public function next()
53			{
54			$this->secondIndex++;
55			if($this->secondIndex >= $this->maxIndex)
56			{
57			$this->firstIndex++;
58			$this->secondIndex = $this->firstIndex + 1;
59			}
60			if($this->secondIndex >= $this->maxIndex)
61			{
62			$this->hasNext = false;
63			return null;
64			}
65
66			return $this->current();
67			}
68
69			public function valid()
70			{
71			return $this->hasNext;
72			}
73			}
74

letournel / path-finder

GitHub Access Token became invalid

Pairs::next() A last analyzed 2017-07-23 14:48 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

Pairs::next() A
last analyzed 2017-07-23 14:48 UTC