Stu\Module\Ship\Lib\Battle\Party\AbstractBattleParty

Complexity

Total Complexity

17

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Test Coverage

Coverage

45.45%

Importance

Changes	1
Bugs	0	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	createSingleton()	0	3	1
A	isBase()	0	3	1
A	isDefeated()	0	3	1
A	getActiveMembers()	0	10	6
A	getPrivateMessageType()	0	5	2
A	getRandomActiveMember()	0	9	2
A	getUser()	0	3	1
A	getLeader()	0	3	1
A	count()	0	3	1
A	__construct()	0	5	1

<?php

namespace Stu\Module\Ship\Lib\Battle\Party;

use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use RuntimeException;
use Stu\Module\Message\Lib\PrivateMessageFolderSpecialEnum;
use Stu\Module\Ship\Lib\ShipWrapperInterface;
use Stu\Orm\Entity\UserInterface;

abstract class AbstractBattleParty implements BattlePartyInterface
{
    private bool $isBase;
    private UserInterface $user;

    /** @var Collection<int, ShipWrapperInterface> $members */
    private ?Collection $members = null;

    public function __construct(
        protected ShipWrapperInterface $leader
    ) {
        $this->isBase = $leader->get()->isBase();
        $this->user = $leader->get()->getUser();
    }

    /** @return Collection<int, ShipWrapperInterface> */
    protected abstract function initMembers(): Collection;

    public function getUser(): UserInterface
    {
        return $this->user;
    }

    public function getLeader(): ShipWrapperInterface
    {
        return $this->leader;
    }

    public function getActiveMembers(bool $canFire = false, bool $filterDisabled = true): Collection
    {
        if ($this->members === null) {
            $this->members = $this->initMembers();
        }

        return $this->members->filter(

The method filter() does not exist on null.

            fn (ShipWrapperInterface $wrapper) => !$wrapper->get()->isDestroyed()
                && (!$filterDisabled || !$wrapper->get()->isDisabled())
                && (!$canFire || $wrapper->canFire())
        );
    }

    public function getRandomActiveMember(): ShipWrapperInterface
    {
        $activeMembers = $this->getActiveMembers();
        $randomActiveMember = $activeMembers->get(array_rand($activeMembers->toArray()));

It seems like array_rand($activeMembers->toArray()) can also be of type array; however, parameter $key of Doctrine\Common\Collections\ReadableCollection::get() does only seem to accept integer|string, maybe add an additional type check?

        if ($randomActiveMember === null) {
            throw new RuntimeException('isDefeated should be called first');
        }

        return $randomActiveMember;
    }

    public function isDefeated(): bool
    {
        return $this->getActiveMembers()->isEmpty();
    }


    public function isBase(): bool
    {
        return $this->isBase;
    }

    /**
     * @return Collection<int, ShipWrapperInterface>
     */
    protected function createSingleton(ShipWrapperInterface $wrapper): Collection
    {
        return new ArrayCollection([$wrapper->get()->getId() => $wrapper]);
    }

    public function count(): int
    {
        return $this->getActiveMembers()->count();
    }

    public function getPrivateMessageType(): int
    {
        return $this->isBase()
            ? PrivateMessageFolderSpecialEnum::PM_SPECIAL_STATION
            : PrivateMessageFolderSpecialEnum::PM_SPECIAL_SHIP;
    }
}