Stu\Module\Tick\Spacecraft\SpacecraftTick

Complexity

Total Complexity

11

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Test Coverage

Coverage

85.11%

Importance

Changes	3
Bugs	2	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	work()	0	5	2
A	workSpacecraftInternal()	0	26	3
A	workSpacecraft()	0	8	1
A	potentialLog()	0	16	2
A	sendMessage()	0	14	2

<?php

namespace Stu\Module\Tick\Spacecraft;

use Stu\Lib\Information\InformationFactoryInterface;
use Stu\Lib\Information\InformationWrapper;
use Stu\Module\Logging\LogTypeEnum;
use Stu\Module\Logging\StuLogger;
use Stu\Module\Message\Lib\PrivateMessageSenderInterface;
use Stu\Module\PlayerSetting\Lib\UserConstants;

The type Stu\Module\PlayerSetting\Lib\UserConstants was not found. Maybe you did not declare it correctly or list all dependencies?

use Stu\Module\Spacecraft\Lib\SpacecraftWrapperFactoryInterface;
use Stu\Module\Spacecraft\Lib\SpacecraftWrapperInterface;
use Stu\Module\Tick\Spacecraft\Handler\SpacecraftTickHandlerInterface;
use Stu\Module\Tick\Spacecraft\ManagerComponent\ManagerComponentInterface;
use Stu\Orm\Entity\Spacecraft;
use Stu\Orm\Repository\SpacecraftRepositoryInterface;

final class SpacecraftTick implements SpacecraftTickInterface, ManagerComponentInterface
{
    /** @param array<SpacecraftTickHandlerInterface> $handlers */
    public function __construct(
        private readonly SpacecraftWrapperFactoryInterface $spacecraftWrapperFactory,
        private readonly PrivateMessageSenderInterface $privateMessageSender,
        private readonly SpacecraftRepositoryInterface $spacecraftRepository,
        private readonly InformationFactoryInterface $informationFactory,
        private readonly array $handlers
    ) {}

    #[\Override]
    public function work(): void
    {
        foreach ($this->spacecraftRepository->getPlayerSpacecraftsForTick() as $spacecraft) {
            $this->workSpacecraft($this->spacecraftWrapperFactory->wrapSpacecraft($spacecraft));
        }
    }

    #[\Override]
    public function workSpacecraft(SpacecraftWrapperInterface $wrapper): void
    {
        $informationWrapper = $this->informationFactory->createInformationWrapper();

        $this->workSpacecraftInternal($wrapper, $informationWrapper);

        $this->sendMessage($wrapper->get(), $informationWrapper);
    }

    private function workSpacecraftInternal(SpacecraftWrapperInterface $wrapper, InformationWrapper $informationWrapper): void
    {
        $spacecraft = $wrapper->get();

        $startTime = microtime(true);

The assignment to $startTime is dead and can be removed.

        try {

            foreach ($this->handlers as $handler) {
                $startTime = microtime(true);
                $handler->handleSpacecraftTick($wrapper, $informationWrapper);
                $this->potentialLog(
                    $spacecraft,
                    $handler,
                    $startTime

It seems like $startTime can also be of type string; however, parameter $startTime of Stu\Module\Tick\Spacecraft\SpacecraftTick::potentialLog() does only seem to accept double, maybe add an additional type check?

                );
            }
        } catch (SpacecraftTickFinishedException) {
            $this->potentialLog(
                $spacecraft,
                $handler,
                $startTime
            );
        }

        $this->spacecraftRepository->save($spacecraft);
    }

    private function potentialLog(Spacecraft $spacecraft, SpacecraftTickHandlerInterface $handler, float $startTime): void
    {
        $endTime = microtime(true);

        if (
            $endTime - $startTime > 0.1
        ) {

            $classParts = explode('\\', get_class($handler));

            StuLogger::log(sprintf(
                "\t\t\t%d: %s, seconds: %F",
                $spacecraft->getId(),
                end($classParts),
                $endTime - $startTime
            ), LogTypeEnum::TICK);
        }
    }

    private function sendMessage(Spacecraft $ship, InformationWrapper $informationWrapper): void
    {
        if ($informationWrapper->isEmpty()) {
            return;
        }

        $text = sprintf("Tickreport der %s\n%s", $ship->getName(), $informationWrapper->getInformationsAsString());

        $this->privateMessageSender->send(
            UserConstants::USER_NOONE,
            $ship->getUser()->getId(),
            $text,
            $ship->getType()->getMessageFolderType(),
            $ship
        );
    }
}