BenGorFile\File\Infrastructure\Persistence\Sql\SqlListOfIdsSpecification

Complexity

Total Complexity

10

Size/Duplication

Total Lines	60
Duplicated Lines	28.33 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	buildQuery()	9	9	1
A	buildCount()	8	8	1
A	buildWhere()	0	6	2
A	buildLimit()	0	6	2
A	buildParameters()	0	13	3

<?php

/*
 * This file is part of the BenGorFile package.
 *
 * (c) Beñat Espiña <benatespina@gmail.com>
 * (c) Gorka Laucirica <gorka.lauzirika@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace BenGorFile\File\Infrastructure\Persistence\Sql;

/**
 * @author Beñat Espiña <benatespina@gmail.com>
 */
class SqlListOfIdsSpecification implements SqlQuerySpecification, SqlCountSpecification
{
    private $ids;
    private $offset;
    private $limit;

    public function __construct(array $ids, $offset, $limit)
    {
        $this->ids = $ids;
        $this->offset = $offset;
        $this->limit = $limit;
    }

    public function buildQuery()

This method seems to be duplicated in your project.

    {
        $whereClause = $this->buildWhere();
        $limitClause = $this->buildLimit();
        $query = "SELECT * FROM file f $whereClause ORDER BY f.updated_on DESC $limitClause";
        $parameters = $this->buildParameters();

        return [$query, $parameters];
    }

    public function buildCount()

This method seems to be duplicated in your project.

    {
        $whereClause = $this->buildWhere();
        $limitClause = $this->buildLimit();
        $query = "SELECT COUNT(*) FROM file f $whereClause ORDER BY f.updated_on DESC $limitClause";

        return [$query, $this->buildParameters()];
    }

    private function buildWhere()
    {
        $whereClause = !empty($this->ids) ? 'WHERE f.id IN (:ids) ' : '';

        return $whereClause;
    }

    private function buildLimit()
    {
        $limitClause = $this->limit > 0 ? 'LIMIT :limit OFFSET :offset' : '';

        return $limitClause;
    }

    private function buildParameters()
    {
        $parameters = [];
        if (!empty($this->name)) {

The property name does not exist. Did you maybe forget to declare it?

            $parameters = array_merge([':ids' => implode(',', $this->ids)], $parameters);
        }

        if ($this->limit > 0) {
            $parameters = array_merge([':limit' => $this->limit, ':offset' => $this->offset], $parameters);
        }

        return $parameters;
    }
}