Bdf\Prime\Sharding\MultiStatement

Complexity

Total Complexity

16

Size/Duplication

Total Lines	121
Duplicated Lines	0 %

Test Coverage

Coverage

84.62%

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	fetchColumn()	0	11	2
A	getIterator()	0	3	1
A	add()	0	3	1
A	setFetchMode()	0	7	2
A	__construct()	0	3	1
A	closeCursor()	0	7	2
A	columnCount()	0	7	2
A	fetchAll()	0	9	2
A	fetch()	0	16	3

<?php

namespace Bdf\Prime\Sharding;

use Doctrine\DBAL\Driver\ResultStatement;

/**
 * Array cache statement
 * 
 * @package Bdf\Prime\Query
 */
class MultiStatement implements \IteratorAggregate, ResultStatement
{
    /**
     * @var ResultStatement[]
     */
    protected $statements;

    /**
     * @var int

Expected "integer" but found "int" for @var tag in member variable comment

     */
    protected $current = 0;

    /**
     * @param array $statements
     */
    public function __construct(array $statements = [])
    {
        $this->statements = $statements;
    }

    /**

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

     * {@inheritdoc}
     */
    public function add(ResultStatement $statement)
    {
        $this->statements[] = $statement;
    }

    /**
     * {@inheritdoc}
     */
    public function closeCursor()
    {
        foreach ($this->statements as $statement) {
            $statement->closeCursor();

The function Doctrine\DBAL\Driver\ResultStatement::closeCursor() has been deprecated: Use Result::free() instead.

        }

        unset($this->statements);
    }

    /**
     * {@inheritdoc}
     */
    public function columnCount()
    {
        if (!isset($this->statements[0])) {
            return 0;
        }

        return $this->statements[0]->columnCount();
    }

    /**

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

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

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

     * {@inheritdoc}
     */
    public function setFetchMode($fetchMode, $arg2 = null, $arg3 = null)

Variable "arg2" contains numbers but this is discouraged

Variable "arg3" contains numbers but this is discouraged

    {
        foreach ($this->statements as $statement) {
            $statement->setFetchMode($fetchMode, $arg2, $arg3);

Variable "arg2" contains numbers but this is discouraged

Variable "arg3" contains numbers but this is discouraged

The function Doctrine\DBAL\Driver\ResultStatement::setFetchMode() has been deprecated: Use one of the fetch- or iterate-related methods.

        }

        return true;
    }

    /**
     * {@inheritdoc}
     */
    public function getIterator()
    {
        return new \ArrayIterator($this->fetchAll());
    }

    /**

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

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

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

     * {@inheritdoc}
     */
    public function fetch($fetchMode = null, $cursorOrientation = \PDO::FETCH_ORI_NEXT, $cursorOffset = 0)

The method parameter $cursorOrientation is never used

The method parameter $cursorOffset is never used

    {
        // Stop the fetch if there s no statement
        if (!isset($this->statements[$this->current])) {
            return false;
        }

        $result = $this->statements[$this->current]->fetch($fetchMode);

The function Doctrine\DBAL\Driver\ResultStatement::fetch() has been deprecated: Use fetchNumeric(), fetchAssociative() or fetchOne() instead.

        if (!$result) {
            // go to the next statement

Inline comments must start with a capital letter

            $this->current++;
            return $this->fetch($fetchMode);
        }

        return $result;
    }

    /**

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

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

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

     * {@inheritdoc}
     */
    public function fetchAll($fetchMode = null, $fetchArgument = null, $ctorArgs = null)

The method parameter $fetchArgument is never used

The method parameter $ctorArgs is never used

    {
        $result = [];

        foreach ($this->statements as $statement) {
            $result = array_merge($result, $statement->fetchAll($fetchMode));

The function Doctrine\DBAL\Driver\ResultStatement::fetchAll() has been deprecated: Use fetchAllNumeric(), fetchAllAssociative() or fetchFirstColumn() instead.

        }

        return $result;
    }

    /**

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

     * {@inheritdoc}
     */
    public function fetchColumn($columnIndex = 0)
    {
        $result = [];

        foreach ($this->statements as $statement) {
            $result[] = $statement->fetchColumn($columnIndex);

The function Doctrine\DBAL\Driver\ResultStatement::fetchColumn() has been deprecated: Use fetchOne() instead.

        }

        //TODO change l'interface de la méthode !

Comments for TODO tasks are often forgotten in the code; it might be better to use a dedicated issue tracker.

There must be no blank line following an inline comment

No space found before comment text; expected "// TODO change l'interface de la méthode !" but found "//TODO change l'interface de la méthode !"

        return $result;
    }
}