Acamposm\IPv4AddressConverter\Converters\HexadecimalIPv4AddressConverter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	convert()	0	13	4

<?php

namespace Acamposm\IPv4AddressConverter\Converters;

use Acamposm\IPv4AddressConverter\Enums\IPAddressFormatEnum as IPAddressFormat;
use Acamposm\IPv4AddressConverter\Traits\MutableIPv4AddressTrait as MutableIPv4Address;

class HexadecimalIPv4AddressConverter extends BaseAddressConverter
{
    use MutableIPv4Address;

    /**
     * Performs the conversion from the input format to Hexadecimal string.
     *
     * @return int|string
     */
    public function convert(): int | string
    {
        return match ($this->inputFormat) {
            IPAddressFormat::BINARY => $this->withDotNotation
                ? $this->fromBinaryToDottedHex($this->address)
                : $this->fromBinaryToHex($this->address),
            IPAddressFormat::DECIMAL => $this->withDotNotation
                ? $this->fromDecimalToDottedHex($this->address)
                : $this->fromDecimalToHex($this->address),
            IPAddressFormat::LONG => $this->withDotNotation ?
                $this->fromLongToDottedHex($this->address) :

It seems like $this->address can also be of type string; however, parameter $address of Acamposm\IPv4AddressConverter\Converters\HexadecimalIPv4AddressConverter::fromLongToDottedHex() does only seem to accept integer, maybe add an additional type check?

                $this->fromLongToHex($this->address),

It seems like $this->address can also be of type string; however, parameter $address of Acamposm\IPv4AddressConverter\Converters\HexadecimalIPv4AddressConverter::fromLongToHex() does only seem to accept integer, maybe add an additional type check?

            default => '',
        };
    }
}