Stu\Component\Spacecraft\Trait\SpacecraftStateTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Test Coverage

Coverage

64.29%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	displayNbsActions()	0	4	2
A	getHealthPercentage()	0	7	1
A	isWarpPossible()	0	3	2
A	isWarped()	0	3	1
A	isTractoring()	0	3	1

<?php

namespace Stu\Component\Spacecraft\Trait;

use Stu\Component\Spacecraft\System\SpacecraftSystemTypeEnum;

trait SpacecraftStateTrait
{
    use SpacecraftTrait;
    use SpacecraftSystemExistenceTrait;

    public function isWarped(): bool
    {
        return $this->getThis()->getWarpDriveState();
    }

    public function getHealthPercentage(): float
    {
        $self = $this->getThis();
        $condition = $self->getCondition();

        return ($condition->getHull() + $condition->getShield())
            / ($self->getMaxHull() + $self->getMaxShield(true)) * 100;
    }

    public function isTractoring(): bool
    {
        return $this->getThis()->getTractoredShip() !== null;
    }

    public function isWarpPossible(): bool
    {
        return $this->hasSpacecraftSystem(SpacecraftSystemTypeEnum::WARPDRIVE) && $this->getSystem() === null;

It seems like getSystem() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    public function displayNbsActions(): bool
    {
        return !$this->getThis()->isCloaked()
            && !$this->getThis()->isWarped();
    }
}