EFParser\Cell

Complexity

Total Complexity

5

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getCellValue()	0	10	1
A	getCellValueByColumnAndRow()	0	10	1
A	getCellsValues()	0	11	2

<?php

namespace EFParser;

use EFParserInterface\ICellParser,
    EFParserInterface\IExcel;
/**
 * Class Cell
 * Класс для парсинга ячеек
 */
class Cell implements ICellParser
{
    private $parser;

    /**
     * Конструктор
     *
     * @param IExcel $excel (Класс реализующий IExcel интерфейс)
     */
    public function __construct(IExcel $excel)
    {
        $this->parser = $excel;
    }

    /**
     * Метод для чтения значения ячейки по адресу (A1 etc)
     *
     * @param string  $cell         (Адрес ячейки)
     * @param string  $sheetName    (Название листа, которому пренадлежит эта ячейка)
     *
     * @return mixed $value        (Значение)
     */
    public function getCellValue($cell, $sheetName)
    {
        $value = $this->parser->sourceReader

Accessing sourceReader on the interface EFParserInterface\IExcel suggest that you code against a concrete implementation. How about adding an instanceof check?

                              ->load($this->parser->sourceFilePath.$this->parser->sourceFile)

Accessing sourceFilePath on the interface EFParserInterface\IExcel suggest that you code against a concrete implementation. How about adding an instanceof check?

Accessing sourceFile on the interface EFParserInterface\IExcel suggest that you code against a concrete implementation. How about adding an instanceof check?

                              ->setActiveSheetIndexByName($sheetName)
                              ->getCell($cell)
                              ->getValue();

        return $value;
    }

    /**
     * Метод для чтения значения ячейки (по координатам)
     *
     * @param string $column    (Буквенный индекс колонки)
     * @param int    $row       (Индекс строки)
     * @param string $sheetName (Название листа, которому пренадлежит эта ячейка)
     *
     * @return mixed $value     (Значение)
     */
    public function getCellValueByColumnAndRow($column, $row, $sheetName)
    {
        $value = $this->parser->sourceReader

Accessing sourceReader on the interface EFParserInterface\IExcel suggest that you code against a concrete implementation. How about adding an instanceof check?

                              ->load($this->parser->sourceFilePath.$this->parser->sourceFile)

Accessing sourceFilePath on the interface EFParserInterface\IExcel suggest that you code against a concrete implementation. How about adding an instanceof check?

Accessing sourceFile on the interface EFParserInterface\IExcel suggest that you code against a concrete implementation. How about adding an instanceof check?

                              ->setActiveSheetIndexByName($sheetName)
                              ->getCellByColumnAndRow($column, $row)
                              ->getValue();

        return $value;
    }

    /**
     * Метод для чтения значениий нескольких ячеек
     *
     * @param array  $cells     (Массив ячеек)
     *
     * ```php
     * $cells = array("A27", "C28", "I27", "E49");
     * ```
     * @param string $sheetName (Название листа, которому пренадлежат эти ячейки)
     *
     * @return array $value     (Массив значений)
     */
    public function getCellsValues(array $cells, $sheetName)
    {
        $value = array();

        foreach ($cells as $cell)
        {
            $value[$cell] = $this->getCellValue($cell, $sheetName);
        }

        return $value;
    }
}