Jackal\Copycat\Reader\CSVFileReader

Complexity

Total Complexity

6

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
A	readCurrentRow()	0	3	1
A	__construct()	0	27	5

<?php

namespace Jackal\Copycat\Reader;

use SplFileObject;
use Symfony\Component\OptionsResolver\OptionsResolver;

/**
 * Class CSVFileReader
 * @package Jackal\Copycat\Reader
 */
class CSVFileReader extends BaseReader
{
    /**
     * @var SplFileObject
     */
    protected $fileObject;

    /**
     * @var array
     */
    protected $options;

    /**
     * CSVFileReader constructor.
     * @param SplFileObject $fileObject
     * @param array $options
     */
    public function __construct(SplFileObject $fileObject, array $options = [])
    {
        $resolver = new OptionsResolver();
        $resolver->setDefaults([
            'delimiter' => ',',
            'enclosure' => '"',
            'header' => true,
        ]);

        $this->options = $resolver->resolve($options);

        $this->fileObject = $fileObject;

        $headers = [];
        if ($this->options['header']) {
            $headers = $this->readCurrentRow();
        }

        while (($data = $this->readCurrentRow()) != false) {
            //skip empty rows
            if($data == [null]){
                continue;
            }
            if ($headers) {
                $this->addItem(array_combine($headers, $data));

It seems like array_combine($headers, $data) can also be of type false; however, parameter $item of Jackal\Copycat\Reader\BaseReader::addItem() does only seem to accept array, maybe add an additional type check?

            } else {
                $this->addItem($data);
            }
        }
    }

    protected function readCurrentRow()
    {
        return $this->fileObject->fgetcsv($this->options['delimiter'], $this->options['enclosure']);
    }
}