SparQLIterator::moveNext() - Code Metrics - Inspection of "3.0.0" - byjg/anydataset - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#2)

by Joao

created 2017-05-27 20:14 UTC

SparQLIterator::moveNext() A

↳ Parent: SparQLIterator

Complexity

Conditions	3
Paths	3

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	3.1406

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	6
cts	8
cp	0.75
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	9
nc	3
nop	0
crap	3.1406

<?php

namespace ByJG\AnyDataset\Dataset;

use ByJG\AnyDataset\Exception\IteratorException;
use SparQL\Result;

class SparQLIterator extends GenericIterator
{

    /**
     * @var Result
     */
    private $sparqlQuery;

    /**
     * Enter description here...
     *
     * @var int
     */
    private $current = 0;

    public function __construct(Result $sparqlQuery)
    {
        $this->sparqlQuery = $sparqlQuery;

        $this->current = 0;
    }

    public function count()
    {
        return ($this->sparqlQuery->numRows());
    }

    /**
     * @access public
     * @return bool
     */
    public function hasNext()
    {
        if ($this->current < $this->count()) {
            return true;
        }

        return false;
    }

    /**
     * @access public
     * @return Row
     * @throws IteratorException
     */
    public function moveNext()
    {
        if (!$this->hasNext()) {
            throw new IteratorException("No more records. Did you used hasNext() before moveNext()?");
        }

        if ($row = $this->sparqlQuery->fetchArray()) {
            $sr = new Row($row);
            $this->current++;
            return $sr;
        } else {
            throw new IteratorException("No more records. Unexpected behavior.");
        }
    }

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


1		<?php
2
3		namespace ByJG\AnyDataset\Dataset;
4
5		use ByJG\AnyDataset\Exception\IteratorException;
6		use SparQL\Result;
7
8		class SparQLIterator extends GenericIterator
9		{
10
11		/**
12		* @var Result
13		*/
14		private $sparqlQuery;
15
16		/**
17		* Enter description here...
18		*
19		* @var int
20		*/
21		private $current = 0;
22
23	2	public function __construct(Result $sparqlQuery)
24		{
25	2	$this->sparqlQuery = $sparqlQuery;
26
27	2	$this->current = 0;
28	2	}
29
30	2	public function count()
31		{
32	2	return ($this->sparqlQuery->numRows());
33		}
34
35		/**
36		* @access public
37		* @return bool
38		*/
39	2	public function hasNext()
40		{
41	2	if ($this->current < $this->count()) {
42	2	return true;
43		}
44
45	1	return false;
46		}
47
48		/**
49		* @access public
50		* @return Row
51		* @throws IteratorException
52		*/
53	1	public function moveNext()
54		{
55	1	if (!$this->hasNext()) {
56		throw new IteratorException("No more records. Did you used hasNext() before moveNext()?");
57		}
58
59	1	if ($row = $this->sparqlQuery->fetchArray()) {
60	1	$sr = new Row($row);
61	1	$this->current++;
62	1	return $sr;
63		} else {
64		throw new IteratorException("No more records. Unexpected behavior.");
65		}
66		}
67
68		public function key()
69		{
70		return $this->current;
71		}
72		}
73

byjg / anydataset

Pull Request — master (#2)

SparQLIterator::moveNext() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like