Sample\ChunkReadFilter

Complexity

Total Complexity

5

Size/Duplication

Total Lines	27
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	5	5	1
A	readCell()	9	9	4

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 14 and the first side effect is on line 8.

namespace Sample;

use PhpOffice\PhpSpreadsheet\IOFactory;
use PhpOffice\PhpSpreadsheet\Reader\IReadFilter;

require __DIR__ . '/../Header.php';

$inputFileType = 'Xls';
$inputFileName = __DIR__ . '/sampleData/example2.xls';

/**  Define a Read Filter class implementing IReadFilter  */
class ChunkReadFilter implements IReadFilter

This class seems to be duplicated in your project.

{
    private $_startRow = 0;
    private $_endRow = 0;

    /**
     * We expect a list of the rows that we want to read to be passed into the constructor.
     *
     * @param mixed $startRow
     * @param mixed $chunkSize
     */
    public function __construct($startRow, $chunkSize)
    {
        $this->_startRow = $startRow;
        $this->_endRow = $startRow + $chunkSize;
    }

    public function readCell($column, $row, $worksheetName = '')
    {
        //  Only read the heading row, and the rows that were configured in the constructor
        if (($row == 1) || ($row >= $this->_startRow && $row < $this->_endRow)) {
            return true;
        }

        return false;
    }
}

$helper->log('Loading file ' . pathinfo($inputFileName, PATHINFO_BASENAME) . ' using IOFactory with a defined reader type of ' . $inputFileType);
/*  Create a new Reader of the type defined in $inputFileType  * */
$reader = IOFactory::createReader($inputFileType);

/*  Define how many rows we want for each "chunk"  * */
$chunkSize = 20;

/*  Loop to read our worksheet in "chunk size" blocks  * */
for ($startRow = 2; $startRow <= 240; $startRow += $chunkSize) {
    $helper->log('Loading WorkSheet using configurable filter for headings row 1 and for rows ' . $startRow . ' to ' . ($startRow + $chunkSize - 1));
    /*  Create a new Instance of our Read Filter, passing in the limits on which rows we want to read  * */
    $chunkFilter = new ChunkReadFilter($startRow, $chunkSize);
    /*  Tell the Reader that we want to use the new Read Filter that we've just Instantiated  * */
    $reader->setReadFilter($chunkFilter);
    /*  Load only the rows that match our filter from $inputFileName to a PhpSpreadsheet Object  * */
    $spreadsheet = $reader->load($inputFileName);

    //	Do some processing here

    $sheetData = $spreadsheet->getActiveSheet()->toArray(null, true, true, true);
    var_dump($sheetData);

var_dump($sheetData); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

}