AppUtils\RGBAColor\FormatsConverter\HEXParser

Complexity

Total Complexity

7

Size/Duplication

Total Lines	94
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	parseHEX6()	0	8	1
A	parseHEX8()	0	8	1
A	parseHEX3()	0	8	1
A	parse()	0	30	4

<?php

declare(strict_types=1);

namespace AppUtils\RGBAColor\FormatsConverter;

use AppUtils\RGBAColor;
use AppUtils\RGBAColor\ColorException;
use AppUtils\RGBAColor\ColorFactory;
use AppUtils\RGBAColor\ColorChannel;

class HEXParser
{
    /**
     * Converts the HEX color string to an 8-Bit color array.
     *
     * @param string $hex
     * @param string $name
     * @return RGBAColor
     *
     * @throws ColorException
     * @see RGBAColor::ERROR_INVALID_HEX_LENGTH
     */
    public function parse(string $hex, string $name='') : RGBAColor
    {
        $hex = ltrim($hex, '#'); // Remove the hash if present
        $hex = strtoupper($hex);
        $length = strlen($hex);

        if($length === 3)
        {
            return $this->parseHEX3($hex, $name);
        }

        if($length === 6)
        {
            return $this->parseHEX6($hex, $name);
        }

        if ($length === 8)
        {
            return $this->parseHEX8($hex, $name);
        }

        throw new ColorException(
            'Invalid HEX color value.',
            sprintf(
                'The hex string [%s] has an invalid length ([%s] characters). '.
                'It must be either 6 characters (RRGGBB) or 8 characters (RRGGBBAA) long.',
                $hex,
                $length
            ),
            RGBAColor::ERROR_INVALID_HEX_LENGTH
        );
    }

    /**
     * Parses a three-letter HEX color string to a color array.
     *
     * @param string $hex
     * @param string $name
     * @return RGBAColor
     */
    private function parseHEX3(string $hex, string $name) : RGBAColor
    {
        return ColorFactory::create(
            ColorChannel::EightBit(hexdec(str_repeat($hex[0], 2))),

It seems like hexdec(str_repeat($hex[0], 2)) can also be of type double; however, parameter $value of AppUtils\RGBAColor\ColorChannel::EightBit() does only seem to accept integer, maybe add an additional type check?

            ColorChannel::EightBit(hexdec(str_repeat($hex[1], 2))),
            ColorChannel::EightBit(hexdec(str_repeat($hex[2], 2))),
            null,
            $name
        );
    }

    /**
     * Parses a six-letter HEX color string to a color array.
     * @param string $hex
     * @param string $name
     * @return RGBAColor
     */
    private function parseHEX6(string $hex, string $name) : RGBAColor
    {
        return ColorFactory::create(
            ColorChannel::EightBit(hexdec(substr($hex, 0, 2))),

It seems like hexdec(substr($hex, 0, 2)) can also be of type double; however, parameter $value of AppUtils\RGBAColor\ColorChannel::EightBit() does only seem to accept integer, maybe add an additional type check?

            ColorChannel::EightBit(hexdec(substr($hex, 2, 2))),
            ColorChannel::EightBit(hexdec(substr($hex, 4, 2))),
            null,
            $name
        );
    }

    /**
     * Parses an eight-letter HEX color string (with alpha) to a color array.
     * @param string $hex
     * @param string $name
     * @return RGBAColor
     */
    private function parseHEX8(string $hex, string $name) : RGBAColor
    {
        return ColorFactory::create(
            ColorChannel::EightBit(hexdec(substr($hex, 0, 2))),

It seems like hexdec(substr($hex, 0, 2)) can also be of type double; however, parameter $value of AppUtils\RGBAColor\ColorChannel::EightBit() does only seem to accept integer, maybe add an additional type check?

            ColorChannel::EightBit(hexdec(substr($hex, 2, 2))),
            ColorChannel::EightBit(hexdec(substr($hex, 4, 2))),
            ColorChannel::EightBit(hexdec(substr($hex, 6, 2))),
            $name
        );
    }
}