fgoldoni / QueryBuilder

src/Query.php

<?php

/**
 * Created by PhpStorm.
 * User: Goldoni
 * Date: 31/10/2018
 * Time: 00:53.
 */

namespace Goldoni\Builder;

use Pagerfanta\Pagerfanta;
use PDO;

/**
 * Class Builder.
 */
class Query implements \IteratorAggregate
{
    /**
     * @var array
     */
    private $select;
    /**
     * @var array
     */
    private $insert;
    /**
     * @var array
     */
    private $update;
    /**
     * @var array
     */
    private $delete;
    /**
     * @var array
     */
    private $values;
    /**
     * @var array
     */
    private $set;
    /**
     * @var array
     */
    private $from;
    /**
     * @var array
     */
    private $where = [];
    /**
     * @var array
     */
    private $joins;
    /**
     * @var string
     */
    private $entity;
    /**
     * @var string
     */
    private $group;
    /**
     * @var array
     */
    private $order;
    /**
     * @var int
     */
    private $limit;

    /**
     * @var \PDO
     */
    private $pdo;

    /**
     * @var array
     */
    private $params = [];

    /**
     * Query constructor.
     *
     * @param null|\PDO $pdo
     */
    public function __construct(?PDO $pdo = null)
    {
        $this->pdo = $pdo;
    }

    /**
     * @param string $table
     * @param string $alias
     *
     * @return \Goldoni\Builder\Query
     */
    public function from(string $table, ?string $alias = null): self
    {
        if ($alias) {
            $this->from[$table] = $alias;
        } else {
            $this->from[] = $table;
        }

        return $this;
    }

    /**
     * @param string ...$fields
     *
     * @return \Goldoni\Builder\Query
     */
    public function select(string ...$fields): self
    {
        $this->select = $fields;

        return $this;
    }

    /**
     * @param string $table
     * @param array  $attributes
     *
     * @return \Goldoni\Builder\Query
     */
    public function insert(string $table, ?array $attributes = null): self
    {
        $this->insert = $table;

It seems like $table of type string is incompatible with the declared type array of property $insert.

        if ($attributes) {
            $this->values = $attributes;
        }

        return $this;
    }

    public function value(array $attributes): self
    {
        $this->values = $attributes;

        return $this;
    }

    /**
     * @param string $table
     * @param array  $attributes
     * @param int    $id
     *
     * @return \Goldoni\Builder\Query
     */
    public function update(string $table, ?array $attributes = null, ?int $id = null): self
    {
        $this->update = $table;

It seems like $table of type string is incompatible with the declared type array of property $update.

        if ($id) {

The expression $id of type integer|null is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== null instead.

            $this->where('id = :id');
            $this->params(['id' => $id]);
        }

        if ($attributes) {
            $this->set = $attributes;
        }

        return $this;
    }

    public function set(array $attributes): self
    {
        $this->set = $attributes;

        return $this;
    }

    public function delete(string $table, ?int $id = null): self
    {
        $this->delete = $table;

It seems like $table of type string is incompatible with the declared type array of property $delete.

        if ($id) {

The expression $id of type integer|null is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== null instead.

            $this->where('id = :id');
            $this->params(['id' => $id]);
        }

        return $this;
    }

    /**
     * @param string ...$conditions
     *
     * @return \Goldoni\Builder\Query
     */
    public function where(string ...$conditions): self
    {
        $this->where = array_merge($this->where, $conditions);

        return $this;
    }

    /**
     * @param string $table
     * @param string $condition
     * @param string $type
     *
     * @return \Goldoni\Builder\Query
     */
    public function join(string $table, string $condition, string  $type = 'left'): self
    {
        $this->joins[$type][] = [$table, $condition];

        return $this;
    }

    /**
     * @throws \Exception
     *
     * @return int
     */
    public function count(): int
    {
        $query = clone $this;
        $table = current($this->from);

        return $query->select("COUNT({$table}.id)")->execute()->fetchColumn();
    }

    /**
     * @param string      $column
     * @param null|string $direction
     *
     * @return \Goldoni\Builder\Query
     */
    public function orderBy(string $column, ?string $direction = 'ASC'): self
    {
        $this->order[$column] = $direction;

        return $this;
    }

    /**
     * @param string $column
     *
     * @return \Goldoni\Builder\Query
     */
    public function groupBy(string $column): self
    {
        $this->group = $column;

        return $this;
    }

    /**
     * @param int $limit
     * @param int $offset
     *
     * @return \Goldoni\Builder\Query
     */
    public function limit(int $limit, int $offset = 0): self
    {
        $this->limit = "$offset, $limit";

The property $limit was declared of type integer, but $offset.', '.$limit is of type string. Maybe add a type cast?

        return $this;
    }

    /**
     * @param string $entity
     *
     * @return \Goldoni\Builder\Query
     */
    public function into(string $entity): self
    {
        $this->entity = $entity;

        return $this;
    }

    public function fetchAll(): QueryResult
    {
        return new QueryResult($this->execute()->fetchAll(\PDO::FETCH_ASSOC), $this->entity);
    }

    public function paginate(int $perPage, int $currentPage = 1)
    {
        $paginator = new Paginator($this, $perPage, $currentPage);

The call to Goldoni\Builder\Paginator::__construct() has too many arguments starting with $perPage.

        return (new Pagerfanta($paginator))
            ->setMaxPerPage($perPage)
            ->setCurrentPage($currentPage);
    }

    public function fetch()
    {
        $record = $this->execute()->fetch(\PDO::FETCH_ASSOC);

        if (false === $record) {
            return false;
        }

        if ($this->entity) {
            return Builder::hydrate($record, $this->entity);
        }

        return $record;
    }

    /**
     * @throws \Exception
     */
    public function fetchOrFail()
    {
        $record = $this->fetch();

        if (false === $record) {
            throw new \Exception('No query results for model');
        }

        return $record;
    }

    /**
     * @param array $params
     * @param bool  $merge
     *
     * @return \Goldoni\Builder\Query
     */
    public function params(array $params, bool $merge = true): self
    {
        if ($merge) {
            $this->params = array_merge($this->params, $params);
        } else {
            $this->params = $params;
        }

        return $this;
    }

    /**
     * @return string
     */
    public function __toString()
    {
        $parts = ['SELECT'];

        if ($this->select) {

The expression $this->select of type array 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.

            $parts[] = implode(', ', $this->select);
        } else {
            $parts[] = '*';
        }

        if ($this->insert) {

The expression $this->insert of type array 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.

            $parts = ['INSERT INTO ' . $this->insert];

Are you sure $this->insert of type array can be used in concatenation?

        }

        if ($this->values) {

The expression $this->values of type array 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.

            $parts[] = '(' . implode(', ', array_keys($this->values)) . ')';
            $parts[] = 'VALUES';
            $parts[] = '(' . implode(', ', array_values($this->values)) . ')';
        }

        if ($this->update) {

The expression $this->update of type array 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.

            $parts = ['UPDATE ' . $this->update . ' SET'];

Are you sure $this->update of type array can be used in concatenation?

        }

        if ($this->set) {

The expression $this->set of type array 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.

            $sets = [];

            foreach ($this->set as $key => $value) {
                $sets[] = "$key = $value";
            }
            $parts[] = implode(', ', $sets);
        }

        if ($this->delete) {

The expression $this->delete of type array 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.

            $parts = ['DELETE FROM ' . $this->delete];

Are you sure $this->delete of type array can be used in concatenation?

        }

        if ($this->from) {

The expression $this->from of type array 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.

            $parts[] = 'FROM';
            $parts[] = $this->buildFrom();
        }

        if (!empty($this->where)) {
            $parts[] = 'WHERE';
            $parts[] = '(' . implode(') AND (', $this->where) . ')';
        }

        if (!empty($this->joins)) {
            foreach ($this->joins as $type => $joins) {
                foreach ($joins as [$table, $condition]) {
                    $parts[] = mb_strtoupper($type) . " JOIN $table ON $condition";
                }
            }
        }

        if ($this->order) {

The expression $this->order of type array 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.

            foreach ($this->order as $key => $value) {
                $parts[] = "ORDER BY $key $value";
            }
        }

        if ($this->group) {
            $parts[] = 'GROUP BY ' . $this->group;
        }

        if ($this->limit) {
            $parts[] = 'LIMIT ' . $this->limit;
        }

        return implode(' ', $parts);
    }

    private function buildFrom(): string
    {
        $from = [];

        foreach ($this->from as $key => $value) {
            if (\is_string($key)) {
                $from[] = "$key as $value";
            } else {
                $from[] = $value;
            }
        }

        return implode(', ', $from);
    }

    public function execute()
    {
        if (!empty($this->params)) {
            $statement = $this->pdo->prepare($this->__toString());

            if (!$statement->execute($this->params)) {
                throw new \Exception("Sql Error by execute query: {$this->__toString()}");
            }

            return $statement;
        }

        return $this->pdo->query($this->__toString());
    }

    /**
     * @return \Goldoni\Builder\QueryResult|\Traversable
     */
    public function getIterator()
    {
        return $this->fetchAll();
    }
}