ParametersList::getNumberOfRequiredParametersFromArray() - Code Metrics - Inspection of "Do not generate code coverage on nightly" - pinepain/js-sandbox - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — wip-public-release ( 8901b1...a92f7d )

by Bogdan

created 2017-10-01 21:26 UTC

getNumberOfRequiredParametersFromArray() A

↳ Parent: ParametersList

Complexity

Conditions	4
Paths	3

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	4.0466

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	6
cts	7
cp	0.8571
rs	9.2
c	0
b	0
f	0
cc	4
eloc	7
nc	3
nop	1
crap	4.0466

<?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;


use Pinepain\JsSandbox\Specs\Parameters\ParameterSpecInterface;


class ParametersList implements ParametersListInterface
{
    /**
     * @var int
     */
    private $number_of_parameters = 0;

    /**
     * @var int
     */
    private $number_of_required_parameters = 0;

    /**
     * @var bool
     */
    private $variadic = false;

    /**
     * @var ParameterSpecInterface[]
     */
    private $parameters;

    /**
     * @param ParameterSpecInterface[] ...$parameters
     */
    public function __construct(ParameterSpecInterface ...$parameters)
    {
        $this->parameters = $parameters;


        $this->number_of_parameters          = $this->getNumberOfParametersFromArray($parameters);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->number_of_required_parameters = $this->getNumberOfRequiredParametersFromArray($parameters);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->variadic                      = $this->getIsVariadicFromArray($parameters);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * {@inheritdoc}
     */
    public function getNumberOfParameters(): int
    {
        return $this->number_of_parameters;
    }

    /**
     * {@inheritdoc}
     */
    public function getNumberOfRequiredParameters(): int
    {
        return $this->number_of_required_parameters;
    }

    /**
     * {@inheritdoc}
     */
    public function isVariadic(): bool
    {
        return $this->variadic;
    }

    /**
     * {@inheritdoc}
     */
    public function getParameters(): array
    {
        return $this->parameters;
    }

    /**
     * @param ParameterSpecInterface[] $parameters
     *
     * @return int
     */
    protected function getNumberOfParametersFromArray(array $parameters): int
    {
        return count($parameters);
    }

    /**
     * @param ParameterSpecInterface[] $parameters
     *
     * @return int
     */
    protected function getNumberOfRequiredParametersFromArray(array $parameters): int
    {
        $required = 0;

        foreach ($parameters as $p) {
            if ($p->isOptional() || $p->isVariadic()) {
                break;
            }

            $required++;
        }

        return $required;
    }

    /**
     * @param ParameterSpecInterface[] $parameters
     *
     * @return bool
     */
    protected function getIsVariadicFromArray(array $parameters): bool
    {
        foreach ($parameters as $p) {
            if ($p->isVariadic()) {
                return true;
            }
        }

        return false;
    }
}


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;
17
18
19		use Pinepain\JsSandbox\Specs\Parameters\ParameterSpecInterface;
20
21
22		class ParametersList implements ParametersListInterface
23		{
24		/**
25		* @var int
26		*/
27		private $number_of_parameters = 0;
28
29		/**
30		* @var int
31		*/
32		private $number_of_required_parameters = 0;
33
34		/**
35		* @var bool
36		*/
37		private $variadic = false;
38
39		/**
40		* @var ParameterSpecInterface[]
41		*/
42		private $parameters;
43
44		/**
45		* @param ParameterSpecInterface[] ...$parameters
46		*/
47	9	public function __construct(ParameterSpecInterface ...$parameters)
48		{
49	9	$this->parameters = $parameters;
		0 ignored issues – show Documentation Bug introduced 2017-10-01 18:09 UTC by Report Bug Copy Issue Report It seems like `$parameters` of type `array<integer,array<inte...rameterSpecInterface>>>` is incompatible with the declared type `array<integer,object<Pin...arameterSpecInterface>>` of property `$parameters`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
50
51	9	$this->number_of_parameters = $this->getNumberOfParametersFromArray($parameters);
		0 ignored issues – show Documentation introduced 2017-10-01 18:09 UTC by Report Bug Copy Issue Report `$parameters` is of type `array<integer,array<inte...rameterSpecInterface>>>`, but the function expects a `array<integer,object<Pin...arameterSpecInterface>>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
52	9	$this->number_of_required_parameters = $this->getNumberOfRequiredParametersFromArray($parameters);
		0 ignored issues – show Documentation introduced 2017-10-01 18:09 UTC by Report Bug Copy Issue Report `$parameters` is of type `array<integer,array<inte...rameterSpecInterface>>>`, but the function expects a `array<integer,object<Pin...arameterSpecInterface>>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
53	9	$this->variadic = $this->getIsVariadicFromArray($parameters);
		0 ignored issues – show Documentation introduced 2017-10-01 18:09 UTC by Report Bug Copy Issue Report `$parameters` is of type `array<integer,array<inte...rameterSpecInterface>>>`, but the function expects a `array<integer,object<Pin...arameterSpecInterface>>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
54	9	}
55
56		/**
57		* {@inheritdoc}
58		*/
59		public function getNumberOfParameters(): int
60		{
61		return $this->number_of_parameters;
62		}
63
64		/**
65		* {@inheritdoc}
66		*/
67		public function getNumberOfRequiredParameters(): int
68		{
69		return $this->number_of_required_parameters;
70		}
71
72		/**
73		* {@inheritdoc}
74		*/
75		public function isVariadic(): bool
76		{
77		return $this->variadic;
78		}
79
80		/**
81		* {@inheritdoc}
82		*/
83	1	public function getParameters(): array
84		{
85	1	return $this->parameters;
86		}
87
88		/**
89		* @param ParameterSpecInterface[] $parameters
90		*
91		* @return int
92		*/
93	9	protected function getNumberOfParametersFromArray(array $parameters): int
94		{
95	9	return count($parameters);
96		}
97
98		/**
99		* @param ParameterSpecInterface[] $parameters
100		*
101		* @return int
102		*/
103	9	protected function getNumberOfRequiredParametersFromArray(array $parameters): int
104		{
105	9	$required = 0;
106
107	9	foreach ($parameters as $p) {
108	1	if ($p->isOptional() \|\| $p->isVariadic()) {
109		break;
110		}
111
112	1	$required++;
113		}
114
115	9	return $required;
116		}
117
118		/**
119		* @param ParameterSpecInterface[] $parameters
120		*
121		* @return bool
122		*/
123	9	protected function getIsVariadicFromArray(array $parameters): bool
124		{
125	9	foreach ($parameters as $p) {
126	1	if ($p->isVariadic()) {
127	1	return true;
128		}
129		}
130
131	9	return false;
132		}
133		}
134

pinepain / js-sandbox

Push — wip-public-release ( 8901b1...a92f7d )

getNumberOfRequiredParametersFromArray() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like