SilverStripe\SsPak\DataExtractor\CsvTableWriter

Complexity

Total Complexity

14

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	start()	0	5	1
A	finish()	0	3	1
A	writeRecord()	0	7	2
A	mapFromColumns()	0	8	3
A	putRow()	0	3	1
A	open()	0	10	3
A	close()	0	6	2

<?php

namespace SilverStripe\SsPak\DataExtractor;

class CsvTableWriter implements TableWriter
{

	private $filename;
	private $handle;
	private $columns;

	function __construct($filename) {

It is generally recommended to explicitly declare the visibility for methods.

		$this->filename = $filename;
	}

	function start($columns) {

It is generally recommended to explicitly declare the visibility for methods.

		$this->open();
		$this->putRow($columns);
		$this->columns = $columns;
	}

	function finish() {

It is generally recommended to explicitly declare the visibility for methods.

		$this->close();
	}

	function writeRecord($record) {

It is generally recommended to explicitly declare the visibility for methods.

		if (!$this->columns) {

The expression $this->columns 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.

			$this->start(array_keys($record));
		}

		$this->putRow($this->mapFromColumns($record));
	}

	private function mapFromColumns($record) {
		$row = [];
		foreach($this->columns as $i => $column)
		{
			$row[$i] = isset($record[$column]) ? $record[$column] : null;
		}
		return $row;
	}

	private function putRow($row) {
		return fputcsv($this->handle, $row);
	}

	private function open() {
		if ($this->handle) {
			fclose($this->handle);
			$this->handle = null;
		}
		$this->handle = fopen($this->filename, 'w');
		if (!$this->handle) {
			throw new \LogicException("Can't open $this->filename for writing.");
		}
	}

	private function close() {
		if ($this->handle) {
			fclose($this->handle);
			$this->handle = null;
		}
	}
}