Serialize\ExcelSerializer

Complexity

Total Complexity

9

Size/Duplication

Total Lines	53
Duplicated Lines	13.21 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setRowFromArray()	7	14	4
A	serializeData()	0	33	5

<?php
namespace Serialize;
require_once dirname(__FILE__).'/../vendor/autoload.php';

use PhpOffice\PhpSpreadsheet\Spreadsheet;
use PhpOffice\PhpSpreadsheet\Writer\Xlsx;
use PhpOffice\PhpSpreadsheet\Writer\Xls;

class ExcelSerializer extends SpreadSheetSerializer
{
    protected $types = array('xlsx', 'xls', 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet', 'application/vnd.ms-excel');

    protected function setRowFromArray(&$sheat, $row, $array, $count = 0)
    {
        if($count === 0)
        {
            $count = count($array);
        }
        for($i = 0; $i < $count; $i++)

This code seems to be duplicated across your project.

        {
            if(isset($array[$i]))
            {
                $sheat->setCellValueByColumnAndRow($i+1, $row, $array[$i]);
            }
        }
    }

    public function serializeData(&$type, $array)
    {
        if($this->supportsType($type) === false)
        {
            return null;
        }
        $data = $this->getArray($array);
        $ssheat = new Spreadsheet();
        $sheat = $ssheat->getActiveSheet();
        $keys = array_shift($data);
        $rowCount = count($data);
        $colCount = count($keys);
        $this->setRowFromArray($sheat, 1, $keys, $colCount);
        for($i = 0; $i < $rowCount; $i++)
        {
            $this->setRowFromArray($sheat, (2 + $i), $data[$i], $colCount);
        }
        $writerType = 'Excel5';

$writerType is not used, you could remove the assignment.

        if(strcasecmp($type, 'xlsx') === 0 || strcasecmp($type, 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet') === 0)
        {
            $writer = new Xlsx($ssheat);
            $writerType = 'Excel2007';

$writerType is not used, you could remove the assignment.

            $type = 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet';
        }
        else
        {
            $writer = new Xls($ssheat);
            $type = 'application/vnd.ms-excel';
        }
        ob_start();
        $writer->save('php://output');
        return ob_get_clean();
    }
}