BrenoRoosevelt\Validation\Rules\Brazilian\DigitoVerificador

Complexity

Total Complexity

6

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	mod11()	0	11	2
A	evaluate()	0	6	1
A	__construct()	0	3	1
A	mod10()	0	12	2

<?php
declare(strict_types=1);

namespace BrenoRoosevelt\Validation\Rules\Brazilian;

use Attribute;
use BrenoRoosevelt\Validation\AbstractValidation;

#[Attribute(Attribute::TARGET_PROPERTY)]
class DigitoVerificador extends AbstractValidation
{
    public function __construct(?string $message = 'Dígito verificador inválido')
    {
        parent::__construct($message);
    }

    public function evaluate($input, array $context = []): bool
    {
        $numbers = preg_replace('/\D/', '', (string) $input);
        $number = substr($numbers, 0, -1);
        $digit = (int) substr($numbers, -1);
        return self::mod11($number) === $digit;
    }

    public static function mod11($input): int
    {
        $numbers = array_reverse(str_split((string) $input));

It seems like str_split((string)$input) can also be of type true; however, parameter $array of array_reverse() does only seem to accept array, maybe add an additional type check?

        $factor = [2, 3, 4, 5, 6, 7, 8, 9];
        $size = count($factor);
        $i = $sum = 0;
        foreach ($numbers as $number) {
            $sum += $number * $factor[$i++ % $size];
        }

        return 11 - (($sum * 10 ) % 11);
    }

    public static function mod10($input): int
    {
        $numbers = array_reverse(str_split((string) $input));

It seems like str_split((string)$input) can also be of type true; however, parameter $array of array_reverse() does only seem to accept array, maybe add an additional type check?

        $factor = [2, 1];
        $size = count($factor);
        $i = $sum = 0;
        foreach ($numbers as $number) {
            $num = $number * $factor[$i++ % $size];
            $sum += array_sum(str_split((string) $num));

It seems like str_split((string)$num) can also be of type true; however, parameter $array of array_sum() does only seem to accept array, maybe add an additional type check?

        }

        return 10 - ($sum % 10);
    }
}