AlecRabbit\Spinner\Extras\Color\AnsiColorParser

Complexity

Total Complexity

7

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	correctFormat()	0	6	2
A	assertValid()	0	6	1
A	parseColor()	0	9	3

<?php

declare(strict_types=1);


namespace AlecRabbit\Spinner\Extras\Color;

use AlecRabbit\Spinner\Exception\InvalidArgumentException;
use AlecRabbit\Spinner\Extras\Color\Contract\IColor;
use AlecRabbit\Spinner\Extras\Contract\IAnsiColorParser;
use AlecRabbit\Spinner\Extras\Contract\IHexColorToAnsiCodeConverter;

final class AnsiColorParser implements IAnsiColorParser
{
    private const FORMAT = '/^#([0-9a-fA-F]{3}|[0-9a-fA-F]{6})$/';

    public function __construct(
        protected IHexColorToAnsiCodeConverter $converter,
    ) {
    }

    public function parseColor(IColor|null|string $color): string
    {
        if ($color === '' || $color === null) {
            return '';
        }

        self::assertValid($color);

        return $this->converter->convert($color);

It seems like $color can also be of type AlecRabbit\Spinner\Extras\Color\Contract\IColor; however, parameter $color of AlecRabbit\Spinner\Extras\Contract\IHexColorToAnsiCodeConverter::convert() does only seem to accept string, maybe add an additional type check?

    }

    private static function assertValid(IColor|string $color): void
    {
        match (true) {
            self::correctFormat($color) => null,
            default => throw new InvalidArgumentException(
                sprintf('Invalid color format: "%s".,', $color)

It seems like $color can also be of type AlecRabbit\Spinner\Extras\Color\Contract\IColor; however, parameter $values of sprintf() does only seem to accept double|integer|string, maybe add an additional type check?

            ),
        };
    }

    private static function correctFormat(IColor|string $color): bool
    {
        if ($color instanceof IColor) {
            return false; // not supported yet
        }
        return (bool)preg_match(self::FORMAT, $color);
    }
}