WebServCo\Framework\CsvCreator

Complexity

Total Complexity

6

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	4	1
B	getCsvData()	0	26	4

<?php
namespace WebServCo\Framework;

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)
    {
        $csvData = $this->getCsvData($data, $addHeader);
        return new \WebServCo\Framework\CsvFile($fileName, $csvData);

It seems like $csvData can also be of type false; however, parameter $fileData of WebServCo\Framework\CsvFile::__construct() does only seem to accept string, maybe add an additional type check?

    }

    public function getCsvData(array $data, bool $addHeader = true)
    {
        $csvData = null;

The assignment to $csvData is dead and can be removed.

        if (empty($data)) {
            return false;
        }

        // 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;
    }
}