Ticaje\FileManager\Infrastructure\Driver\Reader\File\Base

Complexity

Total Complexity

8

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getImplementor()	0	3	1
A	__construct()	0	6	1
A	setSource()	0	7	1
A	getContentCopy()	0	3	1
A	getData()	0	9	4

<?php
declare(strict_types=1);
/**
 * Infrastructure Related Agent
 * @author Max Demian <ticaje@filetea.me>
 */

namespace Ticaje\FileManager\Infrastructure\Driver\Reader\File;

use Iterator;
use Ticaje\FileManager\Infrastructure\Driver\Reader\Interfaces\Gateway\FileGatewayInterface;
use Ticaje\FileManager\Infrastructure\Driver\Reader\Interfaces\SourceInterface;
use Ticaje\FileManager\Infrastructure\Driver\Traits\FileInterface as ContractorTrait;

/**
 * Class Base
 * @package Ticaje\FileManager\Infrastructure\Driver\Reader\File
 */
abstract class Base
{
    use ContractorTrait;

    /** @var FileGatewayInterface $implementor */
    protected $implementor;

    /** @var bool $hasHeader */
    protected $hasHeader;

    /** @var Iterator|array $copy */
    private $copy;

    /** @var Iterator|array $content */
    private $content;


    /**
     * Csv constructor.
     *
     * @param FileGatewayInterface $implementor
     * @param bool                 $hasHeader
     */
    public function __construct(
        FileGatewayInterface $implementor,
        bool $hasHeader = false
    ) {
        $this->implementor = $implementor;
        $this->hasHeader = $hasHeader;
    }

    /**
     * @inheritDoc
     */
    public function setSource(string $fileName): SourceInterface
    {
        $this->content = $this->implementor->fetchData($fileName);
        $this->copy = $this->implementor->fetchData($fileName);
        $this->setHeader();

        return $this;

The expression return $this returns the type Ticaje\FileManager\Infrastructure\Driver\Reader\File\Base which is incompatible with the type-hinted return Ticaje\FileManager\Infrastructure\Driver\Reader\Interfaces\SourceInterface.

    }

    /**
     * @inheritDoc
     */
    public function getData(): Iterator
    {
        $content = $this->getContent();
        if ($content instanceof Iterator) {

$content is always a sub-type of Iterator.

            return $content;
        }
        if (is_array($content)) {
            foreach ($this->getContent() as $item) {
                yield $item;
            }
        }
    }

    /**
     * @inheritDoc
     */
    public function getImplementor(): FileGatewayInterface
    {
        return $this->implementor;
    }

    public function getContentCopy(): Iterator
    {
        return $this->copy;

The expression return $this->copy could return the type array which is incompatible with the type-hinted return Iterator. Consider adding an additional type-check to rule them out.

    }
}