CustomerGauge\Password\Rule\Length

Complexity

Total Complexity

5

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	__invoke()	0	10	4

<?php

declare(strict_types=1);

namespace CustomerGauge\Password\Rule;

use CustomerGauge\Password\Exception\InvalidLength;
use CustomerGauge\Password\Rule;
use function mb_strlen;

final class Length implements Rule
{
    /** @var int */
    private $min;

    /** @var int|null */
    private $max;

    /** @var string */
    private $encoding;

    public function __construct(int $min, ?int $max = null, string $encoding = 'utf8')
    {
        $this->min      = $min;
        $this->max      = $max;
        $this->encoding = $encoding;
    }

    public function __invoke(string $password) : void
    {
        $length = (int) mb_strlen($password, $this->encoding);

        if ($length < $this->min) {
            throw InvalidLength::requires('min', $this->min, $length);
        }

        if ($this->max && $length > $this->max) {

The expression $this->max of type integer|null is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== null instead.

            throw InvalidLength::requires('max', $this->max, $length);
        }
    }
}