HasLength::__construct() - Code Metrics - Inspection of "Add "exactly" functionality to HasLength rule and..." - yiisoft/validator - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#274)

unknown

created 2022-07-25 10:16 UTC

HasLength::__construct() A

↳ Parent: HasLength

Complexity

Conditions	1
Paths	1

Size

Total Lines	53
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
eloc	7
c	0
b	0
f	0
dl	0
loc	53
ccs	2
cts	2
cp	1
rs	10
cc	1
nc	1
nop	11
crap	1

How to fix Long Method Many Parameters

<?php

declare(strict_types=1);

namespace Yiisoft\Validator\Rule;

use Attribute;
use Closure;
use JetBrains\PhpStorm\ArrayShape;
use Yiisoft\Validator\Rule\Trait\LimitTrait;
use Yiisoft\Validator\SerializableRuleInterface;
use Yiisoft\Validator\BeforeValidationInterface;
use Yiisoft\Validator\Rule\Trait\BeforeValidationTrait;
use Yiisoft\Validator\Rule\Trait\RuleNameTrait;
use Yiisoft\Validator\ValidationContext;

/**
 * Validates that the value is of certain length.
 *
 * Note, this rule should only be used with strings.
 */
#[Attribute(Attribute::TARGET_PROPERTY | Attribute::IS_REPEATABLE)]
final class HasLength implements SerializableRuleInterface, BeforeValidationInterface
{
    use BeforeValidationTrait;
    use LimitTrait;
    use RuleNameTrait;

    public function __construct(
        /**
         * @var int|null minimum length. null means no minimum length limit.
         *
         * @see $lessThanMinMessage for the customized message for a too short string.
         */
        ?int $min = null,
        /**
         * @var int|null maximum length. null means no maximum length limit.
         *
         * @see $greaterThanMaxMessage for the customized message for a too long string.
         */
        ?int $max = null,
        /**
         * @var int|null exact length. null means no strict comparison. Mutually exclusive with {@see $min} and
         * {@see $max}.
         */
        ?int $exactly = null,
        /**
         * @var string user-defined error message used when the value is not a string.
         */
        private string $message = 'This value must be a string.',
        /**
         * @var string user-defined error message used when the length of the value is smaller than {@see $min}.
         */
        string $lessThanMinMessage = 'This value must contain at least {min, number} {min, plural, one{character} ' .
        'other{characters}}.',
        /**
         * @var string user-defined error message used when the length of the value is greater than {@see $max}.
         */
        string $greaterThanMaxMessage = 'This value must contain at most {max, number} {max, plural, one{character} ' .
        'other{characters}}.',
        string $notExactlyMessage = 'This value must contain exactly {exactly, number} {exactly, plural, ' .
        'one{character} other{characters}}.',
        /**
         * @var string the encoding of the string value to be validated (e.g. 'UTF-8').
         * If this property is not set, application wide encoding will be used.
         */
        private string $encoding = 'UTF-8',
        private bool $skipOnEmpty = false,
        private bool $skipOnError = false,
        /**
         * @var Closure(mixed, ValidationContext):bool|null
         */
        private ?Closure $when = null
    ) {
        $this->initLimitProperties(
            $min,
            $max,
            $exactly,
            $lessThanMinMessage,
            $greaterThanMaxMessage,
            $notExactlyMessage
        );
    }

    /**
     * @return string
     */
    public function getMessage(): string
    {
        return $this->message;
    }

    /**
     * @return string
     */
    public function getEncoding(): string
    {
        return $this->encoding;
    }

    #[ArrayShape([
        'min' => 'int|null',
        'max' => 'int|null',
        'message' => 'string[]',
        'lessThanMinMessage' => 'array',
        'greaterThanMaxMessage' => 'array',
        'notExactlyMessage' => 'array',
        'encoding' => 'string',
        'skipOnEmpty' => 'bool',
        'skipOnError' => 'bool',
    ])]
    public function getOptions(): array
    {
        return array_merge($this->getLimitOptions(), [
            'message' => [
                'message' => $this->message,
            ],
            'encoding' => $this->encoding,
            'skipOnEmpty' => $this->skipOnEmpty,
            'skipOnError' => $this->skipOnError,
        ]);
    }

    public function getHandlerClassName(): string
    {
        return HasLengthHandler::class;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Yiisoft\Validator\Rule;
6
7		use Attribute;
8		use Closure;
9		use JetBrains\PhpStorm\ArrayShape;
10		use Yiisoft\Validator\Rule\Trait\LimitTrait;
11		use Yiisoft\Validator\SerializableRuleInterface;
12		use Yiisoft\Validator\BeforeValidationInterface;
13		use Yiisoft\Validator\Rule\Trait\BeforeValidationTrait;
14		use Yiisoft\Validator\Rule\Trait\RuleNameTrait;
15		use Yiisoft\Validator\ValidationContext;
16
17		/**
18		* Validates that the value is of certain length.
19		*
20		* Note, this rule should only be used with strings.
21		*/
22		#[Attribute(Attribute::TARGET_PROPERTY \| Attribute::IS_REPEATABLE)]
23		final class HasLength implements SerializableRuleInterface, BeforeValidationInterface
24		{
25		use BeforeValidationTrait;
26		use LimitTrait;
27		use RuleNameTrait;
28
29	5	public function __construct(
30		/**
31		* @var int\|null minimum length. null means no minimum length limit.
32		*
33		* @see $lessThanMinMessage for the customized message for a too short string.
34		*/
35		?int $min = null,
36		/**
37		* @var int\|null maximum length. null means no maximum length limit.
38		*
39		* @see $greaterThanMaxMessage for the customized message for a too long string.
40		*/
41		?int $max = null,
42		/**
43		* @var int\|null exact length. null means no strict comparison. Mutually exclusive with {@see $min} and
44		* {@see $max}.
45		*/
46		?int $exactly = null,
47		/**
48		* @var string user-defined error message used when the value is not a string.
49		*/
50		private string $message = 'This value must be a string.',
51		/**
52		* @var string user-defined error message used when the length of the value is smaller than {@see $min}.
53		*/
54		string $lessThanMinMessage = 'This value must contain at least {min, number} {min, plural, one{character} ' .
55		'other{characters}}.',
56		/**
57		* @var string user-defined error message used when the length of the value is greater than {@see $max}.
58		*/
59		string $greaterThanMaxMessage = 'This value must contain at most {max, number} {max, plural, one{character} ' .
60		'other{characters}}.',
61		string $notExactlyMessage = 'This value must contain exactly {exactly, number} {exactly, plural, ' .
62		'one{character} other{characters}}.',
63		/**
64		* @var string the encoding of the string value to be validated (e.g. 'UTF-8').
65		* If this property is not set, application wide encoding will be used.
66		*/
67		private string $encoding = 'UTF-8',
68		private bool $skipOnEmpty = false,
69		private bool $skipOnError = false,
70		/**
71		* @var Closure(mixed, ValidationContext):bool\|null
72		*/
73		private ?Closure $when = null
74		) {
75	5	$this->initLimitProperties(
76		$min,
77		$max,
78		$exactly,
79		$lessThanMinMessage,
80		$greaterThanMaxMessage,
81		$notExactlyMessage
82		);
83		}
84
85		/**
86		* @return string
87		*/
88	5	public function getMessage(): string
89		{
90	5	return $this->message;
91		}
92
93		/**
94		* @return string
95		*/
96	24	public function getEncoding(): string
97		{
98	24	return $this->encoding;
99		}
100
101	3	#[ArrayShape([
102		'min' => 'int\|null',
103		'max' => 'int\|null',
104		'message' => 'string[]',
105		'lessThanMinMessage' => 'array',
106		'greaterThanMaxMessage' => 'array',
107		'notExactlyMessage' => 'array',
108		'encoding' => 'string',
109		'skipOnEmpty' => 'bool',
110		'skipOnError' => 'bool',
111		])]
112		public function getOptions(): array
113		{
114	3	return array_merge($this->getLimitOptions(), [
115		'message' => [
116	3	'message' => $this->message,
117		],
118	3	'encoding' => $this->encoding,
119	3	'skipOnEmpty' => $this->skipOnEmpty,
120	3	'skipOnError' => $this->skipOnError,
121		]);
122		}
123
124	3	public function getHandlerClassName(): string
125		{
126	3	return HasLengthHandler::class;
127		}
128		}
129

yiisoft / validator

Pull Request — master (#274)

HasLength::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Many Parameters

Long Method

Many Parameters

Duplication Side-by-Side

Filter issues like