Extraload\Extractor\Doctrine\QueryExtractor

Complexity

Total Complexity

8

Size/Duplication

Total Lines	50
Duplicated Lines	24 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	current()	0	4	1
A	key()	0	4	1
A	next()	0	4	1
A	rewind()	0	4	1
A	valid()	0	4	1
A	extract()	12	12	2

<?php

namespace Extraload\Extractor\Doctrine;

use Doctrine\DBAL\Connection;
use Extraload\Extractor\ExtractorInterface;

class QueryExtractor implements ExtractorInterface
{
    private $position = 0;

    private $data;

    public function __construct(Connection $conn, string $sql)
    {
        $this->position = 0;
        $this->data = $conn->query($sql)->fetchAll();
    }

    public function extract()

This method seems to be duplicated in your project.

    {
        if ($this->position >= count($this->data)) {
            return;
        }

        $data = $this->current();

        $this->next();

        return $data;
    }

    public function current()
    {
        return $this->data[$this->position];
    }

    public function key()
    {
        return $this->position;
    }

    public function next()
    {
        $this->position += 1;
    }

    public function rewind()
    {
        $this->position = 0;
    }

    public function valid()
    {
        return isset($this->data[$this->position]);
    }
}