WebServCo\Framework\Files\CsvCreator

Complexity

Total Complexity

7

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getCsvFile()	0	7	2
B	getCsvData()	0	23	4

<?php
namespace WebServCo\Framework\Files;

use WebServCo\Framework\Exceptions\ApplicationException;

final class CsvCreator
{
    protected $delimiter;
    protected $enclosure;

    public function __construct($delimiter = ',', $enclosure = '"')
    {
        $this->delimiter = $delimiter;
        $this->enclosure = $enclosure;
    }

    public function getCsvFile(string $fileName, array $data, bool $addHeader = true)
    {
        if (empty($data)) {
            throw new ApplicationException('Empty data');
        }
        $csvData = $this->getCsvData($data, $addHeader);
        return new \WebServCo\Framework\Files\CsvFile($fileName, $csvData);
    }

    public function getCsvData(array $data, bool $addHeader = true)
    {
        try {
            // temporary memory wrapper; if bigger than 5MB will be written to temp file.
            $handle = fopen('php://temp/maxmemory: ' . (5*1024*1024), 'r+');

            if ($addHeader) {
                fputcsv($handle, array_keys(current($data)), $this->delimiter, $this->enclosure);

It seems like $handle can also be of type false; however, parameter $handle of fputcsv() does only seem to accept resource, maybe add an additional type check?

            }

            foreach ($data as $item) {
                fputcsv($handle, $item, $this->delimiter, $this->enclosure);
            }

            rewind($handle);

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

            $csvData = stream_get_contents($handle);

It seems like $handle can also be of type false; however, parameter $handle of stream_get_contents() does only seem to accept resource, maybe add an additional type check?

            fclose($handle);

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

            return $csvData;
        } catch (\Exception $e) {
            throw new ApplicationException($e->getMessage());
        }
    }
}