GroupKeyword::__construct() - Code Metrics - Inspection of "Add GroupKeyword class to fix postgres GROUP BY" - phpmyadmin/sql-parser - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#155)

unknown

created 2017-05-31 16:06 UTC

GroupKeyword::__construct() A

↳ Parent: GroupKeyword

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	5
Ratio	100 %

Importance

Changes

Metric	Value
dl	5
loc	5
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	3
nc	1
nop	2

<?php

/**
 * `GROUP BY` keyword parser.
 */

namespace PhpMyAdmin\SqlParser\Components;

use PhpMyAdmin\SqlParser\Component;
use PhpMyAdmin\SqlParser\Parser;
use PhpMyAdmin\SqlParser\Token;
use PhpMyAdmin\SqlParser\TokensList;

/**
 * `GROUP BY` keyword parser.
 *
 * @category   Keywords
 *
 * @license    https://www.gnu.org/licenses/gpl-2.0.txt GPL-2.0+
 */
class GroupKeyword extends Component

{
    /**
     * The expression that is used for ordering.
     *
     * @var Expression
     */
    public $expr;
    
    /**
     * The order type.
     *
     * @var string
     */
    public $type;

    /**
     * Constructor.
     *
     * @param Expression $expr the expression that we are sorting by

     * @param string     $type the sorting type
     */
    public function __construct($expr = null, $type = '')
    {
        $this->expr = $expr;
        $this->type = $type;
    }

    /**
     * @param Parser     $parser  the parser that serves as context
     * @param TokensList $list    the list of tokens that are being parsed
     * @param array      $options parameters for parsing
     *
     * @return GroupKeyword[]
     */
    public static function parse(Parser $parser, TokensList $list, array $options = array())
    {
        $ret = array();

        $expr = new self();

        /**
         * The state of the parser.
         *
         * Below are the states of the parser.
         *
         *      0 --------------------[ expression ]-------------------> 1
         *
         *      1 ------------------------[ , ]------------------------> 0
         *      1 -------------------[ ASC / DESC ]--------------------> 1
         *
         * @var int
         */
        $state = 0;

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

            // End of statement.
            if ($token->type === Token::TYPE_DELIMITER) {
                break;
            }

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

            if ($state === 0) {
                $expr->expr = Expression::parse($parser, $list);
                $state = 1;
            } elseif ($state === 1) {
                if (($token->type === Token::TYPE_KEYWORD)
                    && (($token->keyword === 'ASC') || ($token->keyword === 'DESC'))
                ) {
                    $expr->type = $token->keyword;
                } elseif (($token->type === Token::TYPE_OPERATOR)
                    && ($token->value === ',')
                ) {
                    if (!empty($expr->expr)) {
                        $ret[] = $expr;
                    }
                    $expr = new self();
                    $state = 0;
                } else {
                    break;
                }
            }
        }

        // Last iteration was not processed.
        if (!empty($expr->expr)) {
            $ret[] = $expr;
        }

        --$list->idx;

        return $ret;
    }

    /**
     * @param GroupKeyword|GroupKeyword[] $component the component to be built
     * @param array                       $options   parameters for building
     *
     * @return string
     */
    public static function build($component, array $options = array())
    {
        if (is_array($component)) {
            return implode(', ', $component);
        }

        return trim($component->expr . ' ' . $component->type);

    }
}


1		<?php
2
3		/**
4		* `GROUP BY` keyword parser.
5		*/
6
7		namespace PhpMyAdmin\SqlParser\Components;
8
9		use PhpMyAdmin\SqlParser\Component;
10		use PhpMyAdmin\SqlParser\Parser;
11		use PhpMyAdmin\SqlParser\Token;
12		use PhpMyAdmin\SqlParser\TokensList;
13
14		/**
15		* `GROUP BY` keyword parser.
16		*
17		* @category Keywords
18		*
19		* @license https://www.gnu.org/licenses/gpl-2.0.txt GPL-2.0+
20		*/
21	View Code Duplication	class GroupKeyword extends Component
		0 ignored issues – show Duplication introduced 2017-05-31 16:23 UTC by Report Bug Copy Issue Report This class seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
22		{
23		/**
24		* The expression that is used for ordering.
25		*
26		* @var Expression
27		*/
28		public $expr;
29
30		/**
31		* The order type.
32		*
33		* @var string
34		*/
35		public $type;
36
37		/**
38		* Constructor.
39		*
40		* @param Expression $expr the expression that we are sorting by
		0 ignored issues – show Documentation introduced 2016-02-12 21:51 UTC by Report Bug Copy Issue Report Should the type for parameter `$expr` not be `Expression\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
41		* @param string $type the sorting type
42		*/
43		public function __construct($expr = null, $type = '')
44		{
45		$this->expr = $expr;
46		$this->type = $type;
47		}
48
49		/**
50		* @param Parser $parser the parser that serves as context
51		* @param TokensList $list the list of tokens that are being parsed
52		* @param array $options parameters for parsing
53		*
54		* @return GroupKeyword[]
55		*/
56		public static function parse(Parser $parser, TokensList $list, array $options = array())
57		{
58		$ret = array();
59
60		$expr = new self();
61
62		/**
63		* The state of the parser.
64		*
65		* Below are the states of the parser.
66		*
67		* 0 --------------------[ expression ]-------------------> 1
68		*
69		* 1 ------------------------[ , ]------------------------> 0
70		* 1 -------------------[ ASC / DESC ]--------------------> 1
71		*
72		* @var int
73		*/
74		$state = 0;
75
76		for (; $list->idx < $list->count; ++$list->idx) {
77		/**
78		* Token parsed at this moment.
79		*
80		* @var Token
81		*/
82		$token = $list->tokens[$list->idx];
83
84		// End of statement.
85		if ($token->type === Token::TYPE_DELIMITER) {
86		break;
87		}
88
89		// Skipping whitespaces and comments.
90		if (($token->type === Token::TYPE_WHITESPACE) \|\| ($token->type === Token::TYPE_COMMENT)) {
91		continue;
92		}
93
94		if ($state === 0) {
95		$expr->expr = Expression::parse($parser, $list);
96		$state = 1;
97		} elseif ($state === 1) {
98		if (($token->type === Token::TYPE_KEYWORD)
99		&& (($token->keyword === 'ASC') \|\| ($token->keyword === 'DESC'))
100		) {
101		$expr->type = $token->keyword;
102		} elseif (($token->type === Token::TYPE_OPERATOR)
103		&& ($token->value === ',')
104		) {
105		if (!empty($expr->expr)) {
106		$ret[] = $expr;
107		}
108		$expr = new self();
109		$state = 0;
110		} else {
111		break;
112		}
113		}
114		}
115
116		// Last iteration was not processed.
117		if (!empty($expr->expr)) {
118		$ret[] = $expr;
119		}
120
121		--$list->idx;
122
123		return $ret;
124		}
125
126		/**
127		* @param GroupKeyword\|GroupKeyword[] $component the component to be built
128		* @param array $options parameters for building
129		*
130		* @return string
131		*/
132		public static function build($component, array $options = array())
133		{
134		if (is_array($component)) {
135		return implode(', ', $component);
136		}
137
138		return trim($component->expr . ' ' . $component->type);
139
140		}
141		}
142

phpmyadmin / sql-parser

Pull Request — master (#155)

GroupKeyword::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like