Brazanation\Documents\Renavam

Complexity

Total Complexity

6

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

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

<?php

namespace Brazanation\Documents;

final class Renavam extends AbstractDocument
{
    const LENGTH = 11;

    const LABEL = 'Renavam';

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

    const NUMBER_OF_DIGITS = 1;

    /**
     * Renavam constructor.
     *
     * @param string $renavam Only accept numbers
     */
    public function __construct($renavam)
    {
        $renavam = preg_replace('/\D/', '', $renavam);
        $renavam = $this->padNumber($renavam);
        parent::__construct($renavam, 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().

    }

    /**
     * Pad left a number to length(11) with 0(ZERO)
     *
     * @param string $number
     *
     * @return string
     */
    private function padNumber($number)
    {
        if (empty($number)) {
            return '';
        }

        return str_pad($number, self::LENGTH, 0, STR_PAD_LEFT);
    }

    /**
     * {@inheritdoc}
     */
    public function format()
    {
        return preg_replace(self::REGEX, '$1.$2-$3', "{$this}");
    }

    /**
     * {@inheritdoc}
     */
    public function calculateDigit($baseNumber)
    {
        $calculator = new DigitCalculator($baseNumber);
        //$calculator->withMultipliers([3, 2, 9, 8, 7, 6, 5, 4, 3, 2]);

73% of this comment could be valid code. Did you maybe forget this after debugging?

        $calculator->withMultipliers([2, 3, 4, 5, 6, 7, 8, 9, 2, 3]);
        $calculator->replaceWhen('0', 10);
        $calculator->withModule(DigitCalculator::MODULE_11);
        $calculator->multiplySumBy(10);
        $digit = $calculator->calculate();

        return "{$digit}";
    }
}