Iterator - Code Metrics - Inspection of "working on database and tasks, old code cleanup" - comodojo/extender.framework - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 2.0 ( 2c7009...c833bc )

by Marco

created 2016-08-18 22:24 UTC

Iterator A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	5
c	1
b	0
f	0
lcom	1
cbo	0
dl	0
loc	54
rs	10

5 Methods

Rating	Name	Size	Complexity
A	rewind()	5	1
A	current()	5	1
A	key()	5	1
A	next()	5	1
A	valid()	5	1

<?php namespace Comodojo\Extender\Components;

/**
 * @package     Comodojo Framework
 * @author      Marco Giovinazzi <[email protected]>
 * @author      Marco Castiello <[email protected]>
 * @license     GPL-3.0+
 *
 * LICENSE:
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program. If not, see <http://www.gnu.org/licenses/>.
 */

trait Iterator {

    /**
     * Reset the iterator
     */
    public function rewind() {

        reset($this->data);
class MyClass { }

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

    }

    /**
     * Get the current element
     *
     * @return mixed
     */
    public function current() {

        return current($this->data);

    }

    /**
     * Return the current key
     *
     * @return string|int
     */
    public function key() {

        return key($this->data);

    }

    /**
     * Move to next element
     */
    public function next() {

        return next($this->data);

    }

    /**
     * Check if element is valid (isset)
     *
     * @return boolean
     */
    public function valid() {

        return isset($this->data[$this->key()]);

    }

}


1			<?php namespace Comodojo\Extender\Components;
2
3			/**
4			* @package Comodojo Framework
5			* @author Marco Giovinazzi <[email protected]>
6			* @author Marco Castiello <[email protected]>
7			* @license GPL-3.0+
8			*
9			* LICENSE:
10			*
11			* This program is free software: you can redistribute it and/or modify
12			* it under the terms of the GNU Affero General Public License as
13			* published by the Free Software Foundation, either version 3 of the
14			* License, or (at your option) any later version.
15			*
16			* This program is distributed in the hope that it will be useful,
17			* but WITHOUT ANY WARRANTY; without even the implied warranty of
18			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
19			* GNU Affero General Public License for more details.
20			*
21			* You should have received a copy of the GNU Affero General Public License
22			* along with this program. If not, see <http://www.gnu.org/licenses/>.
23			*/
24
25			trait Iterator {
26
27			/**
28			* Reset the iterator
29			*/
30			public function rewind() {
31
32			reset($this->data);
			0 ignored issues – show Bug introduced 2016-08-18 22:27 UTC by Report Bug Copy Issue Report The property `data` 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...
33
34			}
35
36			/**
37			* Get the current element
38			*
39			* @return mixed
40			*/
41			public function current() {
42
43			return current($this->data);
44
45			}
46
47			/**
48			* Return the current key
49			*
50			* @return string\|int
51			*/
52			public function key() {
53
54			return key($this->data);
55
56			}
57
58			/**
59			* Move to next element
60			*/
61			public function next() {
62
63			return next($this->data);
64
65			}
66
67			/**
68			* Check if element is valid (isset)
69			*
70			* @return boolean
71			*/
72			public function valid() {
73
74			return isset($this->data[$this->key()]);
75
76			}
77
78			}
79

comodojo / extender.framework

Push — 2.0 ( 2c7009...c833bc )

Iterator A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

5 Methods

Duplication Side-by-Side

Filter issues like