Jackal\Copycat\Writer\CSVFileWriter

Complexity

Total Complexity

15

Size/Duplication

Total Lines	98
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
B	writeItem()	0	19	7
A	prepare()	0	16	5
A	__construct()	0	17	2
A	finish()	0	3	1

<?php

namespace Jackal\Copycat\Writer;

use Exception;
use Symfony\Component\OptionsResolver\OptionsResolver;

/**
 * Class CSVFileWriter
 * @package Jackal\Copycat\Writer
 */
class CSVFileWriter implements WriterInterface
{
    /**
     * @var string
     */
    protected $outputFilePathname;

    /**
     * @var resource
     */
    protected $handle;

    /**
     * @var int
     */
    protected $index = 0;

    /**
     * @var array
     */
    protected $options = [];

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

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

        $this->outputFilePathname = $outputFilePathname;

        if (!is_dir(dirname($this->outputFilePathname))) {
            mkdir(dirname($this->outputFilePathname), 0775, true);
        }
    }

    /**
     * @throws Exception
     */
    public function prepare()
    {
        if (!is_dir(dirname($this->outputFilePathname))) {
            mkdir(dirname($this->outputFilePathname), 0775, true);
        }

        $fileExists = file_exists($this->outputFilePathname);
        if ($fileExists and !$this->options['replace_file']) {
            throw new Exception('File ' . realpath($this->outputFilePathname) . ' already exists');
        }

        if ($fileExists) {
            unlink($this->outputFilePathname);
        }

        $this->handle = fopen($this->outputFilePathname, 'w');

It seems like fopen($this->outputFilePathname, 'w') can also be of type false. However, the property $handle is declared as type resource. Maybe add an additional type check?

    }

    /**
     * @param array $item
     */
    public function writeItem(array $item)
    {
        if ($this->options['columns'] and array_keys($item) != $this->options['columns']) {
            $itemOrdered = [];
            foreach ($this->options['columns'] as $orderedColumn) {
                if (!array_key_exists($orderedColumn, $item)) {
                    $itemOrdered[$orderedColumn] = null;
                } else {
                    $itemOrdered[$orderedColumn] = $item[$orderedColumn];
                }
            }
            $item = $itemOrdered;
        }

        if ($this->index == 0 and $this->options['header']) {
            fputcsv($this->handle, array_keys($item), $this->options['delimiter'], $this->options['enclosure']);
        }
        fputcsv($this->handle, array_values($item), $this->options['delimiter'], $this->options['enclosure']);
        $this->index++;
    }

    /**
     *
     */
    public function finish()
    {
        fclose($this->handle);
    }
}