smi2 / phpClickHouse

src/Statement.php

<?php

namespace ClickHouseDB;

use ClickHouseDB\Exception\ClickHouseUnavailableException;
use ClickHouseDB\Exception\DatabaseException;
use ClickHouseDB\Exception\QueryException;
use ClickHouseDB\Query\Query;
use ClickHouseDB\Transport\CurlerRequest;
use ClickHouseDB\Transport\CurlerResponse;

class Statement implements \Iterator
{
    private const CLICKHOUSE_ERROR_REGEX = "%Code:\s(\d+)\.\s*DB::Exception\s*:\s*(.*)(?:,\s*e\.what|\(version).*%ius";

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$_rawData with single line content, use one-line comment instead.

     * @var string|mixed
     */
    private $_rawData;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$_http_code with single line content, use one-line comment instead.

     * @var int
     */
    private $_http_code = -1;

Member variable "_http_code" is not in valid camel caps format

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$_request with single line content, use one-line comment instead.

     * @var CurlerRequest
     */
    private $_request = null;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$_init with single line content, use one-line comment instead.

     * @var bool
     */
    private $_init = false;

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

    /**
     * @var mixed
     */
    private $format;

    /**
     * @var string
     */
    private $sql = '';

    /**
     * @var array
     */
    private $meta;

    /**
     * @var array
     */
    private $totals;

    /**
     * @var array
     */
    private $extremes;

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

    /**
     * @var bool|integer
     */
    private $rows_before_limit_at_least = false;

Member variable "rows_before_limit_at_least" is not in valid camel caps format

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

Member variable "array_data" is not in valid camel caps format

    /**
     * @var array|null
     */
    private $statistics = null;

    /**
     * @var int
     */
    public $iterator = 0;


    public function __construct(CurlerRequest $request)
    {
        $this->_request = $request;
        $this->format = $this->_request->getRequestExtendedInfo('format');
        $this->query = $this->_request->getRequestExtendedInfo('query');

Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space

        $this->sql = $this->_request->getRequestExtendedInfo('sql');

Equals sign not aligned with surrounding assignments; expected 6 spaces but found 1 space

    }

    /**
     * @return CurlerRequest
     */
    public function getRequest()
    {
        return $this->_request;
    }

    /**
     * @return CurlerResponse
     * @throws Exception\TransportException
     */
    private function response()
    {
        return $this->_request->response();
    }

    /**
     * @return mixed
     * @throws Exception\TransportException
     */
    public function responseInfo()
    {
        return $this->response()->info();
    }

    /**
     * @return mixed|string
     */
    public function sql()
    {
        return $this->sql;
    }

    /**
     * @param string $body
     * @return array|bool

@return annotation of method \ClickHouseDB\Statement::parseErrorClickHouse() does not specify type hint for items of its traversable return value.

     */
    private function parseErrorClickHouse(string $body)
    {
        $body = trim($body);

Function trim() should not be referenced via a fallback global name, but via a use statement.

        $matches = [];

        // Code: 115. DB::Exception: Unknown setting readonly[0], e.what() = DB::Exception
        // Code: 192. DB::Exception: Unknown user x, e.what() = DB::Exception
        // Code: 60. DB::Exception: Table default.ZZZZZ doesn't exist., e.what() = DB::Exception
        // Code: 516. DB::Exception: test_username: Authentication failed: password is incorrect or there is no user with such name. (AUTHENTICATION_FAILED) (version 22.8.3.13 (official build))

        if (preg_match(self::CLICKHOUSE_ERROR_REGEX, $body, $matches)) {

Function preg_match() should not be referenced via a fallback global name, but via a use statement.

            return ['code' => $matches[1], 'message' => $matches[2]];
        }

        return false;
    }

    private function hasErrorClickhouse(string $body): bool {

        return preg_match(self::CLICKHOUSE_ERROR_REGEX, $body) === 1;
    }

    /**
     * @return bool

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    public function error()
    {
        if (!$this->isError()) {

Expected 1 space after NOT operator; 0 found

            return false;
        }

        $body = $this->response()->body();

Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space

        $error_no = $this->response()->error_no();

The variable $error_no should be in camel caps format.

        $error = $this->response()->error();

Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space

        $dumpStatement = false;
        if (!$error_no && !$error) {
            $parse = $this->parseErrorClickHouse($body);

            if ($parse) {

The expression $parse 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.

$parse is a non-empty array, thus is always true.

                throw new DatabaseException($parse['message'] . "\nIN:" . $this->sql(), $parse['code']);
            } else {
                $code = $this->response()->http_code();

Equals sign not aligned with surrounding assignments; expected 10 spaces but found 1 space

                $message = "HttpCode:" . $this->response()->http_code() . " ; " . $this->response()->error() . " ;" . $body;
                $dumpStatement = true;
            }
        } else {
            $code = $error_no;

The variable $error_no should be in camel caps format.

Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space

            $message = $this->response()->error();
        }

        $exception = new QueryException($message, $code);
        if ($code === CURLE_COULDNT_CONNECT) {
            $exception = new ClickHouseUnavailableException($message, $code);
        }

        if ($dumpStatement) {
            $exception->setRequestDetails($this->_request->getDetails());
            $exception->setResponseDetails($this->response()->getDetails());
        }

        throw $exception;
    }

    /**
     * @return bool

Method \ClickHouseDB\Statement::isError() has useless @return annotation.

     * @throws Exception\TransportException
     */
    public function isError(): bool
    {
        if ($this->response()->http_code() !== 200) {
            return true;
        }

        if ($this->response()->error_no()) {
            return true;
        }

        if ($this->hasErrorClickhouse($this->response()->body())) {
            return true;
        }

        return false;
    }

    private function check(): bool
    {
        if (!$this->_request->isResponseExists()) {

Expected 1 space after NOT operator; 0 found

            throw QueryException::noResponse();
        }

        if ($this->isError()) {
            $this->error();
        }

        return true;
    }

    /**
     * @return bool
     */
    public function isInited()
    {
        return $this->_init;
    }

    /**
     * @return bool

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    private function init()
    {
        if ($this->_init) {
            return false;
        }

        $this->check();

        $this->_rawData = $this->response()->rawDataOrJson($this->format);

        if (!$this->_rawData) {

Expected 1 space after NOT operator; 0 found

            $this->_init = true;
            return false;
        }

        $data = [];
        foreach (['meta', 'data', 'totals', 'extremes', 'rows', 'rows_before_limit_at_least', 'statistics'] as $key) {

            if (isset($this->_rawData[$key])) {

Use early exit to reduce code nesting.

                if ($key=='data') {
                    $data=$this->_rawData[$key];
                } else {
                    $this->{$key} = $this->_rawData[$key];
                }
            }
        }

        if (empty($this->meta)) {
            throw new QueryException('Can`t find meta');
        }

        $isJSONCompact=(stripos($this->format,'JSONCompact')!==false?true:false);
        $this->array_data = [];

Member variable "array_data" is not in valid camel caps format

        foreach ($data as $rows) {
            $r = [];

            if ($isJSONCompact) {
                $r[] = $rows;
            } else {
                foreach ($this->meta as $meta) {
                    $r[$meta['name']] = $rows[$meta['name']];
                }
            }

            $this->array_data[] = $r;

Member variable "array_data" is not in valid camel caps format

        }

        $this->_init = true;

        return true;
    }

    /**
     * @return array
     * @throws \Exception
     */
    public function extremes()
    {
        $this->init();
        return $this->extremes;
    }

    /**
     * @return mixed
     * @throws Exception\TransportException
     */
    public function totalTimeRequest()
    {
        $this->check();
        return $this->response()->total_time();

    }

    /**
     * @return array
     * @throws \Exception
     */
    public function extremesMin()
    {
        $this->init();

        if (empty($this->extremes['min'])) {
            return [];
        }

        return $this->extremes['min'];
    }

    /**
     * @return array
     * @throws \Exception
     */
    public function extremesMax()
    {
        $this->init();

        if (empty($this->extremes['max'])) {
            return [];
        }

        return $this->extremes['max'];
    }

    /**
     * @return array
     * @throws Exception\TransportException
     */
    public function totals()
    {
        $this->init();
        return $this->totals;
    }

    /**

Empty comment

     *
     */
    public function dump()
    {
        $this->_request->dump();
        $this->response()->dump();
    }

    /**
     * @return bool|int
     * @throws Exception\TransportException
     */
    public function countAll()
    {
        $this->init();
        return $this->rows_before_limit_at_least;
    }

    /**
     * @param bool $key
     * @return array|mixed|null
     * @throws Exception\TransportException
     */
    public function statistics($key = false)
    {
        $this->init();

        if (!is_array($this->statistics)) {
            return null;
        }

        if (!$key) return $this->statistics;

        if (!isset($this->statistics[$key])) {
            return null;
        }

        return $this->statistics[$key];

    }

    /**
     * @return int

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    public function count()
    {
        $this->init();
        return $this->rows;
    }

    /**
     * @return mixed|string
     * @throws Exception\TransportException
     */
    public function rawData()
    {
        if ($this->_init) {
            return $this->_rawData;
        }

        $this->check();

        return $this->response()->rawDataOrJson($this->format);
    }

    /**

Empty comment

     *
     */
    public function resetIterator()
    {
        $this->iterator=0;
    }

    public function fetchRow($key = null)
    {
        $this->init();

        $position=$this->iterator;

        if (!isset($this->array_data[$position])) {
            return null;
        }

        $this->iterator++;

        if (!$key) {

Expected 1 lines after "if", found 0.

            return $this->array_data[$position];
        }
        if (!isset($this->array_data[$position][$key])) {
            return null;
        }

        return $this->array_data[$position][$key];

    }
    /**
     * @param string $key
     * @return mixed|null
     * @throws Exception\TransportException
     */
    public function fetchOne($key = null)
    {
        $this->init();
        if (!isset($this->array_data[0])) {
            return null;
        }

        if (!$key) {
            return $this->array_data[0];
        }

        if (!isset($this->array_data[0][$key])) {
            return null;
        }

        return $this->array_data[0][$key];
    }

    /**
     * @param string|null $path
     * @return array

@return annotation of method \ClickHouseDB\Statement::rowsAsTree() does not specify type hint for items of its traversable return value.

     * @throws Exception\TransportException
     */
    public function rowsAsTree($path)
    {
        $this->init();

        $out = [];
        foreach ($this->array_data as $row) {

Member variable "array_data" is not in valid camel caps format

            $d = $this->array_to_tree($row, $path);

Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space

            $out = array_replace_recursive($d, $out);

Function array_replace_recursive() should not be referenced via a fallback global name, but via a use statement.

        }

        return $out;
    }

    /**
     * Return size_upload,upload_content,speed_upload,time_request
     *
     * @return array

Incorrect annotations group.

@return annotation of method \ClickHouseDB\Statement::info_upload() does not specify type hint for items of its traversable return value.

     * @throws Exception\TransportException
     */
    public function info_upload()
    {
        $this->check();
        return [
            'size_upload'    => $this->response()->size_upload(),
            'upload_content' => $this->response()->upload_content_length(),
            'speed_upload'   => $this->response()->speed_upload(),
            'time_request'   => $this->response()->total_time()
        ];
    }

    /**
     * Return size_upload,upload_content,speed_upload,time_request,starttransfer_time,size_download,speed_download
     *
     * @return array
     * @throws Exception\TransportException
     */
    public function info()
    {
        $this->check();
        return [
            'starttransfer_time'    => $this->response()->starttransfer_time(),
            'size_download'    => $this->response()->size_download(),
            'speed_download'    => $this->response()->speed_download(),
            'size_upload'    => $this->response()->size_upload(),
            'upload_content' => $this->response()->upload_content_length(),
            'speed_upload'   => $this->response()->speed_upload(),
            'time_request'   => $this->response()->total_time()
        ];
    }

    /**
     * get format in sql
     * @return mixed
     */
    public function getFormat()
    {
        return $this->format;
    }

    /**
     * @return array

Incorrect annotations group.

@return annotation of method \ClickHouseDB\Statement::rows() does not specify type hint for items of its traversable return value.

     * @throws Exception\TransportException
     */
    public function rows()
    {
        $this->init();
        return $this->array_data;
    }

    /**
     * @return false|string
    */
    public function jsonRows()
    {
        return json_encode($this->rows(), JSON_PRETTY_PRINT);
    }

    /**
     * @param array|string $arr
     * @param null|string|array $path
     * @return array

@return annotation of method \ClickHouseDB\Statement::array_to_tree() does not specify type hint for items of its traversable return value.

     */
    private function array_to_tree($arr, $path = null)

Method name "Statement::array_to_tree" is not in camel caps format

Method \ClickHouseDB\Statement::array_to_tree() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "array".

    {
        if (is_array($path)) {

Function is_array() should not be referenced via a fallback global name, but via a use statement.

            $keys = $path;
        } else {
            $args = func_get_args();

Function func_get_args() should not be referenced via a fallback global name, but via a use statement.

            array_shift($args);

            if (sizeof($args) < 2) {

Function sizeof() should not be referenced via a fallback global name, but via a use statement.

The use of function sizeof() is forbidden; use count() instead

                $separator = '.';
                $keys = explode($separator, $path);
            } else {
                $keys = $args;
            }
        }

        //
        $tree = $arr;
        while (count($keys)) {
            $key = array_pop($keys);

Function array_pop() should not be referenced via a fallback global name, but via a use statement.

            if (isset($arr[$key])) {
                $val = $arr[$key];
            } else {
                $val = $key;
            }

            $tree = array($val => $tree);

Short array syntax must be used to define arrays

        }
        if (!is_array($tree)) {
            return [];
        }
        return $tree;
    }


    public function rewind(): void {
        $this->iterator = 0;
    }

    /**
     * @return mixed
     */
    #[\ReturnTypeWillChange]
    public function current() {
        if (!isset($this->array_data[$this->iterator])) {
            return null;
        }
        return $this->array_data[$this->iterator];
    }

    public function key(): int {
        return $this->iterator;
    }

    public function next(): void {
        ++$this->iterator;
    }

    public function valid(): bool {
        $this->init();
        return isset($this->array_data[$this->iterator]);
    }
}