ParameterSpecBuilder::buildDefaultValue() - Code Metrics - Inspection of "WIP: cleanup and phpstan level 3" - pinepain/js-sandbox - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — wip-public-release ( 7c11a5...54ec9d )

by Bogdan

created 2017-10-01 18:03 UTC

ParameterSpecBuilder::buildDefaultValue() C

↳ Parent: ParameterSpecBuilder

Complexity

Conditions	13
Paths	10

Size

Total Lines	40
Code Lines	21

Duplication

Lines	6
Ratio	15 %

Importance

Changes

Metric	Value
dl	6
loc	40
c	0
b	0
f	0
rs	5.1234
cc	13
eloc	21
nc	10
nop	1

How to fix Complexity

<?php declare(strict_types=1);

/*
 * This file is part of the pinepain/php-v8-js-sandbox PHP library.
 *
 * Copyright (c) 2016-2017 Bogdan Padalko <[email protected]>
 *
 * Licensed under the MIT license: http://opensource.org/licenses/MIT
 *
 * For the full copyright and license information, please view the
 * LICENSE file that was distributed with this source or visit
 * http://opensource.org/licenses/MIT
 */


namespace Pinepain\JsSandbox\Specs\Builder;


use Pinepain\JsSandbox\Extractors\ExtractorDefinitionBuilderException;
use Pinepain\JsSandbox\Extractors\ExtractorDefinitionBuilderInterface;
use Pinepain\JsSandbox\Specs\Builder\Exceptions\ParameterSpecBuilderException;
use Pinepain\JsSandbox\Specs\Parameters\MandatoryParameterSpec;
use Pinepain\JsSandbox\Specs\Parameters\OptionalParameterSpec;
use Pinepain\JsSandbox\Specs\Parameters\ParameterSpecInterface;
use Pinepain\JsSandbox\Specs\Parameters\VariadicParameterSpec;

use function strlen;


class ParameterSpecBuilder implements ParameterSpecBuilderInterface
{
    protected $regexp = '/
        ^
        (?:
            (?<rest>\.{3})
            \s*
        )?
        (?:
            (?<type>(\w+\b(?:\(.*\))?)(?:\s*\|\s*(?-1))*)
            \s*
        )
        (?<name>[_a-z]\w*)
        (?:
            \s* = \s*
            (?<default>
                (?:[+-]?[0-9]+\.?[0-9]*)    # numbers (no exponential notation)
                |
                (?:\\\'[^\\\']*\\\')        # single-quoted string
                |
                (?:\"[^\"]*\")              # double-quoted string
                |
                (?:\[\s*\])                 # empty array
                |
                true | false | null
            )
        )?
        $
        /xi';
    /**
     * @var ExtractorDefinitionBuilderInterface
     */
    private $builder;

    public function __construct(ExtractorDefinitionBuilderInterface $builder)
    {
        $this->builder = $builder;
    }

    /**
     * @param string $definition
     *
     * @return ParameterSpecInterface
     * @throws ParameterSpecBuilderException
     */
    public function build(string $definition): ParameterSpecInterface
    {
        $definition = trim($definition);

        if (!$definition) {
            throw new ParameterSpecBuilderException('Definition must be non-empty string');
        }

        if (preg_match($this->regexp, $definition, $matches)) {
            try {
                if ($matches['rest'] ?? false) {
                    return $this->buildVariadicParameterSpec($matches);
                }

                if ($matches['default'] ?? false) {
                    return $this->buildOptionalParameterSpec($matches);
                }

                return $this->buildMandatoryParameterSpec($matches);
            } catch (ExtractorDefinitionBuilderException $e) {
                throw new ParameterSpecBuilderException("Unable to parse definition because of extractor failure: " . $e->getMessage());
            }
        }

        throw new ParameterSpecBuilderException("Unable to parse definition: '{$definition}'");
    }

    protected function buildVariadicParameterSpec(array $matches): VariadicParameterSpec
    {
        if (isset($matches['default'])) {
            throw new ParameterSpecBuilderException('Variadic parameter should have no default value');
        }

        return new VariadicParameterSpec($matches['name'], $this->builder->build($matches['type']));
    }

    protected function buildOptionalParameterSpec(array $matches): OptionalParameterSpec
    {
        $default = $this->buildDefaultValue($matches['default']);

        return new OptionalParameterSpec($matches['name'], $this->builder->build($matches['type']), $default);
    }

    protected function buildMandatoryParameterSpec(array $matches): MandatoryParameterSpec
    {
        return new MandatoryParameterSpec($matches['name'], $this->builder->build($matches['type']));
    }

    protected function buildDefaultValue(string $definition)
    {
        if (is_numeric($definition)) {
            if (false !== strpos($definition, '.')) {
                return (float)$definition;
            }

            return (int)$definition;
        }

        switch (strtolower($definition)) {
            case 'null':
switch ($expr) {
    case "A": { //wrong
        doSomething();
        break;
    }
    case "B"; //wrong
        doSomething();
        break;
    case "C": //right
        doSomething();
        break;
}
                return null;
            case 'true':
                return true;
            case 'false':
                return false;
        }

        // after this point all expected definition values MUST be at least 2 chars length

        if (strlen($definition) < 2) {
            // Less likely we will ever get here because it should fail at a parsing step, but just in case
            throw new ParameterSpecBuilderException("Unknown default value format '{$definition}'");
        }

        if ('[' == $definition[0] && ']' == $definition[-1]) {
            return [];
        }

        if ('"' == $definition[0] && '"' == $definition[-1]) {

            return trim($definition, '"');
        }
        if ("'" == $definition[0] && "'" == $definition[-1]) {

            return trim($definition, "'");
        }

        // Less likely we will ever get here because it should fail at a parsing step, but just in case
        throw new ParameterSpecBuilderException("Unknown default value format '{$definition}'");
    }
}


1		<?php declare(strict_types=1);
2
3		/*
4		* This file is part of the pinepain/php-v8-js-sandbox PHP library.
5		*
6		* Copyright (c) 2016-2017 Bogdan Padalko <[email protected]>
7		*
8		* Licensed under the MIT license: http://opensource.org/licenses/MIT
9		*
10		* For the full copyright and license information, please view the
11		* LICENSE file that was distributed with this source or visit
12		* http://opensource.org/licenses/MIT
13		*/
14
15
16		namespace Pinepain\JsSandbox\Specs\Builder;
17
18
19		use Pinepain\JsSandbox\Extractors\ExtractorDefinitionBuilderException;
20		use Pinepain\JsSandbox\Extractors\ExtractorDefinitionBuilderInterface;
21		use Pinepain\JsSandbox\Specs\Builder\Exceptions\ParameterSpecBuilderException;
22		use Pinepain\JsSandbox\Specs\Parameters\MandatoryParameterSpec;
23		use Pinepain\JsSandbox\Specs\Parameters\OptionalParameterSpec;
24		use Pinepain\JsSandbox\Specs\Parameters\ParameterSpecInterface;
25		use Pinepain\JsSandbox\Specs\Parameters\VariadicParameterSpec;
26
27		use function strlen;
28
29
30		class ParameterSpecBuilder implements ParameterSpecBuilderInterface
31		{
32		protected $regexp = '/
33		^
34		(?:
35		(?<rest>\.{3})
36		\s*
37		)?
38		(?:
39		(?<type>(\w+\b(?:\(.\))?)(?:\s\\|\s(?-1)))
40		\s*
41		)
42		(?<name>[_a-z]\w*)
43		(?:
44		\s* = \s*
45		(?<default>
46		(?:[+-]?[0-9]+\.?[0-9]*) # numbers (no exponential notation)
47		\|
48		(?:\\\'[^\\\']*\\\') # single-quoted string
49		\|
50		(?:\"[^\"]*\") # double-quoted string
51		\|
52		(?:\[\s*\]) # empty array
53		\|
54		true \| false \| null
55		)
56		)?
57		$
58		/xi';
59		/**
60		* @var ExtractorDefinitionBuilderInterface
61		*/
62		private $builder;
63
64		public function __construct(ExtractorDefinitionBuilderInterface $builder)
65		{
66		$this->builder = $builder;
67		}
68
69		/**
70		* @param string $definition
71		*
72		* @return ParameterSpecInterface
73		* @throws ParameterSpecBuilderException
74		*/
75		public function build(string $definition): ParameterSpecInterface
76		{
77		$definition = trim($definition);
78
79		if (!$definition) {
80		throw new ParameterSpecBuilderException('Definition must be non-empty string');
81		}
82
83		if (preg_match($this->regexp, $definition, $matches)) {
84		try {
85		if ($matches['rest'] ?? false) {
86		return $this->buildVariadicParameterSpec($matches);
87		}
88
89		if ($matches['default'] ?? false) {
90		return $this->buildOptionalParameterSpec($matches);
91		}
92
93		return $this->buildMandatoryParameterSpec($matches);
94		} catch (ExtractorDefinitionBuilderException $e) {
95		throw new ParameterSpecBuilderException("Unable to parse definition because of extractor failure: " . $e->getMessage());
96		}
97		}
98
99		throw new ParameterSpecBuilderException("Unable to parse definition: '{$definition}'");
100		}
101
102		protected function buildVariadicParameterSpec(array $matches): VariadicParameterSpec
103		{
104		if (isset($matches['default'])) {
105		throw new ParameterSpecBuilderException('Variadic parameter should have no default value');
106		}
107
108		return new VariadicParameterSpec($matches['name'], $this->builder->build($matches['type']));
109		}
110
111		protected function buildOptionalParameterSpec(array $matches): OptionalParameterSpec
112		{
113		$default = $this->buildDefaultValue($matches['default']);
114
115		return new OptionalParameterSpec($matches['name'], $this->builder->build($matches['type']), $default);
116		}
117
118		protected function buildMandatoryParameterSpec(array $matches): MandatoryParameterSpec
119		{
120		return new MandatoryParameterSpec($matches['name'], $this->builder->build($matches['type']));
121		}
122
123		protected function buildDefaultValue(string $definition)
124		{
125		if (is_numeric($definition)) {
126		if (false !== strpos($definition, '.')) {
127		return (float)$definition;
128		}
129
130		return (int)$definition;
131		}
132
133		switch (strtolower($definition)) {
134		case 'null':
		0 ignored issues – show Coding Style introduced 2017-10-01 18:09 UTC by Report Bug Copy Issue Report `case` statements should be defined using a colon. As per the PSR-2 coding standard, case statements should not be wrapped in curly braces. There is no need for braces, since each case is terminated by the next `break`. There is also the option to use a semicolon instead of a colon, this is discouraged because many programmers do not even know it works and the colon is universal between programming languages. switch ($expr) { case "A": { //wrong doSomething(); break; } case "B"; //wrong doSomething(); break; case "C": //right doSomething(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
135		return null;
136		case 'true':
137		return true;
138		case 'false':
139		return false;
140		}
141
142		// after this point all expected definition values MUST be at least 2 chars length
143
144		if (strlen($definition) < 2) {
145		// Less likely we will ever get here because it should fail at a parsing step, but just in case
146		throw new ParameterSpecBuilderException("Unknown default value format '{$definition}'");
147		}
148
149		if ('[' == $definition[0] && ']' == $definition[-1]) {
150		return [];
151		}
152
153	View Code Duplication	if ('"' == $definition[0] && '"' == $definition[-1]) {
		0 ignored issues – show Duplication introduced 2017-10-01 18:09 UTC by Report Bug Copy Issue Report This code seems to be duplicated across 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...
154		return trim($definition, '"');
155		}
156	View Code Duplication	if ("'" == $definition[0] && "'" == $definition[-1]) {
		0 ignored issues – show Duplication introduced 2017-10-01 18:09 UTC by Report Bug Copy Issue Report This code seems to be duplicated across 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...
157		return trim($definition, "'");
158		}
159
160		// Less likely we will ever get here because it should fail at a parsing step, but just in case
161		throw new ParameterSpecBuilderException("Unknown default value format '{$definition}'");
162		}
163		}
164

pinepain / js-sandbox

Push — wip-public-release ( 7c11a5...54ec9d )

ParameterSpecBuilder::buildDefaultValue() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like