CsvParser\Writer\StringWriter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

90.91%

Importance

Changes	1
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	write()	0	22	4

<?php

namespace CsvParser\Writer;

class StringWriter implements WriterInterface
{
    public static function write(\CsvParser\Parser $parser, \CsvParser\Csv $csv)
    {
        $data = $csv->getData();

        if ($data && !empty($data)) {

The expression $data of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $output = array(
                implode($parser->fieldDelimiter, array_map(function ($value) use ($parser) {
                    return $parser->fieldEnclosure . str_replace($parser->fieldEnclosure, $parser->fieldEnclosure.$parser->fieldEnclosure, $value) . $parser->fieldEnclosure;
                }, array_keys(current($data))))
            );

            foreach ($data as $line) {
                $output[] = implode($parser->fieldDelimiter, array_map(function ($value) use ($parser) {
                    return $parser->fieldEnclosure . str_replace($parser->fieldEnclosure, $parser->fieldEnclosure.$parser->fieldEnclosure, $value) . $parser->fieldEnclosure;
                }, $line));
            }

            return implode($parser->lineDelimiter, $output);
        } else {
            return '';
        }
    }
}