OneSheet\Workbook

Complexity

Total Complexity

15

Size/Duplication

Total Lines	114
Duplicated Lines	15.79 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	setPrintTitleRange()	0	7	4
A	getWorkbookXml()	0	7	1
A	getWorkbookRelsXml()	9	9	2
A	getWorkbookSheetsXml()	9	9	2
A	getDefinedNamesXml()	0	13	4
A	createSingleDefinedNameXml()	0	11	2

<?php

namespace OneSheet;

use OneSheet\Xml\WorkbookXml;

class Workbook
{
    /**
     * @var int[]
     */
    private $printTitleStarts;

    /**
     * @var int[]
     */
    private $printTitleEnds;

    /**
     * Set the range of rows to repeat when printing the excel file.
     * $startRow=1 and $endRow=1 will repeat the first row only.
     *
     * @param int    $startRow
     * @param int    $endRow
     * @param string $sheetName
     */
    public function setPrintTitleRange($startRow, $endRow, $sheetName)
    {
        if ($startRow <= $endRow && is_numeric($startRow) && is_numeric($endRow)) {
            $this->printTitleStarts[$sheetName] = (int)$startRow;
            $this->printTitleEnds[$sheetName] = (int)$endRow;
        }
    }

    /**
     * Get final/combined XML for workbook.xml file.
     *
     * @param string[] $sheetNames
     * @return string
     */
    public function getWorkbookXml(array $sheetNames)
    {
        $sheets = $this->getWorkbookSheetsXml($sheetNames);
        $definedNames = $this->getDefinedNamesXml($sheetNames);

        return sprintf(WorkbookXml::WORKBOOK_XML, $sheets, $definedNames);
    }

    /**
     * Get XML for sheet relations of the workbook.
     *
     * @param string[] $sheetNames
     * @return string
     */
    public function getWorkbookRelsXml($sheetNames)

This method seems to be duplicated in your project.

    {
        $relations = '';
        foreach ($sheetNames as $key => $sheetName) {
            $relations .= sprintf(WorkbookXml::WORKBOOK_REL_XML, $key + 1, $sheetName);
        }

        return sprintf(WorkbookXml::WORKBOOK_RELS_XML, $relations);
    }

    /**
     * Generate xml with all sheet names that should be linked to the workbook.
     *
     * @param string[] $sheetNames
     * @return string
     */
    private function getWorkbookSheetsXml($sheetNames)

This method seems to be duplicated in your project.

    {
        $sheets = '';
        foreach ($sheetNames as $key => $sheetName) {
            $sheets .= sprintf(WorkbookXml::WORKBOOK_SHEETS_XML, $sheetName, $key + 1, $key + 1);
        }

        return $sheets;
    }

    /**
     * Generate xml used for repeatable headers when printing of exporting to PDF,
     * or empty string if none are set.
     *
     * @param string[] $sheetNames
     * @return string
     */
    private function getDefinedNamesXml(array $sheetNames)
    {
        $definedNames = '';
        if ($this->printTitleStarts && $this->printTitleEnds) {

The expression $this->printTitleStarts of type integer[] 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.

The expression $this->printTitleEnds of type integer[] 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.

            $definedNameTags = '';
            foreach ($sheetNames as $key => $sheetName) {
                $definedNameTags .= $this->createSingleDefinedNameXml($sheetName, $key);
            }
            $definedNames = sprintf(WorkbookXml::DEFINED_NAMES_XML, $definedNameTags);
        }

        return $definedNames;
    }

    /**
     * $localSheetId is the only one that has to start from 0 instead of 1.
     *
     * @param string $sheetName
     * @param int    $localSheetId
     * @return string
     */
    private function createSingleDefinedNameXml($sheetName, $localSheetId)
    {
        return isset($this->printTitleEnds[$sheetName])
            ? sprintf(
                WorkbookXml::DEFINED_NAME_XML,
                $localSheetId,
                $sheetName,
                $this->printTitleStarts[$sheetName],
                $this->printTitleEnds[$sheetName])
            : '';
    }
}