Bindings::compile_binds() - Code Metrics - Inspection of "Fix bindings" - smi2/phpClickHouse - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#84)

by Šimon

created 2018-08-14 14:08 UTC

Bindings::compile_binds() A

↳ Parent: Bindings

Complexity

Conditions	2
Paths	1

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes

Metric	Value
eloc	5
dl	0
loc	9
ccs	5
cts	5
cp	1
rs	10
c	0
b	0
f	0
cc	2
nc	1
nop	3
crap	2

<?php

declare(strict_types=1);

namespace ClickHouseDB\Query\Degeneration;

use ClickHouseDB\Exception\UnsupportedParameterType;
use ClickHouseDB\Query\Degeneration;
use DateTimeInterface;
use function array_map;
use function implode;
use function is_array;
use function is_bool;
use function is_callable;
use function is_float;
use function is_int;
use function is_string;
use function sprintf;

class Bindings implements Degeneration
{
    /**

     * @var array

     */
    protected $bindings = [];

    /**
     * @param array $bindings

     */
    public function bindParams(array $bindings)

    {
        $this->bindings = [];
        foreach ($bindings as $column => $value) {
            $this->bindParam($column, $value);
        }
    }

    /**
     * @param string $column
     * @param mixed  $value
     */
    public function bindParam($column, $value)

    {
        $this->bindings[$column] = $value;
    }

    /**
     * Escape an string
     *
     * @param string $value
     * @return string
     */
    private function escapeString($value)

    {
        return addslashes($value);

    }

    /**
     * Binds a list of values to the corresponding parameters.
     * This is similar to [[bindValue()]] except that it binds multiple values at a time.
     *
     * @param string $sql
     * @param array $binds

     * @param string $pattern
     * @return string
     */
    public function compile_binds($sql, $binds,$pattern)

    {
        return preg_replace_callback($pattern, function($m) use ($binds){

            if(isset($binds[$m[1]])){ // If it exists in our array

                return $binds[$m[1]]; // Then replace it from our array
            }

            return $m[0]; // Otherwise return the whole match (basically we won't change it)
        }, $sql);
    }

    /**
     * Compile Bindings
     *
     * @param string $sql
     * @return mixed
     */
    public function process($sql)

    {
        $bindFormatted=[];
$a = "a";
$ab = "ab";
$abc = "abc";
        $bindRaw=[];
$a = "a";
$ab = "ab";
$abc = "abc";
        foreach ($this->bindings as $key => $value) {
            if (is_array($value)) {
                $valueSet = implode(', ', $value);

                $values = array_map(
                    function ($value) {
                        return $this->formatParameter($value);
                    },
                    $value
                );

                $formattedParameter = implode(',', $values);
            } else {
                $valueSet = $value;
$a = "a";
$ab = "ab";
$abc = "abc";
                $formattedParameter = $this->formatParameter($value);
            }

            if ($formattedParameter !== null) {
                $bindFormatted[$key]=$formattedParameter;

            }

            if ($valueSet !== null) {

                $bindRaw[$key]=$valueSet;

            }
        }

        for ($loop=0;$loop<2;$loop++)

        {
            // dipping in binds
            // example ['A' => '{B}' , 'B'=>':C','C'=>123]
            $sql=$this->compile_binds($sql,$bindRaw,'#{([\w+]+)}#');

        }
        $sql=$this->compile_binds($sql,$bindFormatted,'#:([\w+]+)#');


        return $sql;
    }

    /**
     * @param mixed $value
     * @return mixed
     */
    private function formatParameter($value)
    {
        if ($value instanceof DateTimeInterface) {
            $value = $value->format('Y-m-d H:i:s');
        }

        if (is_float($value) || is_int($value) || is_bool($value) || $value === null) {
            return $value;
        }

        if (is_object($value) && is_callable([$value, '__toString'])) {

            $value = (string) $value;
        }

        if (is_string($value)) {
            return $this->formatStringParameter($this->escapeString($value));
        }

        throw UnsupportedParameterType::new($value);
    }

    /**
     * @return string
     */
    private function formatStringParameter($value)

    {
        return sprintf("'%s'", $value);
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace ClickHouseDB\Query\Degeneration;
6
7		use ClickHouseDB\Exception\UnsupportedParameterType;
8		use ClickHouseDB\Query\Degeneration;
9		use DateTimeInterface;
10		use function array_map;
11		use function implode;
12		use function is_array;
13		use function is_bool;
14		use function is_callable;
15		use function is_float;
16		use function is_int;
17		use function is_string;
18		use function sprintf;
19
20		class Bindings implements Degeneration
21		{
22		/**
		0 ignored issues – show introduced 2018-08-14 14:10 UTC by Report Bug Copy Issue Report Found multi-line comment for property \ClickHouseDB\Query\Degeneration\Bindings::$bindings with single line content, use one-line comment instead. Loading history...
23		* @var array
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report @var annotation of property \ClickHouseDB\Query\Degeneration\Bindings::$bindings does not specify type hint for its items. Loading history...
24		*/
25		protected $bindings = [];
26
27		/**
28		* @param array $bindings
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report @param annotation of method \ClickHouseDB\Query\Degeneration\Bindings::bindParams() does not specify type hint for items of its traversable parameter $bindings. Loading history...
29		*/
30	57	public function bindParams(array $bindings)
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::bindParams() does not have void return type hint. Loading history...
31		{
32	57	$this->bindings = [];
33	57	foreach ($bindings as $column => $value) {
34	22	$this->bindParam($column, $value);
35		}
36	57	}
37
38		/**
39		* @param string $column
40		* @param mixed $value
41		*/
42	22	public function bindParam($column, $value)
		0 ignored issues – show introduced 2018-08-14 14:10 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::bindParam() does not have parameter type hint for its parameter $column but it should be possible to add it based on @param annotation "string". Loading history... introduced 2018-08-14 14:10 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::bindParam() does not have void return type hint. Loading history...
43		{
44	22	$this->bindings[$column] = $value;
45	22	}
46
47		/**
48		* Escape an string
49		*
50		* @param string $value
51		* @return string
52		*/
53	17	private function escapeString($value)
		0 ignored issues – show introduced 2018-08-14 14:10 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::escapeString() does not have parameter type hint for its parameter $value but it should be possible to add it based on @param annotation "string". Loading history... introduced 2018-08-14 14:10 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::escapeString() does not have return type hint for its return value but it should be possible to add it based on @return annotation "string". Loading history...
54		{
55	17	return addslashes($value);
		0 ignored issues – show introduced 2018-08-14 14:10 UTC by Report Bug Copy Issue Report Function addslashes() should not be referenced via a fallback global name, but via a use statement. Loading history...
56		}
57
58		/**
59		* Binds a list of values to the corresponding parameters.
60		* This is similar to [[bindValue()]] except that it binds multiple values at a time.
61		*
62		* @param string $sql
63		* @param array $binds
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report @param annotation of method \ClickHouseDB\Query\Degeneration\Bindings::compile_binds() does not specify type hint for items of its traversable parameter $binds. Loading history... Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Expected 2 spaces after parameter type; 1 found Loading history...
64		* @param string $pattern
65		* @return string
66		*/
67	57	public function compile_binds($sql, $binds,$pattern)
		0 ignored issues – show Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method name "Bindings::compile_binds" is not in camel caps format Loading history... introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::compile_binds() does not have parameter type hint for its parameter $sql but it should be possible to add it based on @param annotation "string". Loading history... introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::compile_binds() does not have parameter type hint for its parameter $binds but it should be possible to add it based on @param annotation "array". Loading history... introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::compile_binds() does not have parameter type hint for its parameter $pattern but it should be possible to add it based on @param annotation "string". Loading history... introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::compile_binds() does not have return type hint for its return value but it should be possible to add it based on @return annotation "string". Loading history... Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Expected 1 space between comma and argument "$pattern"; 0 found Loading history...
68		{
69		return preg_replace_callback($pattern, function($m) use ($binds){
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Function preg_replace_callback() should not be referenced via a fallback global name, but via a use statement. Loading history... Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Expected 1 space after FUNCTION keyword; 0 found Loading history... Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Expected 1 space before opening brace; found 0 Loading history...
70	21	if(isset($binds[$m[1]])){ // If it exists in our array
		0 ignored issues – show Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Expected 1 space(s) after IF keyword; 0 found Loading history...
71	21	return $binds[$m[1]]; // Then replace it from our array
72		}
73
74	3	return $m[0]; // Otherwise return the whole match (basically we won't change it)
75	57	}, $sql);
76		}
77
78		/**
79		* Compile Bindings
80		*
81		* @param string $sql
82		* @return mixed
83		*/
84	57	public function process($sql)
		0 ignored issues – show introduced 2018-08-14 14:10 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::process() does not have parameter type hint for its parameter $sql but it should be possible to add it based on @param annotation "string". Loading history...
85		{
86	57	$bindFormatted=[];
		0 ignored issues – show Coding Style introduced 2018-08-14 14:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 1 space but found 0 spaces This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
87	57	$bindRaw=[];
		0 ignored issues – show Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 7 spaces but found 0 spaces This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
88	57	foreach ($this->bindings as $key => $value) {
89	22	if (is_array($value)) {
90	6	$valueSet = implode(', ', $value);
91
92	6	$values = array_map(
93		function ($value) {
94	6	return $this->formatParameter($value);
95	6	},
96	6	$value
97		);
98
99	6	$formattedParameter = implode(',', $values);
100		} else {
101	18	$valueSet = $value;
		0 ignored issues – show Coding Style introduced 2018-08-14 14:10 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 11 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
102	18	$formattedParameter = $this->formatParameter($value);
103		}
104
105	21	if ($formattedParameter !== null) {
106	21	$bindFormatted[$key]=$formattedParameter;
		0 ignored issues – show Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Equals sign not aligned correctly; expected 1 space but found 0 spaces Loading history...
107		}
108
109	21	if ($valueSet !== null) {
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Use early exit to reduce code nesting. Loading history...
110	21	$bindRaw[$key]=$valueSet;
		0 ignored issues – show Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Equals sign not aligned correctly; expected 1 space but found 0 spaces Loading history...
111		}
112		}
113
114	57	for ($loop=0;$loop<2;$loop++)
		0 ignored issues – show Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Expected 1 space after first semicolon of FOR loop; 0 found Loading history... Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Expected 1 space after second semicolon of FOR loop; 0 found Loading history...
115		{
116		// dipping in binds
117		// example ['A' => '{B}' , 'B'=>':C','C'=>123]
118	57	$sql=$this->compile_binds($sql,$bindRaw,'#{([\w+]+)}#');
		0 ignored issues – show Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Equals sign not aligned correctly; expected 1 space but found 0 spaces Loading history...
119		}
120	57	$sql=$this->compile_binds($sql,$bindFormatted,'#:([\w+]+)#');
		0 ignored issues – show Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Equals sign not aligned correctly; expected 1 space but found 0 spaces Loading history...
121
122	57	return $sql;
123		}
124
125		/**
126		* @param mixed $value
127		* @return mixed
128		*/
129	22	private function formatParameter($value)
130		{
131	22	if ($value instanceof DateTimeInterface) {
132	1	$value = $value->format('Y-m-d H:i:s');
133		}
134
135	22	if (is_float($value) \|\| is_int($value) \|\| is_bool($value) \|\| $value === null) {
136	9	return $value;
137		}
138
139	18	if (is_object($value) && is_callable([$value, '__toString'])) {
		0 ignored issues – show introduced 2018-08-14 14:10 UTC by Report Bug Copy Issue Report Function is_object() should not be referenced via a fallback global name, but via a use statement. Loading history...
140	1	$value = (string) $value;
141		}
142
143	18	if (is_string($value)) {
144	17	return $this->formatStringParameter($this->escapeString($value));
145		}
146
147	1	throw UnsupportedParameterType::new($value);
148		}
149
150		/**
151		* @return string
152		*/
153	17	private function formatStringParameter($value)
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::formatStringParameter() does not have parameter type hint nor @param annotation for its parameter $value. Loading history... introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Query\Degeneration\Bindings::formatStringParameter() does not have return type hint for its return value but it should be possible to add it based on @return annotation "string". Loading history...
154		{
155	17	return sprintf("'%s'", $value);
156		}
157		}
158

smi2 / phpClickHouse

Pull Request — master (#84)

Bindings::compile_binds() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like