PrepareCommand::onNext() - Code Metrics - Inspection of "Current state" - PlasmaPHP/driver-mysql - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( d13c9c...463153 )

by Charlotte

created 2018-11-14 21:27 UTC

PrepareCommand::onNext() B

↳ Parent: PrepareCommand

Complexity

Conditions	10
Paths	6

Size

Total Lines	28
Code Lines	22

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	110

Importance

Changes

Metric	Value
cc	10
eloc	22
nc	6
nop	1
dl	0
loc	28
ccs	0
cts	25
cp	0
crap	110
rs	7.6666
c	0
b	0
f	0

How to fix Complexity

<?php
/**
 * Plasma Driver MySQL component
 * Copyright 2018 PlasmaPHP, All Rights Reserved
 *
 * Website: https://github.com/PlasmaPHP
 * License: https://github.com/PlasmaPHP/driver-mysql/blob/master/LICENSE
*/

namespace Plasma\Drivers\MySQL\Commands;

/**
 * Prepare command.
 * @internal
 */
class PrepareCommand extends PromiseCommand {
    /**
     * The identifier for this command.
     * @var int
     * @source
     */
    const COMMAND_ID = 0x16;
    
    /**
     * @var \Plasma\ClientInterface
     */
    protected $client;
    
    /**
     * @var \Plasma\DriverInterface
     */
    protected $driver;
    
    /**
     * @var string
     */
    protected $query;
    
    /**
     * @var string
     */
    protected $rewrittenQuery;
    
    /**
     * @var array
     */
    protected $rewrittenParams;
    
    /**
     * @var \Plasma\Drivers\MySQL\Messages\PrepareStatementOkMessage
     */
    protected $okResponse;
    
    /**
     * @var \Plasma\ColumnDefinitionInterface[]
     */
    protected $params = array();
    
    /**
     * @var \Plasma\ColumnDefinitionInterface[]
     */
    protected $fields = array();
    
    /**
     * @var \Plasma\StatementInterface|null
     */
    protected $resolveValue;
    
    /**
     * Constructor.
     * @param \Plasma\ClientInterface  $client
     * @param \Plasma\DriverInterface  $driver
     * @param string                   $query
     */
    function __construct(\Plasma\ClientInterface $client, \Plasma\DriverInterface $driver, string $query) {
        parent::__construct();
        
        $this->client = $client;
        $this->driver = $driver;
        $this->query = $query;
        
        [ $this->rewrittenQuery, $this->rewrittenParams ] = \Plasma\Utility::parseParameters($this->query, '?');
    }
    
    /**
     * Get the encoded message for writing to the database connection.
     * @return string
     */
    function getEncodedMessage(): string {
        return \chr(static::COMMAND_ID).$this->rewrittenQuery;
    }
    
    /**
     * Sends the next received value into the command.
     * @param mixed  $value
     * @return void
     * @throws \Plasma\Drivers\MySQL\Messages\ParseException
     */
    function onNext($value): void {
        if($value instanceof \Plasma\Drivers\MySQL\Messages\PrepareStatementOkMessage) {
            $this->okResponse = $value;
        } elseif($value instanceof \Plasma\Drivers\MySQL\ProtocolOnNextCaller) {
            $parsed = $this->handleQueryOnNextCallerColumns($value);
            
            if($this->okResponse->numParams >= \count($this->params)) {
                $this->params[] = $parsed;
            } elseif($this->okResponse->numColumns >= \count($this->fields)) {
                $this->fields[$parsed->getName()] = $parsed;
            } else {
                throw new \Plasma\Drivers\MySQL\Messages\ParseException('Command received more column definition packets than defined');
            }
        } elseif(

            ($value instanceof \Plasma\Drivers\MySQL\Messages\EOFMessage || $value instanceof \Plasma\Drivers\MySQL\Messages\OkResponseMessage)

            && $this->okResponse->numParams <= \count($this->params) && $this->okResponse->numColumns <= \count($this->fields)
        ) {
            $this->finished = true;
            
            $id = $this->okResponse->statementID;
            $queryr = $this->rewrittenQuery;
            $paramsr = $this->rewrittenParams;
            
            $this->resolveValue = new \Plasma\Drivers\MySQL\Statement($this->client, $this->driver, $id, $this->query, $queryr, $paramsr, $this->params, $this->columns);

            $this->deferred->resolve($this->resolveValue);
        } else {
            throw new \Plasma\Drivers\MySQL\Messages\ParseException('Command received value of type '
                .(\is_object($value) ? \get_class($value) : \gettype($value)).' it can not handle');
        }
    }
    
    /**
     * Whether the sequence ID should be resetted.
     * @return bool
     */
    function resetSequence(): bool {
        return true;
    }
}


GitHub Access Token became invalid

Push — master ( d13c9c...463153 )

PrepareCommand::onNext() B

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

1			<?php
2			/**
3			* Plasma Driver MySQL component
4			* Copyright 2018 PlasmaPHP, All Rights Reserved
5			*
6			* Website: https://github.com/PlasmaPHP
7			* License: https://github.com/PlasmaPHP/driver-mysql/blob/master/LICENSE
8			*/
9
10			namespace Plasma\Drivers\MySQL\Commands;
11
12			/**
13			* Prepare command.
14			* @internal
15			*/
16			class PrepareCommand extends PromiseCommand {
17			/**
18			* The identifier for this command.
19			* @var int
20			* @source
21			*/
22			const COMMAND_ID = 0x16;
23
24			/**
25			* @var \Plasma\ClientInterface
26			*/
27			protected $client;
28
29			/**
30			* @var \Plasma\DriverInterface
31			*/
32			protected $driver;
33
34			/**
35			* @var string
36			*/
37			protected $query;
38
39			/**
40			* @var string
41			*/
42			protected $rewrittenQuery;
43
44			/**
45			* @var array
46			*/
47			protected $rewrittenParams;
48
49			/**
50			* @var \Plasma\Drivers\MySQL\Messages\PrepareStatementOkMessage
51			*/
52			protected $okResponse;
53
54			/**
55			* @var \Plasma\ColumnDefinitionInterface[]
56			*/
57			protected $params = array();
58
59			/**
60			* @var \Plasma\ColumnDefinitionInterface[]
61			*/
62			protected $fields = array();
63
64			/**
65			* @var \Plasma\StatementInterface\|null
66			*/
67			protected $resolveValue;
68
69			/**
70			* Constructor.
71			* @param \Plasma\ClientInterface $client
72			* @param \Plasma\DriverInterface $driver
73			* @param string $query
74			*/
75			function __construct(\Plasma\ClientInterface $client, \Plasma\DriverInterface $driver, string $query) {
76			parent::__construct();
77
78			$this->client = $client;
79			$this->driver = $driver;
80			$this->query = $query;
81
82			[ $this->rewrittenQuery, $this->rewrittenParams ] = \Plasma\Utility::parseParameters($this->query, '?');
83			}
84
85			/**
86			* Get the encoded message for writing to the database connection.
87			* @return string
88			*/
89			function getEncodedMessage(): string {
90			return \chr(static::COMMAND_ID).$this->rewrittenQuery;
91			}
92
93			/**
94			* Sends the next received value into the command.
95			* @param mixed $value
96			* @return void
97			* @throws \Plasma\Drivers\MySQL\Messages\ParseException
98			*/
99			function onNext($value): void {
100			if($value instanceof \Plasma\Drivers\MySQL\Messages\PrepareStatementOkMessage) {
101			$this->okResponse = $value;
102			} elseif($value instanceof \Plasma\Drivers\MySQL\ProtocolOnNextCaller) {
103			$parsed = $this->handleQueryOnNextCallerColumns($value);
104
105			if($this->okResponse->numParams >= \count($this->params)) {
106			$this->params[] = $parsed;
107			} elseif($this->okResponse->numColumns >= \count($this->fields)) {
108			$this->fields[$parsed->getName()] = $parsed;
109			} else {
110			throw new \Plasma\Drivers\MySQL\Messages\ParseException('Command received more column definition packets than defined');
111			}
112			} elseif(
			1 ignored issue – show Coding Style introduced 2018-11-13 12:15 UTC by Report Bug Copy Issue Report First condition of a multi-line IF statement must directly follow the opening parenthesis Loading history...
113			($value instanceof \Plasma\Drivers\MySQL\Messages\EOFMessage \|\| $value instanceof \Plasma\Drivers\MySQL\Messages\OkResponseMessage)
			1 ignored issue – show Coding Style introduced 2018-11-13 12:15 UTC by Report Bug Copy Issue Report Each line in a multi-line IF statement must begin with a boolean operator Loading history...
114			&& $this->okResponse->numParams <= \count($this->params) && $this->okResponse->numColumns <= \count($this->fields)
115			) {
116			$this->finished = true;
117
118			$id = $this->okResponse->statementID;
119			$queryr = $this->rewrittenQuery;
120			$paramsr = $this->rewrittenParams;
121
122			$this->resolveValue = new \Plasma\Drivers\MySQL\Statement($this->client, $this->driver, $id, $this->query, $queryr, $paramsr, $this->params, $this->columns);
			0 ignored issues – show Bug Best Practice introduced 2018-11-13 12:15 UTC by Report Bug Copy Issue Report The property `columns` does not exist on `Plasma\Drivers\MySQL\Commands\PrepareCommand`. Did you maybe forget to declare it? Loading history...
123			$this->deferred->resolve($this->resolveValue);
124			} else {
125			throw new \Plasma\Drivers\MySQL\Messages\ParseException('Command received value of type '
126			.(\is_object($value) ? \get_class($value) : \gettype($value)).' it can not handle');
127			}
128			}
129
130			/**
131			* Whether the sequence ID should be resetted.
132			* @return bool
133			*/
134			function resetSequence(): bool {
135			return true;
136			}
137			}
138

PlasmaPHP / driver-mysql

GitHub Access Token became invalid

Push — master ( d13c9c...463153 )

PrepareCommand::onNext() B

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like