database\Read

Complexity

Total Complexity

9

Size/Duplication

Total Lines	163
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getRowCount()	0	3	1
A	find()	0	19	3
A	query()	0	13	2
A	all()	0	11	1
A	getResult()	0	3	1
A	__construct()	0	2	1

<?php


namespace database;

use database\DB;

/**
 * Class Read
 * @package src
 */
class Read
{

    /**
     * @var
     */
    private $db;

The private property $db is not used, and could be removed.

    /**
     * @var
     */
    private $table;
    /**
     * @var
     */
    private $read;
    /**
     * @var
     */
    private $statement;
    /**
     * @var
     */
    private $terms;


    /**
     * @var
     */
    private $query;

    /**
     * @var
     */
    private $limit;

The private property $limit is not used, and could be removed.

    /**
     * @var
     */
    private $offset;

The private property $offset is not used, and could be removed.

    /**
     * @var
     */
    private $page;

The private property $page is not used, and could be removed.

    /**
     * @var
     */
    private $link;

The private property $link is not used, and could be removed.

    /**
     * @var
     */
    private $rows;
    /**
     * @var
     */
    private $maxLinks;

The private property $maxLinks is not used, and could be removed.

    /**
     * @var
     */
    private $first;

The private property $first is not used, and could be removed.

    /**
     * @var
     */
    private $last;

The private property $last is not used, and could be removed.

    private $paginator;

The private property $paginator is not used, and could be removed.

    /**
     * @var
     */
    private $result;


    /**
     * Read constructor.
     */
    public function __construct()
    {

    }

    /**
     * @return mixed
     */
    public function getResult()
    {
        return $this->result;
    }

    /**
     * @param $table
     * @param null $parse

Are you sure the doc-type for parameter $parse is correct as it would always require null to be passed?

     * @return array
     */

    public function query(string $query, string $parse = null)
    {
        if ($parse) {
            parse_str($parse, $this->statement);

        }

        $this->query = $query;
        $this->read = DB::connect()->prepare($this->query);
        $this->read->execute($this->statement);
        $this->rows = $this->read->rowCount();
        $this->result = $this->read->fetchAll(\PDO::FETCH_OBJ);
        return $this;

The expression return $this returns the type database\Read which is incompatible with the documented return type array.

    }

    /**
     * @param $table
     * @return $this
     */
    public function all($table)
    {
        $this->table = (string)$table;


        $this->read = DB::connect()->prepare("SELECT * FROM {$this->table}");
        $this->read->execute();
        $this->rows = $this->read->rowCount();
        return $this->result = $this->read->fetchAll(\PDO::FETCH_OBJ);

The expression return $this->result = $this->read->fetchAll(PDO::FETCH_OBJ) returns the type array which is incompatible with the documented return type database\Read.

        return $this;

return $this is not reachable.

    }

    /**
     * @param string $table
     * @param string $terms
     * @param string $parse
     * @return mixed
     */
    public function find(string $table, string $terms, string $parse)
    {
        $this->table = $table;
        $this->terms = $terms;

        if ($parse) {
            parse_str($parse, $this->statement);
        }

        try {
            $this->read = DB::connect()->prepare("SELECT * FROM {$this->table} {$this->terms}");
            $this->read->execute($this->statement);
            $this->rows = $this->read->rowCount();
            return $this->result = $this->read->fetchAll(\PDO::FETCH_OBJ);
        } catch (\PDOException $e) {
            echo $e->getMessage() . " in " . $e->getFile();
        }

        return $this;
    }

    /**
     * @return int
     */
    public function getRowCount()
    {
        return $this->rows;

    }



}