Acamposm\IPv4AddressConverter\Converters\BinaryIPv4AddressConverter

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 BinaryIPv4AddressConverter extends BaseAddressConverter
{
    use MutableIPv4Address;

    /**
     * Performs the conversion from the input format to Binary string.
     *
     * @return int|string
     */
    public function convert(): int | string
    {
        return match ($this->inputFormat) {
            IPAddressFormat::DECIMAL => $this->withDotNotation
                ? $this->fromDecimalToDottedBinary($this->address)
                : $this->fromDecimalToBinary($this->address),
            IPAddressFormat::HEXADECIMAL => $this->withDotNotation
                ? $this->fromHexadecimalToDottedBinary($this->address)
                : $this->fromHexadecimalToBinary($this->address),
            IPAddressFormat::LONG => $this->withDotNotation
                ? $this->fromLongToDottedBinary($this->address)
                : $this->fromLongToBinary($this->address),

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

            default => $this->address,
        };
    }
}