AmkaValidator::validate() - Code Metrics - Inspection of "Merge pull request #4 from phpgr/patch-4" - thanosKontos/greek-validators-symfony - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 7bcc6f...9b4044 )

by Thanos

created 2018-11-12 07:58 UTC

AmkaValidator::validate() B

↳ Parent: AmkaValidator

Complexity

Conditions	6
Paths	5

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	6.0163

Importance

Changes

Metric	Value
dl	0
loc	22
ccs	12
cts	13
cp	0.9231
rs	8.9457
c	0
b	0
f	0
cc	6
nc	5
nop	2
crap	6.0163

<?php

namespace SymfonyGreekValidation;

use Symfony\Component\Validator\Constraint;
use Symfony\Component\Validator\ConstraintValidator;
use Symfony\Component\Validator\Exception\UnexpectedTypeException;

class AmkaValidator extends ConstraintValidator
{
    public function validate($value, Constraint $constraint)
    {
        if (!$constraint instanceof Amka) {
            throw new UnexpectedTypeException($constraint, Amka::class);
        }

        if (empty($value)) {
            return;
        }

        if (!preg_match('/^[0-9]{11}$/', $value) || $value === '00000000000') {
            $this->buildViolation($value, $constraint);
            return;
        }

        $checksum = $this->calculateChecksum($value);

        if ($checksum % 10 !== 0) {
            $this->buildViolation($value, $constraint);
            return;
        }
    }

    private function calculateChecksum(string $amka): int
    {
        $sum = 0;
        foreach (str_split($amka) as $index => $char) {
            $tempDigit = intval($char);
            if ($this->isOdd($index)) {
                $tempDigit *= 2;
                if ($tempDigit > 9) {
                    $tempDigit -= 9;
                }
            }

            $sum += $tempDigit;
        }

        return $sum;
    }

    private function isOdd($index): bool
    {
        return $index % 2 === 1;
    }

    private function buildViolation($value, $constraint): void
    {
        $this->context->buildViolation($constraint->message)
            ->setParameter('{{ string }}', $value)
            ->addViolation();
    }
}

1		<?php
2
3		namespace SymfonyGreekValidation;
4
5		use Symfony\Component\Validator\Constraint;
6		use Symfony\Component\Validator\ConstraintValidator;
7		use Symfony\Component\Validator\Exception\UnexpectedTypeException;
8
9		class AmkaValidator extends ConstraintValidator
10		{
11	21	public function validate($value, Constraint $constraint)
12		{
13	21	if (!$constraint instanceof Amka) {
14		throw new UnexpectedTypeException($constraint, Amka::class);
15		}
16
17	21	if (empty($value)) {
18	2	return;
19		}
20
21	19	if (!preg_match('/^[0-9]{11}$/', $value) \|\| $value === '00000000000') {
22	3	$this->buildViolation($value, $constraint);
23	3	return;
24		}
25
26	16	$checksum = $this->calculateChecksum($value);
27
28	16	if ($checksum % 10 !== 0) {
29	2	$this->buildViolation($value, $constraint);
30	2	return;
31		}
32	14	}
33
34	16	private function calculateChecksum(string $amka): int
35		{
36	16	$sum = 0;
37	16	foreach (str_split($amka) as $index => $char) {
38	16	$tempDigit = intval($char);
39	16	if ($this->isOdd($index)) {
40	16	$tempDigit *= 2;
41	16	if ($tempDigit > 9) {
42	15	$tempDigit -= 9;
43		}
44		}
45
46	16	$sum += $tempDigit;
47		}
48
49	16	return $sum;
50		}
51
52	16	private function isOdd($index): bool
53		{
54	16	return $index % 2 === 1;
55		}
56
57	5	private function buildViolation($value, $constraint): void
58		{
59	5	$this->context->buildViolation($constraint->message)
60	5	->setParameter('{{ string }}', $value)
61	5	->addViolation();
62		}
63		}

thanosKontos / greek-validators-symfony

Push — master ( 7bcc6f...9b4044 )

AmkaValidator::validate() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like