Talentify\ValueObject\Geography\Address\ByCountry\Us\ZipCode::setValue()

Complexity

Conditions	5
Paths	4

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    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);
    }