Brazanation\Documents\Cns

Complexity

Total Complexity

5

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	createFromString()	0	4	1
A	format()	0	4	1
A	calculateDigit()	0	12	2

<?php

namespace Brazanation\Documents;

use Brazanation\Documents\Cns\CnsCalculator;
use Brazanation\Documents\Cns\TemporaryCalculator;

final class Cns extends AbstractDocument
{
    const LENGTH = 15;

    const LABEL = 'CNS';

    const REGEX = '/^([\d]{3})([\d]{4})([\d]{4})([\d]{4})$/';

    const FORMAT = '$1 $2 $3 $4';

    const NUMBER_OF_DIGITS = 1;

    /**
     * CNS constructor.
     *
     * @param string $number
     */
    public function __construct($number)
    {
        $number = preg_replace('/\D/', '', $number);
        parent::__construct($number, self::LENGTH, self::NUMBER_OF_DIGITS, self::LABEL);
    }

    public static function createFromString($number)
    {
        return parent::tryCreateFromString(self::class, $number, self::LENGTH, self::NUMBER_OF_DIGITS, self::LABEL);

It seems like you call parent on a different method (tryCreateFromString() instead of createFromString()). Are you sure this is correct? If so, you might want to change this to $this->tryCreateFromString().

    }

    /**
     * {@inheritdoc}
     */
    public function format()
    {
        return preg_replace(self::REGEX, self::FORMAT, "{$this}");
    }

    /**
     * {@inheritdoc}
     *
     * Based on given number, it will decide what kind of calculator will use.
     *
     * For numbers starting with 7, 8 or 9 will use TemporaryCalculator,
     * otherwise CnsCalculator.
     */
    public function calculateDigit($baseNumber)
    {
        $calculator = new CnsCalculator();

        if (in_array(substr($baseNumber, 0, 1), [7, 8, 9])) {
            $calculator = new TemporaryCalculator();
        }

        $digit = $calculator->calculateDigit($baseNumber);

        return "{$digit}";
    }
}