Hello\CsvFileIterator

Complexity

Total Complexity

7

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	rewind()	0	5	1
A	key()	0	3	1
A	valid()	0	3	1
A	current()	0	3	1
A	__construct()	0	3	1
A	next()	0	4	1
A	__destruct()	0	3	1

<?php
/**
 * @author: jiangyi
 * @date: 下午3:03 2018/11/27
 */

namespace Hello;

class CsvFileIterator implements \Iterator
{
    protected $file;
    protected $key = 0;
    protected $current;

    public function __construct($file)
    {
        $this->file = fopen($file, 'r');
    }

    public function __destruct()
    {
        fclose($this->file);

It seems like $this->file can also be of type boolean; however, parameter $handle of fclose() does only seem to accept resource, maybe add an additional type check?

    }

    public function rewind()
    {
        rewind($this->file);

It seems like $this->file can also be of type boolean; however, parameter $handle of rewind() does only seem to accept resource, maybe add an additional type check?

        $this->current = fgetcsv($this->file);

It seems like $this->file can also be of type boolean; however, parameter $handle of fgetcsv() does only seem to accept resource, maybe add an additional type check?

        $this->key = 0;
    }

    public function valid()
    {
        return !feof($this->file);

It seems like $this->file can also be of type boolean; however, parameter $handle of feof() does only seem to accept resource, maybe add an additional type check?

    }

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

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

    public function next()
    {
        $this->current = fgetcsv($this->file);

It seems like $this->file can also be of type boolean; however, parameter $handle of fgetcsv() does only seem to accept resource, maybe add an additional type check?

        $this->key++;
    }
}