Artack\Color\Converter\HEXToRGBConverter

Complexity

Total Complexity

3

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	convert()	0	10	1
A	supportsTo()	0	3	1
A	supportsFrom()	0	3	1

<?php

declare(strict_types=1);

namespace Artack\Color\Converter;

use Artack\Color\Color\Color;
use Artack\Color\Color\HEX;
use Artack\Color\Color\RGB;
use Webmozart\Assert\Assert;

class HEXToRGBConverter implements ConverterInterface
{
    public function convert(Color $color): Color
    {
        /* @var HEX $color */
        Assert::isInstanceOf($color, HEX::class, sprintf('color should be an instance of [%s]', HEX::class));

        $red = hexdec($color->getRed());
        $green = hexdec($color->getGreen());
        $blue = hexdec($color->getBlue());

        return new RGB($red, $green, $blue);

It seems like $red can also be of type double; however, parameter $red of Artack\Color\Color\RGB::__construct() does only seem to accept integer, maybe add an additional type check?

It seems like $green can also be of type double; however, parameter $green of Artack\Color\Color\RGB::__construct() does only seem to accept integer, maybe add an additional type check?

It seems like $blue can also be of type double; however, parameter $blue of Artack\Color\Color\RGB::__construct() does only seem to accept integer, maybe add an additional type check?

    }

    public static function supportsFrom(): string
    {
        return HEX::class;
    }

    public static function supportsTo(): string
    {
        return RGB::class;
    }
}