CoffeeCode\DataLayer\DataLayer

Complexity

Total Complexity

37

Size/Duplication

Total Lines	345
Duplicated Lines	0 %

Importance

Changes	14
Bugs	0	Features	0

22 Methods

Rating	Name	Duplication	Size	Complexity
A	data()	0	3	1
A	find()	0	10	2
A	fail()	0	3	1
A	offset()	0	4	1
A	limit()	0	4	1
A	order()	0	4	1
A	group()	0	4	1
A	__get()	0	12	3
A	toCamelCase()	0	5	1
A	count()	0	5	1
A	required()	0	9	3
A	__isset()	0	3	1
A	save()	0	30	6
A	where()	0	5	1
A	destroy()	0	10	2
A	findById()	0	3	1
A	join()	0	12	1
A	safe()	0	5	1
A	select()	0	4	1
A	__set()	0	7	2
A	__construct()	0	6	1
A	fetch()	0	18	4

<?php

namespace CoffeeCode\DataLayer;

use Exception;
use PDO;
use PDOException;
use stdClass;

/**
 * Class DataLayer
 * @package CoffeeCode\DataLayer
 */
abstract class DataLayer
{
    use CrudTrait;

    /** @var string $entity database table */
    private $entity;

    /** @var string $primary table primary key field */
    private $primary;

    /** @var array $required table required fields */
    private $required;

    /** @var string $timestamps control created and updated at */
    private $timestamps;

    /** @var string */
    protected $statement;

    /** @var string */
    protected $join;

    /** @var string */
    protected $params;

    /** @var string */
    protected $group;

    /** @var string */
    protected $order;

    /** @var int */
    protected $limit;

    /** @var int */
    protected $offset;

    /** @var \PDOException|null */
    protected $fail;

    /** @var object|null */
    protected $data;

    /**
     * DataLayer constructor.
     * @param string $entity
     * @param array $required
     * @param string $primary
     * @param bool $timestamps
     */
    public function __construct(string $entity, array $required, string $primary = 'id', bool $timestamps = true)
    {
        $this->entity = $entity;
        $this->primary = $primary;
        $this->required = $required;
        $this->timestamps = $timestamps;

The property $timestamps was declared of type string, but $timestamps is of type boolean. Maybe add a type cast?

    }

    /**
     * @param $name
     * @param $value
     */
    public function __set($name, $value)
    {
        if (empty($this->data)) {
            $this->data = new stdClass();
        }

        $this->data->$name = $value;
    }

    /**
     * @param $name
     * @return bool
     */
    public function __isset($name)
    {
        return isset($this->data->$name);
    }

    /**
     * @param $name
     * @return string|null
     */
    public function __get($name)
    {
        $method = $this->toCamelCase($name);
        if (method_exists($this, $method)) {
            return $this->$method();
        }

        if (method_exists($this, $name)) {
            return $this->$name();
        }

        return ($this->data->$name ?? null);
    }

    /**
     * @return object|null
     */
    public function data(): ?object
    {
        return $this->data;
    }

    /**
     * @return PDOException|Exception|null
     */
    public function fail()
    {
        return $this->fail;
    }

    /**
     * @param string|null $terms
     * @param string|null $params
     * @param string $columns
     * @return DataLayer
     */
    public function find(?string $terms = null, ?string $params = null, string $columns = "*"): DataLayer
    {
        if ($terms) {
            $this->statement = "SELECT {$columns} FROM {$this->entity} WHERE {$terms}";
            parse_str($params, $this->params);

$this->params of type string is incompatible with the type array|null expected by parameter $arr of parse_str().

            return $this;
        }

        $this->statement = "SELECT {$columns} FROM {$this->entity}";
        return $this;
    }

    /**
     * @param int $id
     * @param string $columns
     * @return DataLayer|null
     */
    public function findById(int $id, string $columns = "*"): ?DataLayer
    {
        return $this->find("{$this->primary} = :id", "id={$id}", $columns)->fetch();

The expression return $this->find($this->primary.' = :id', 'id='.$id, $columns)->fetch() could return the type array which is incompatible with the type-hinted return CoffeeCode\DataLayer\DataLayer|null. Consider adding an additional type-check to rule them out.

    }

    /**
     * @param string $column
     * @return DataLayer|null
     */
    public function group(string $column): ?DataLayer
    {
        $this->group = " GROUP BY {$column}";
        return $this;
    }

    /**
     * @param string $columnOrder
     * @return DataLayer|null
     */
    public function order(string $columnOrder): ?DataLayer
    {
        $this->order = " ORDER BY {$columnOrder}";
        return $this;
    }

    /**
     * @param int $limit
     * @return DataLayer|null
     */
    public function limit(int $limit): ?DataLayer
    {
        $this->limit = " LIMIT {$limit}";

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

        return $this;
    }

    /**
     * @param int $offset
     * @return DataLayer|null
     */
    public function offset(int $offset): ?DataLayer
    {
        $this->offset = " OFFSET {$offset}";

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

        return $this;
    }
    
    /**
     * @param string $columns
     * @return DataLayer|null
     */
    public function select(string $columns = "*"): ?DataLayer
    {
        $this->statement = "SELECT {$columns} FROM {$this->entity}";
        return $this;
    }
    /**
     * @param string|null $terms
     * @param string|null $params
     * @return DataLayer|null
     */
    public function where(?string $terms = null, ?string $params = null): ?DataLayer
    {
        $this->statement .= " WHERE {$terms}";
        parse_str($params, $this->params);

$this->params of type string is incompatible with the type array|null expected by parameter $arr of parse_str().

        return $this;
    }

    /**
     * @param DataLayer $otherEntity
     * @param string $column1
     * @param string $operation
     * @param string $column2
     * @param string $type INNER, LEFT (OUTER) or RIGHT (OUTER)
     * @return DataLayer|null
     */
    public function join(
        DataLayer $otherEntity, 
        string $column1,
        string $operation,
        string $column2,
        string $type = 'INNER'
    ): ?DataLayer {
        $join = "${type} JOIN {$otherEntity->entity} on {$column1} {$operation} {$column2}";

        $this->statement .= " " . $join;

        return $this;
    }

    /**
     * @param bool $all
     * @return array|mixed|null
     */
    public function fetch(bool $all = false)
    {
        try {
            $stmt = Connect::getInstance()->prepare($this->statement . $this->group . $this->order . $this->limit . $this->offset);
            $stmt->execute($this->params);

$this->params of type string is incompatible with the type array expected by parameter $input_parameters of PDOStatement::execute().

            if (!$stmt->rowCount()) {
                return null;
            }

            if ($all) {
                return $stmt->fetchAll(PDO::FETCH_CLASS, static::class);
            }
            
            return $stmt->fetchObject(static::class);
        } catch (PDOException $exception) {
            $this->fail = $exception;
            return null;
        }
    }

    /**
     * @return int
     */
    public function count(): int
    {
        $stmt = Connect::getInstance()->prepare($this->statement);
        $stmt->execute($this->params);

$this->params of type string is incompatible with the type array expected by parameter $input_parameters of PDOStatement::execute().

        return $stmt->rowCount();
    }

    /**
     * @return bool
     */
    public function save(): bool
    {
        $primary = $this->primary;
        $id = null;

        try {
            if (!$this->required()) {
                throw new Exception("Preencha os campos necessários");
            }

            /** Update */
            if (!empty($this->data->$primary)) {
                $id = $this->data->$primary;
                $this->update($this->safe(), "{$this->primary} = :id", "id={$id}");
            }

            /** Create */
            if (empty($this->data->$primary)) {
                $id = $this->create($this->safe());
            }

            if (!$id) {
                return false;
            }

            $this->data = $this->findById($id)->data();
            return true;
        } catch (Exception $exception) {
            $this->fail = $exception;

$exception is of type Exception, but the property $fail was declared to be of type PDOException|null. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

            return false;
        }
    }

    /**
     * @return bool
     */
    public function destroy(): bool
    {
        $primary = $this->primary;
        $id = $this->data->$primary;

        if (empty($id)) {
            return false;
        }

        return $this->delete("{$this->primary} = :id", "id={$id}");
    }

    /**
     * @return bool
     */
    protected function required(): bool
    {
        $data = (array)$this->data();
        foreach ($this->required as $field) {
            if (empty($data[$field])) {
                return false;
            }
        }
        return true;
    }

    /**
     * @return array|null
     */
    protected function safe(): ?array
    {
        $safe = (array)$this->data;
        unset($safe[$this->primary]);
        return $safe;
    }


    /**
     * @param string $string
     * @return string
     */
    protected function toCamelCase(string $string): string
    {
        $camelCase = str_replace(' ', '', ucwords(str_replace('_', ' ', $string)));
        $camelCase[0] = strtolower($camelCase[0]);
        return $camelCase;
    }
}