Stu\Lib\Map\Location

Complexity

Total Complexity

11

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	hasAnomaly()	0	9	3
A	__construct()	0	10	6
A	getSectorString()	0	3	1
A	getShips()	0	3	1

<?php

declare(strict_types=1);

namespace Stu\Lib\Map;

use Doctrine\Common\Collections\Collection;
use InvalidArgumentException;
use Stu\Orm\Entity\MapInterface;
use Stu\Orm\Entity\ShipInterface;
use Stu\Orm\Entity\StarSystemMapInterface;

class Location
{
    private MapInterface|StarSystemMapInterface $location;

    public function __construct(?MapInterface $map, ?StarSystemMapInterface $sysMap)
    {
        if (
            $map === null && $sysMap === null

Consider adding parentheses for clarity. Current Interpretation: ($map === null && $sysMap === null) || $map !== null && $sysMap !== null, Probably Intended Meaning: $map === null && ($sysMap === null || $map !== null && $sysMap !== null)

            || $map !== null && $sysMap !== null
        ) {
            throw new InvalidArgumentException('Either map or systemMap has to be filled');
        }

        $this->location = $map !== null ? $map : $sysMap;
    }

    /**
     * @return Collection<int, ShipInterface>
     */
    public function getShips(): Collection
    {
        return $this->location->getShips();
    }

    public function getSectorString(): string
    {
        return $this->location->getSectorString();
    }

    public function hasAnomaly(int $anomalyType): bool
    {
        foreach ($this->location->getAnomalies() as $anomaly) {
            if ($anomaly->getAnomalyType()->getId() === $anomalyType) {
                return true;
            }
        }

        return false;
    }
}