Talentify\ValueObject\Geography\Address\ByCountry\Us\ZipCode

Complexity

Total Complexity

5

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	setValue()	0	16	5

<?php

declare(strict_types=1);

namespace Talentify\ValueObject\Geography\Address\ByCountry\Us;

use Talentify\ValueObject\Geography\Address\PostalCode;
use Talentify\ValueObject\StringUtils;

/**
 * @see https://en.wikipedia.org/wiki/ZIP_Code
 */
class ZipCode extends PostalCode
{
    public function setValue(string $value): void
    {
        $value = StringUtils::trimSpacesWisely($value);
        $changedValue = StringUtils::removeNonWordCharacters($value);

It seems like $value can also be of type null; however, parameter $value of Talentify\ValueObject\StringUtils::removeNonWordCharacters() does only seem to accept string, maybe add an additional type check?

        $characters = StringUtils::countCharacters($changedValue);

        if ($characters == 4) {
            $value = '0' . $value;
            $characters++;
        }

        if (empty($value) || $characters != 5 && $characters != 9) {
            throw new \InvalidArgumentException(sprintf('The value "%s" is not a valid postal code.', $value));
        }

        parent::setValue($value);
    }
}