CsvFileObject::seek() - Code Metrics - gielfeldt/iterators - Measure and Improve Code Quality continuously with Scrutinizer

CsvFileObject::seek() A
last analyzed 2018-11-04 00:57 UTC

↳ Parent: CsvFileObject

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	7
rs	10
c	0
b	0
f	0
ccs	4
cts	4
cp	1
cc	2
nc	2
nop	1
crap	2

<?php

namespace Gielfeldt\Iterators;

/**
 * Provide fielded rows of an array from a csv file.
 *
 * Input:
 * [
 *   ['column1 header', 'column2 header', 'column3 header'],
 *   ['value 1', 'value 2', 'value 3'],
 *   ['value 4', 'value 5', 'value 6'],
 * ]
 *
 * Output:
 * [
 *   ['column1 header' => 'value 1', 'column2 header' => 'value 2', 'column3 header' => 'value 3'],
 *   ['column1 header' => 'value 4', 'column2 header' => 'value 5', 'column3 header' => 'value 6'],
 * ]
 */
class CsvFileObject extends \SplFileObject implements \JsonSerializable, \Countable
{
    /**
     * The fields of a row.
     *
     * @var array
     */
    protected $fields;

    public $rows;

    /**
     * SplFileObject constructor.
     *
     * Setup SplFileObject in CSV read-mode.
     */
    public function __construct($filename, $open_mode = "r", $use_include_path = false, $context = null)
    {
        parent::__construct($filename, $open_mode, $use_include_path, $context);
        $this->setFlags(\SplFileObject::READ_CSV | \SplFileObject::READ_AHEAD | \SplFileObject::SKIP_EMPTY | \SplFileObject::DROP_NEW_LINE);
        $this->processHeader();
    }

    /**
     * Get determined fields for this csv file.
     */
    public function getFields()
    {
        return $this->fields;
    }

    /**
     * Fetch fields from the current row.
     *
     * This method should only be called when file is rewound.
     */
    protected function processHeader()
    {
        $this->fields = array_values(parent::current());
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
        parent::next();
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
    }

    /**
     * Rewind file and fetch fields from first row.
     */
    public function rewind()
    {
        parent::rewind();
        $this->processHeader();
    }

    /**
     * Map row into fields.
     */
    public function current()
    {
        $row = parent::current();
        $fieldedRow = [];
        foreach ($this->fields as $idx => $field) {
            $fieldedRow[$field] = $row[$idx] ?? null;
        }
        return $fieldedRow;
    }

    public function valid()
    {
        $valid = parent::valid();
        if (!$valid && !isset($this->rows) && !is_null($this->key())) {
            $this->rows = $this->key();
        }
        return $valid;
    }

    /**
     * Subtract 1 row from index, so that we don't include the header in our count.
     */
    public function key()
    {
        return parent::key() - 1;
    }

    /**
     * Add 1 to skip the header when seeking.
     */
    public function seek($position)
    {
        if ($position < 0) {
            throw new \LogicException("Can't seek file " . $this->getFilename() . " to negative line $position");
        }
        return parent::seek($position + 1);
    }

    /**
     * Implements jsonSerialize.
     */
    public function jsonSerialize()
    {
        return iterator_to_array($this);
    }

    /**
     * Implements Countable.
     */
    public function count()
    {
        if (!isset($this->rows)) {
            $this->seek($this->getSize());
            $this->rows = $this->key();
        }
        return $this->rows;
    }
}


1		<?php
2
3		namespace Gielfeldt\Iterators;
4
5		/**
6		* Provide fielded rows of an array from a csv file.
7		*
8		* Input:
9		* [
10		* ['column1 header', 'column2 header', 'column3 header'],
11		* ['value 1', 'value 2', 'value 3'],
12		* ['value 4', 'value 5', 'value 6'],
13		* ]
14		*
15		* Output:
16		* [
17		* ['column1 header' => 'value 1', 'column2 header' => 'value 2', 'column3 header' => 'value 3'],
18		* ['column1 header' => 'value 4', 'column2 header' => 'value 5', 'column3 header' => 'value 6'],
19		* ]
20		*/
21		class CsvFileObject extends \SplFileObject implements \JsonSerializable, \Countable
22		{
23		/**
24		* The fields of a row.
25		*
26		* @var array
27		*/
28		protected $fields;
29
30		public $rows;
31
32		/**
33		* SplFileObject constructor.
34		*
35		* Setup SplFileObject in CSV read-mode.
36		*/
37	11	public function __construct($filename, $open_mode = "r", $use_include_path = false, $context = null)
38		{
39	11	parent::__construct($filename, $open_mode, $use_include_path, $context);
40	11	$this->setFlags(\SplFileObject::READ_CSV \| \SplFileObject::READ_AHEAD \| \SplFileObject::SKIP_EMPTY \| \SplFileObject::DROP_NEW_LINE);
41	11	$this->processHeader();
42	11	}
43
44		/**
45		* Get determined fields for this csv file.
46		*/
47	2	public function getFields()
48		{
49	2	return $this->fields;
50		}
51
52		/**
53		* Fetch fields from the current row.
54		*
55		* This method should only be called when file is rewound.
56		*/
57	11	protected function processHeader()
58		{
59	11	$this->fields = array_values(parent::current());
		0 ignored issues – show Comprehensibility Bug introduced 2017-04-21 08:03 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`current()` instead of `processHeader()`). Are you sure this is correct? If so, you might want to change this to `$this->current()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
60	11	parent::next();
		0 ignored issues – show Comprehensibility Bug introduced 2017-04-21 08:03 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`next()` instead of `processHeader()`). Are you sure this is correct? If so, you might want to change this to `$this->next()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
61	11	}
62
63		/**
64		* Rewind file and fetch fields from first row.
65		*/
66	8	public function rewind()
67		{
68	8	parent::rewind();
69	8	$this->processHeader();
70	8	}
71
72		/**
73		* Map row into fields.
74		*/
75	7	public function current()
76		{
77	7	$row = parent::current();
78	7	$fieldedRow = [];
79	7	foreach ($this->fields as $idx => $field) {
80	7	$fieldedRow[$field] = $row[$idx] ?? null;
81		}
82	7	return $fieldedRow;
83		}
84
85	8	public function valid()
86		{
87	8	$valid = parent::valid();
88	8	if (!$valid && !isset($this->rows) && !is_null($this->key())) {
89	8	$this->rows = $this->key();
90		}
91	8	return $valid;
92		}
93
94		/**
95		* Subtract 1 row from index, so that we don't include the header in our count.
96		*/
97	9	public function key()
98		{
99	9	return parent::key() - 1;
100		}
101
102		/**
103		* Add 1 to skip the header when seeking.
104		*/
105	4	public function seek($position)
106		{
107	4	if ($position < 0) {
108	1	throw new \LogicException("Can't seek file " . $this->getFilename() . " to negative line $position");
109		}
110	4	return parent::seek($position + 1);
111		}
112
113		/**
114		* Implements jsonSerialize.
115		*/
116	3	public function jsonSerialize()
117		{
118	3	return iterator_to_array($this);
119		}
120
121		/**
122		* Implements Countable.
123		*/
124	3	public function count()
125		{
126	3	if (!isset($this->rows)) {
127	3	$this->seek($this->getSize());
128	3	$this->rows = $this->key();
129		}
130	3	return $this->rows;
131		}
132		}
133

gielfeldt / iterators

GitHub Access Token became invalid

CsvFileObject::seek() A last analyzed 2018-11-04 00:57 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

CsvFileObject::seek() A
last analyzed 2018-11-04 00:57 UTC