Stu\Component\Spacecraft\Trait\SpacecraftStateTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Test Coverage

Coverage

56.25%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	displayNbsActions()	0	4	2
A	isUnderRepair()	0	4	2
A	getHealthPercentage()	0	5	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\SpacecraftStateEnum;
use Stu\Component\Spacecraft\System\SpacecraftSystemTypeEnum;

trait SpacecraftStateTrait
{
    use SpacecraftTrait;
    use SpacecraftSystemExistenceTrait;

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

    public function isUnderRepair(): bool
    {
        return $this->getThis()->getState() === SpacecraftStateEnum::REPAIR_ACTIVE
            || $this->getThis()->getState() === SpacecraftStateEnum::REPAIR_PASSIVE;
    }

    public function getHealthPercentage(): float
    {
        $self = $this->getThis();
        return ($self->getHull() + $self->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();
    }
}