Bdf\Prime\Connection\Result\ArrayResultSet

Complexity

Total Complexity

25

Size/Duplication

Total Lines	183
Duplicated Lines	0 %

Test Coverage

Coverage

69.09%

Importance

Changes

0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	fetchMode()	0	6	1
A	isWrite()	0	3	1
A	isRead()	0	3	1
A	asColumn()	0	3	1
A	hasWrite()	0	3	1
A	fetchValue()	0	20	6
A	all()	0	5	2
A	asList()	0	3	1
A	asClass()	0	3	1
A	asObject()	0	3	1
A	fetchColum()	0	11	3
A	asAssociative()	0	3	1
A	current()	0	3	1
A	fetchClass()	0	18	4

<?php

namespace Bdf\Prime\Connection\Result;

use Bdf\Prime\Exception\DBALException;

/**
 * Wrap simple associative array to ResultSet
 * This result is usefull for caches
 */
final class ArrayResultSet extends \ArrayIterator implements ResultSetInterface
{
    /**
     * @var string
     */
    private $fetchMode = self::FETCH_ASSOC;

    /**
     * @var mixed
     */
    private $fetchOptions;

    /**
     * @var string[]
     */
    private $columns;

    /**
     * @var \ReflectionClass
     */
    private $reflectionClass;

    /**
     * @var \ReflectionProperty[]
     */
    private $reflectionProperties;


    /**

Parameter $mode should have a doc-comment as per coding-style.

Parameter $options should have a doc-comment as per coding-style.

     * {@inheritdoc}
     */
    public function fetchMode($mode, $options = null)

Expected 1 blank line before function; 2 found

    {
        $this->fetchMode = $mode;
        $this->fetchOptions = $options;

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function asAssociative(): ResultSetInterface
    {
        return $this->fetchMode(self::FETCH_ASSOC);

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_ASSOC has been deprecated: Use asAssociative() instead

    }

    /**
     * {@inheritdoc}
     */
    public function asList(): ResultSetInterface
    {
        return $this->fetchMode(self::FETCH_NUM);

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_NUM has been deprecated: Use asList() instead

    }

    /**
     * {@inheritdoc}
     */
    public function asObject(): ResultSetInterface
    {
        return $this->fetchMode(self::FETCH_OBJECT);

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_OBJECT has been deprecated: Use asObject() instead

    }

    /**

Parameter $className should have a doc-comment as per coding-style.

     * {@inheritdoc}
     */
    public function asClass(string $className): ResultSetInterface
    {
        return $this->fetchMode(self::FETCH_CLASS, $className);

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_CLASS has been deprecated: Use asClass() instead

    }

    /**

Parameter $column should have a doc-comment as per coding-style.

     * {@inheritdoc}
     */
    public function asColumn(int $column = 0): ResultSetInterface
    {
        return $this->fetchMode(self::FETCH_COLUMN, $column);

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_COLUMN has been deprecated: Use asColumn() instead

    }

    /**
     * {@inheritdoc}
     */
    public function all()
    {
        return $this->fetchMode === self::FETCH_ASSOC

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_ASSOC has been deprecated: Use asAssociative() instead

The expression return $this->fetchMode === self::FETCH_ASSOC ? $this->getArrayCopy() : array_map(array($this, 'fetchValue'), $this->getArrayCopy()) returns the type array which is incompatible with the return type mandated by Bdf\Prime\Connection\Result\ResultSetInterface::all() of Bdf\Prime\Connection\Result\list.

            ? $this->getArrayCopy()

Inline shorthand IF statement must be declared on a single line

            : array_map([$this, 'fetchValue'], $this->getArrayCopy())

Space after closing parenthesis of function call prohibited

        ;

Space found before semicolon; expected ");" but found ")
;"

    }

    /**
     * {@inheritdoc}
     */
    public function current()
    {
        return $this->fetchValue(parent::current());
    }

    /**
     * Transform the value according to the fetch mode
     *
     * @param array $current
     *
     * @return mixed
     */

Missing @throws tag in function comment

    private function fetchValue($current)

Type hint "array" missing for $current

Private method name "ArrayResultSet::fetchValue" must be prefixed with an underscore

    {
        switch ($this->fetchMode) {
            case self::FETCH_ASSOC:

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_ASSOC has been deprecated: Use asAssociative() instead

                return $current;

Case breaking statement indented incorrectly; expected 14 spaces, found 16

            case self::FETCH_NUM:

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_NUM has been deprecated: Use asList() instead

                return array_values($current);

Case breaking statement indented incorrectly; expected 14 spaces, found 16

            case self::FETCH_COLUMN:

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_COLUMN has been deprecated: Use asColumn() instead

                return $this->fetchColum($current);

Case breaking statement indented incorrectly; expected 14 spaces, found 16

            case self::FETCH_OBJECT:

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_OBJECT has been deprecated: Use asObject() instead

                return (object) $current;

Case breaking statement indented incorrectly; expected 14 spaces, found 16

A cast statement should not be followed as per the coding-style.

            case self::FETCH_CLASS:

The constant Bdf\Prime\Connection\Result\ResultSetInterface::FETCH_CLASS has been deprecated: Use asClass() instead

                return $this->fetchClass($current);

Case breaking statement indented incorrectly; expected 14 spaces, found 16

            default:
                throw new DBALException('Unsupported fetch mode '.$this->fetchMode);

Case breaking statement indented incorrectly; expected 14 spaces, found 16

        }
    }

    /**
     * {@inheritdoc}
     */
    public function isRead(): bool
    {
        return true;
    }

    /**
     * {@inheritdoc}
     */
    public function isWrite(): bool
    {
        return false;
    }

    /**
     * {@inheritdoc}
     */
    public function hasWrite(): bool
    {
        return false;
    }

    private function fetchColum($current)

Missing doc comment for function fetchColum()

Private method name "ArrayResultSet::fetchColum" must be prefixed with an underscore

Missing function doc comment

    {
        if (!$this->fetchOptions) {
            return reset($current);
        }

        if (!$this->columns) {

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

            $this->columns = array_keys($current);
        }

        return $current[$this->columns[$this->fetchOptions]];
    }

    private function fetchClass($current)

Missing doc comment for function fetchClass()

Missing function doc comment

Private method name "ArrayResultSet::fetchClass" must be prefixed with an underscore

    {
        if (!$this->reflectionClass) {
            $this->reflectionClass = new \ReflectionClass($this->fetchOptions);
        }

        $object = $this->reflectionClass->newInstance();

        foreach ($current as $property => $value) {
            if (!isset($this->reflectionProperties[$property])) {
                $this->reflectionProperties[$property] = $this->reflectionClass->getProperty($property);
                $this->reflectionProperties[$property]->setAccessible(true);
            }

            $this->reflectionProperties[$property]->setValue($object, $value);
        }

        return $object;
    }
}