WithStatement::parse() - Code Metrics - Inspection of "Add WITH support" - phpmyadmin/sql-parser - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#334)

by Antoine

created 2021-03-11 13:47 UTC

WithStatement::parse() C

↳ Parent: WithStatement

Complexity

Conditions	14
Paths	12

Size

Total Lines	79
Code Lines	44

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	14
eloc	44
nc	12
nop	2
dl	0
loc	79
rs	6.2666
c	1
b	0
f	0

How to fix Long Method Complexity

<?php
/**
 * `WITH` statement.
 */

declare(strict_types=1);

namespace PhpMyAdmin\SqlParser\Statements;

use PhpMyAdmin\SqlParser\Components\Array2d;
use PhpMyAdmin\SqlParser\Components\OptionsArray;
use PhpMyAdmin\SqlParser\Components\WithKeyword;
use PhpMyAdmin\SqlParser\Exceptions\ParserException;
use PhpMyAdmin\SqlParser\Statement;
use PhpMyAdmin\SqlParser\Parser;
use PhpMyAdmin\SqlParser\Token;
use PhpMyAdmin\SqlParser\TokensList;
use PhpMyAdmin\SqlParser\Translator;

use function array_slice;
use function count;

/**
 * `WITH` statement.

 *  WITH [RECURSIVE] query_name [ (column_name [,...]) ] AS (SELECT ...) [, ...]
 */
class WithStatement extends Statement
{
    /**
     * Options for `WITH` statements and their slot ID.
     *
     * @var mixed[]
     */
    public static $OPTIONS = ['RECURSIVE' => 1];

    /**
     * The clauses of this statement, in order.
     *
     * @see Statement::$CLAUSES
     *
     * @var mixed[]
     */
    public static $CLAUSES = [
        'WITH' => [
            'WITH',
            2,
        ],
        // Used for options.
        '_OPTIONS' => [
            '_OPTIONS',
            1,
        ],
        'AS' => [
            'AS',
            2,
        ],
    ];

    /** @var WithKeyword[] */
    public $withers = [];

    /**
     * @param Parser     $parser the instance that requests parsing
     * @param TokensList $list   the list of tokens to be parsed
     */
    public function parse(Parser $parser, TokensList $list)
    {
        ++$list->idx; // Skipping `WITH`.

        // parse any options if provided
        $this->options = OptionsArray::parse(
            $parser,
            $list,
            static::$OPTIONS
        );
        ++$list->idx;

        $state = 0;
        $wither = null;

        for (; $list->idx < $list->count; ++$list->idx) {
            /**
             * Token parsed at this moment.
             *
             * @var Token
             */
            $token = $list->tokens[$list->idx];

            // Skipping whitespaces and comments.
            if ($token->type === Token::TYPE_WHITESPACE || $token->type === Token::TYPE_COMMENT) {
                continue;
            }

            if ($token->type === Token::TYPE_NONE) {
                $wither = $token->value;
                $this->withers[$wither] = new WithKeyword($wither);
                $state = 1;
                continue;
            }

            if ($state === 1) {
                if ($token->value === '(') {
                    $this->withers[$wither]->columns = Array2d::parse($parser, $list);
                    continue;
                }

                if ($token->keyword === 'AS') {
                    ++$list->idx;
                    $state = 2;
                    continue;
                }
            } elseif ($state === 2) {
                if ($token->value === '(') {
                    ++$list->idx;
                    $subList = $this->getSubTokenList($list);
                    if ($subList instanceof ParserException) {
                        $parser->errors[] = $subList;
                        continue;
                    }

                    $subParser = new Parser($subList);

                    if (count($subParser->errors)) {
                        foreach ($subParser->errors as $error) {
                            $parser->errors[] = $error;
                        }
                    }

                    $this->withers[$wither]->statement = $subParser;
                    continue;
                }

                if ($token->value === ',') {
                    $list->idx++;
                    $state = 0;
                    continue;
                }

                // nothing else
                break;
            }
        }

        --$list->idx;
    }

    /**
     * {@inheritdoc}
     */
    public function build()
    {
        $str = 'WITH ';

        foreach ($this->withers as $wither) {
            $str .= $str === 'WITH ' ? '' : ', ';
            $str .= WithKeyword::build($wither);
        }

        return $str;
    }

    /**
     * @return ParserException | TokensList
     */
    private function getSubTokenList(TokensList $list)
    {
        $idx = $list->idx;
        /** @var Token $token */
        $token = $list->tokens[$list->idx];
        $openParenthesis = 0;

        while ($list->idx < $list->count) {
            if ($token->value === '(') {
                ++$openParenthesis;
            } elseif ($token->value === ')') {
                if (--$openParenthesis === -1) {
                    break;
                }
            }

            ++$list->idx;
            if (! isset($list->tokens[$list->idx])) {
                break;
            }

            $token = $list->tokens[$list->idx];
        }

        // performance improvement: return the error to avoid a try/catch in the loop
        if ($list->idx === $list->count) {
            --$list->idx;

            return new ParserException(
                Translator::gettext('A closing bracket was expected.'),
                $token
            );
        }

        $length = $list->idx - $idx;

        return new TokensList(array_slice($list->tokens, $idx, $length), $length);
    }
}


1			<?php
2			/**
3			* `WITH` statement.
4			*/
5
6			declare(strict_types=1);
7
8			namespace PhpMyAdmin\SqlParser\Statements;
9
10			use PhpMyAdmin\SqlParser\Components\Array2d;
11			use PhpMyAdmin\SqlParser\Components\OptionsArray;
12			use PhpMyAdmin\SqlParser\Components\WithKeyword;
13			use PhpMyAdmin\SqlParser\Exceptions\ParserException;
14			use PhpMyAdmin\SqlParser\Statement;
15			use PhpMyAdmin\SqlParser\Parser;
16			use PhpMyAdmin\SqlParser\Token;
17			use PhpMyAdmin\SqlParser\TokensList;
18			use PhpMyAdmin\SqlParser\Translator;
19
20			use function array_slice;
21			use function count;
22
23			/**
24			* `WITH` statement.
25
26			* WITH [RECURSIVE] query_name [ (column_name [,...]) ] AS (SELECT ...) [, ...]
27			*/
28			class WithStatement extends Statement
29			{
30			/**
31			* Options for `WITH` statements and their slot ID.
32			*
33			* @var mixed[]
34			*/
35			public static $OPTIONS = ['RECURSIVE' => 1];
36
37			/**
38			* The clauses of this statement, in order.
39			*
40			* @see Statement::$CLAUSES
41			*
42			* @var mixed[]
43			*/
44			public static $CLAUSES = [
45			'WITH' => [
46			'WITH',
47			2,
48			],
49			// Used for options.
50			'_OPTIONS' => [
51			'_OPTIONS',
52			1,
53			],
54			'AS' => [
55			'AS',
56			2,
57			],
58			];
59
60			/** @var WithKeyword[] */
61			public $withers = [];
62
63			/**
64			* @param Parser $parser the instance that requests parsing
65			* @param TokensList $list the list of tokens to be parsed
66			*/
67			public function parse(Parser $parser, TokensList $list)
68			{
69			++$list->idx; // Skipping `WITH`.
70
71			// parse any options if provided
72			$this->options = OptionsArray::parse(
73			$parser,
74			$list,
75			static::$OPTIONS
76			);
77			++$list->idx;
78
79			$state = 0;
80			$wither = null;
81
82			for (; $list->idx < $list->count; ++$list->idx) {
83			/**
84			* Token parsed at this moment.
85			*
86			* @var Token
87			*/
88			$token = $list->tokens[$list->idx];
89
90			// Skipping whitespaces and comments.
91			if ($token->type === Token::TYPE_WHITESPACE \|\| $token->type === Token::TYPE_COMMENT) {
92			continue;
93			}
94
95			if ($token->type === Token::TYPE_NONE) {
96			$wither = $token->value;
97			$this->withers[$wither] = new WithKeyword($wither);
98			$state = 1;
99			continue;
100			}
101
102			if ($state === 1) {
103			if ($token->value === '(') {
104			$this->withers[$wither]->columns = Array2d::parse($parser, $list);
105			continue;
106			}
107
108			if ($token->keyword === 'AS') {
109			++$list->idx;
110			$state = 2;
111			continue;
112			}
113			} elseif ($state === 2) {
114			if ($token->value === '(') {
115			++$list->idx;
116			$subList = $this->getSubTokenList($list);
117			if ($subList instanceof ParserException) {
118			$parser->errors[] = $subList;
119			continue;
120			}
121
122			$subParser = new Parser($subList);
123
124			if (count($subParser->errors)) {
125			foreach ($subParser->errors as $error) {
126			$parser->errors[] = $error;
127			}
128			}
129
130			$this->withers[$wither]->statement = $subParser;
131			continue;
132			}
133
134			if ($token->value === ',') {
135			$list->idx++;
136			$state = 0;
137			continue;
138			}
139
140			// nothing else
141			break;
142			}
143			}
144
145			--$list->idx;
146			}
147
148			/**
149			* {@inheritdoc}
150			*/
151			public function build()
152			{
153			$str = 'WITH ';
154
155			foreach ($this->withers as $wither) {
156			$str .= $str === 'WITH ' ? '' : ', ';
157			$str .= WithKeyword::build($wither);
158			}
159
160			return $str;
161			}
162
163			/**
164			* @return ParserException \| TokensList
165			*/
166			private function getSubTokenList(TokensList $list)
167			{
168			$idx = $list->idx;
169			/** @var Token $token */
170			$token = $list->tokens[$list->idx];
171			$openParenthesis = 0;
172
173			while ($list->idx < $list->count) {
174			if ($token->value === '(') {
175			++$openParenthesis;
176			} elseif ($token->value === ')') {
177			if (--$openParenthesis === -1) {
178			break;
179			}
180			}
181
182			++$list->idx;
183			if (! isset($list->tokens[$list->idx])) {
184			break;
185			}
186
187			$token = $list->tokens[$list->idx];
188			}
189
190			// performance improvement: return the error to avoid a try/catch in the loop
191			if ($list->idx === $list->count) {
192			--$list->idx;
193
194			return new ParserException(
195			Translator::gettext('A closing bracket was expected.'),
196			$token
197			);
198			}
199
200			$length = $list->idx - $idx;
201
202			return new TokensList(array_slice($list->tokens, $idx, $length), $length);
203			}
204			}
205

phpmyadmin / sql-parser

Pull Request — master (#334)

WithStatement::parse() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like