Iteration - Code Metrics - ThrusterIO/xml-iterator - Measure and Improve Code Quality continuously with Scrutinizer

Iteration A
last analyzed 2016-09-22 13:16 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	80
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

96%

Importance

Changes

Metric	Value
wmc	9
lcom	1
cbo	1
dl	0
loc	80
ccs	24
cts	25
cp	0.96
rs	10
c	0
b	0
f	0

7 Methods

Rating	Name	Size	Complexity
A	__construct()	4	1
A	skipNextRead()	4	1
A	current()	4	1
A	next()	11	2
A	key()	4	1
A	valid()	4	1
A	rewind()	9	2

<?php

namespace Thruster\Component\XMLIterator;

use Iterator;
use BadMethodCallException;

/**
 * Class Iteration
 *
 * @package Thruster\Component\XMLIterator
 * @author  Aurimas Niekis <[email protected]>
 */
class Iteration implements Iterator
{
    /**
     * @var XMLReader
     */
    private $reader;

    /**
     * @var bool
     */
    private $valid;

    /**
     * @var int
     */
    private $index;

    /**
     * @var bool
     */
    private $skipNextRead;

    public function __construct(XMLReader $reader)
    {
        $this->reader = $reader;
    }

    /**
     * skip the next read on next next()
     *
     * this is useful of the reader has moved to the next node already inside a foreach iteration and the next
     * next would move the reader one off.
     *
     * @see next
     */
    public function skipNextRead()
    {
        $this->skipNextRead = true;
    }

    /**
     * @return XMLReader
     */
    public function current()
    {
        return $this->reader;
    }

    public function next()
    {
        $this->index++;

        if ($this->skipNextRead) {
            $this->skipNextRead = false;
            $this->valid        = $this->reader->nodeType;
$answer = 42;

$correct = false;

$correct = (bool) $answer;
        } else {
            $this->valid = $this->reader->read();
        }
    }

    public function key()
    {
        return $this->index;
    }

    public function valid()
    {
        return $this->valid;
    }

    public function rewind()
    {
        if ($this->reader->nodeType !== XMLReader::NONE) {
            throw new BadMethodCallException('Reader can not be rewound');
        }

        $this->index = 0;
        $this->valid = $this->reader->read();
    }
}


1		<?php
2
3		namespace Thruster\Component\XMLIterator;
4
5		use Iterator;
6		use BadMethodCallException;
7
8		/**
9		* Class Iteration
10		*
11		* @package Thruster\Component\XMLIterator
12		* @author Aurimas Niekis <[email protected]>
13		*/
14		class Iteration implements Iterator
15		{
16		/**
17		* @var XMLReader
18		*/
19		private $reader;
20
21		/**
22		* @var bool
23		*/
24		private $valid;
25
26		/**
27		* @var int
28		*/
29		private $index;
30
31		/**
32		* @var bool
33		*/
34		private $skipNextRead;
35
36	3	public function __construct(XMLReader $reader)
37		{
38	3	$this->reader = $reader;
39	3	}
40
41		/**
42		* skip the next read on next next()
43		*
44		* this is useful of the reader has moved to the next node already inside a foreach iteration and the next
45		* next would move the reader one off.
46		*
47		* @see next
48		*/
49	1	public function skipNextRead()
50		{
51	1	$this->skipNextRead = true;
52	1	}
53
54		/**
55		* @return XMLReader
56		*/
57	2	public function current()
58		{
59	2	return $this->reader;
60		}
61
62	2	public function next()
63		{
64	2	$this->index++;
65
66	2	if ($this->skipNextRead) {
67	1	$this->skipNextRead = false;
68	1	$this->valid = $this->reader->nodeType;
		0 ignored issues – show Documentation Bug introduced 2016-09-22 13:16 UTC by Report Bug Copy Issue Report The property `$valid` was declared of type `boolean`, but `$this->reader->nodeType` is of type `integer`. Maybe add a type cast? This check looks for assignments to scalar types that may be of the wrong type. To ensure the code behaves as expected, it may be a good idea to add an explicit type cast. $answer = 42; $correct = false; $correct = (bool) $answer; Loading history...
69		} else {
70	2	$this->valid = $this->reader->read();
71		}
72	2	}
73
74	2	public function key()
75		{
76	2	return $this->index;
77		}
78
79	2	public function valid()
80		{
81	2	return $this->valid;
82		}
83
84	2	public function rewind()
85		{
86	2	if ($this->reader->nodeType !== XMLReader::NONE) {
87		throw new BadMethodCallException('Reader can not be rewound');
88		}
89
90	2	$this->index = 0;
91	2	$this->valid = $this->reader->read();
92	2	}
93		}
94

ThrusterIO / xml-iterator

Iteration A last analyzed 2016-09-22 13:16 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

7 Methods

Duplication Side-by-Side

Filter issues like

Iteration A
last analyzed 2016-09-22 13:16 UTC