ClearingCheckDigitValidator::validate() - Code Metrics - Inspection of "Don't report as unknown if cleringnummer matches" - byrokrat/banking - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#13)

unknown

created 2017-11-08 16:04 UTC

ClearingCheckDigitValidator::validate() A

↳ Parent: ClearingCheckDigitValidator

Complexity

Conditions	3
Paths	3

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
dl	0
loc	8
ccs	0
cts	8
cp	0
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	4
nc	3
nop	1
crap	12

<?php

namespace byrokrat\banking\Validator;

use byrokrat\banking\Validator;
use byrokrat\banking\AccountNumber;
use byrokrat\banking\Exception\InvalidCheckDigitException;
use byrokrat\checkdigit\Modulo10;

/**
 * Validate clearing number check digits
 */
class ClearingCheckDigitValidator implements Validator
{
    /**
     * @var Modulo10 Checksum calculator
     */
    private $checksum;

    /**
     * Load checksum calculator
     *
     * @param Modulo10 $checksum
     */
    public function __construct(Modulo10 $checksum = null)
    {
        $this->checksum = $checksum ?: new Modulo10;
    }

    /**
     * Validate clearing number check digit
     *
     * If no clearing check digit exists number is considered valid.
     *
     * @param  AccountNumber $number
     * @return null
     * @throws InvalidCheckDigitException If check digit is not valid
     */
    public function validate(AccountNumber $number)
    {
        if ($checkDigit = $number->getClearingCheckDigit()) {
            if (!$this->checksum->isValid($number->getClearingNumber() . $checkDigit)) {
                throw new InvalidCheckDigitException("Invalid clearing number check digit in $number");
            }
        }
    }
}


1			<?php
2
3			namespace byrokrat\banking\Validator;
4
5			use byrokrat\banking\Validator;
6			use byrokrat\banking\AccountNumber;
7			use byrokrat\banking\Exception\InvalidCheckDigitException;
8			use byrokrat\checkdigit\Modulo10;
9
10			/**
11			* Validate clearing number check digits
12			*/
13			class ClearingCheckDigitValidator implements Validator
14			{
15			/**
16			* @var Modulo10 Checksum calculator
17			*/
18			private $checksum;
19
20			/**
21			* Load checksum calculator
22			*
23			* @param Modulo10 $checksum
24			*/
25			public function __construct(Modulo10 $checksum = null)
26			{
27			$this->checksum = $checksum ?: new Modulo10;
28			}
29
30			/**
31			* Validate clearing number check digit
32			*
33			* If no clearing check digit exists number is considered valid.
34			*
35			* @param AccountNumber $number
36			* @return null
37			* @throws InvalidCheckDigitException If check digit is not valid
38			*/
39			public function validate(AccountNumber $number)
40			{
41			if ($checkDigit = $number->getClearingCheckDigit()) {
42			if (!$this->checksum->isValid($number->getClearingNumber() . $checkDigit)) {
43			throw new InvalidCheckDigitException("Invalid clearing number check digit in $number");
44			}
45			}
46			}
47			}
48

byrokrat / banking

Pull Request — master (#13)

ClearingCheckDigitValidator::validate() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like