AbstractIterator::getIterator() - Code Metrics - Inspection of "Add Zippy factory and adapters to ease bootstrap" - aztech-dev/Zippy - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — full-rewrite (4754d3)

by Thibaud

created 2016-03-20 21:50 UTC

AbstractIterator::getIterator() A

↳ Parent: AbstractIterator

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	8
rs	9.4285
cc	2
eloc	4
nc	2
nop	0

<?php

namespace Alchemy\Zippy\Package\Iterator;

use Alchemy\Zippy\Package\PackagedResourceIterator;

abstract class AbstractIterator implements PackagedResourceIterator
{

   /**
     * @return \Iterator
     */
    abstract protected function buildIterator();

    /**
     * @return \Iterator
     */
    public function getIterator()
    {
        if (! isset($this->iterator)) {
            $this->iterator = $this->buildIterator();
class MyClass { }

$x = new MyClass();
$x->foo = true;
        }

        return $this->iterator;
    }

    /**
     * (PHP 5 &gt;= 5.0.0)<br/>
     * Move forward to next element
     * @link http://php.net/manual/en/iterator.next.php
     * @return void Any returned value is ignored.
     */
    public function next()
    {
        $this->getIterator()->next();
    }

    /**
     * (PHP 5 &gt;= 5.0.0)<br/>
     * Return the key of the current element
     * @link http://php.net/manual/en/iterator.key.php
     * @return mixed scalar on success, or null on failure.
     */
    public function key()
    {
        return $this->getIterator()->key();
    }

    /**
     * (PHP 5 &gt;= 5.0.0)<br/>
     * Checks if current position is valid
     * @link http://php.net/manual/en/iterator.valid.php
     * @return boolean The return value will be casted to boolean and then evaluated.
     * Returns true on success or false on failure.
     */
    public function valid()
    {
        return $this->getIterator()->valid();
    }

    /**
     * (PHP 5 &gt;= 5.0.0)<br/>
     * Rewind the Iterator to the first element
     * @link http://php.net/manual/en/iterator.rewind.php
     * @return void Any returned value is ignored.
     */
    public function rewind()
    {
        $this->getIterator()->rewind();
    }


}


1			<?php
2
3			namespace Alchemy\Zippy\Package\Iterator;
4
5			use Alchemy\Zippy\Package\PackagedResourceIterator;
6
7			abstract class AbstractIterator implements PackagedResourceIterator
8			{
9
10			/**
11			* @return \Iterator
12			*/
13			abstract protected function buildIterator();
14
15			/**
16			* @return \Iterator
17			*/
18			public function getIterator()
19			{
20			if (! isset($this->iterator)) {
21			$this->iterator = $this->buildIterator();
			0 ignored issues – show Bug introduced 2016-03-20 21:53 UTC by Report Bug Copy Issue Report The property `iterator` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
22			}
23
24			return $this->iterator;
25			}
26
27			/**
28			* (PHP 5 >= 5.0.0)<br/>
29			* Move forward to next element
30			* @link http://php.net/manual/en/iterator.next.php
31			* @return void Any returned value is ignored.
32			*/
33			public function next()
34			{
35			$this->getIterator()->next();
36			}
37
38			/**
39			* (PHP 5 >= 5.0.0)<br/>
40			* Return the key of the current element
41			* @link http://php.net/manual/en/iterator.key.php
42			* @return mixed scalar on success, or null on failure.
43			*/
44			public function key()
45			{
46			return $this->getIterator()->key();
47			}
48
49			/**
50			* (PHP 5 >= 5.0.0)<br/>
51			* Checks if current position is valid
52			* @link http://php.net/manual/en/iterator.valid.php
53			* @return boolean The return value will be casted to boolean and then evaluated.
54			* Returns true on success or false on failure.
55			*/
56			public function valid()
57			{
58			return $this->getIterator()->valid();
59			}
60
61			/**
62			* (PHP 5 >= 5.0.0)<br/>
63			* Rewind the Iterator to the first element
64			* @link http://php.net/manual/en/iterator.rewind.php
65			* @return void Any returned value is ignored.
66			*/
67			public function rewind()
68			{
69			$this->getIterator()->rewind();
70			}
71
72
73			}
74

aztech-dev / Zippy

Branch — full-rewrite (4754d3)

AbstractIterator::getIterator() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like