wenjy / hello

src/CsvFileIterator.php

<?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++;
    }
}