DaveRandom\LibLifxLan\DataTypes\Service

Complexity

Total Complexity

9

Size/Duplication

Total Lines	46
Duplicated Lines	34.78 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	16	16	5
A	getTypeId()	0	4	1
A	getPort()	0	4	1
A	getName()	0	8	2

<?php declare(strict_types=1);

namespace DaveRandom\LibLifxLan\DataTypes;

use DaveRandom\LibLifxLan\Exceptions\InvalidValueException;
use const DaveRandom\LibLifxLan\UINT32_MAX;
use const DaveRandom\LibLifxLan\UINT32_MIN;

final class Service
{
    private $typeId;
    private $port;

    /**
     * @param int $typeId
     * @param int $port
     * @throws InvalidValueException
     */
    public function __construct(int $typeId, int $port)

This method seems to be duplicated in your project.

    {
        if ($typeId < 0 || $typeId > 255) {
            throw new InvalidValueException("Service type ID {$typeId} outside allowable range of 0 - 255");
        }

        // Protocol spec states this is a uint32 rather than a uint16, so allow any uint32 value at the protocol level
        if ($port < UINT32_MIN || $port > UINT32_MAX) {
            throw new InvalidValueException(
                "Port {$port} outside allowable range of " . UINT32_MIN . " - " . UINT32_MAX
            );
        }

        $this->typeId = $typeId;
        $this->port = $port;
    }

    public function getTypeId(): int
    {
        return $this->typeId;
    }

    public function getPort(): int
    {
        return $this->port;
    }

    public function getName(): string
    {
        try {
            return ServiceTypes::parseValue($this->typeId);
        } catch (\InvalidArgumentException $e) {
            return "Unknown({$this->typeId})";
        }
    }
}