AlecRabbit\Spinner\Core\Factory\LoopFactory

Complexity

Total Complexity

5

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getLoopProbe()	0	3	1
A	getLoop()	0	6	2
A	createLoop()	0	3	1

<?php

declare(strict_types=1);

namespace AlecRabbit\Spinner\Core\Factory;

use AlecRabbit\Spinner\Core\Contract\Loop\Contract\ILoop;
use AlecRabbit\Spinner\Core\Contract\Loop\Contract\ILoopProbe;
use AlecRabbit\Spinner\Core\Factory\Contract\ILoopProbeFactory;
use AlecRabbit\Spinner\Core\Factory\Contract\ILoopFactory;

final class LoopFactory implements ILoopFactory
{
    private static ?ILoop $loop = null;

    public function __construct(
        protected ILoopProbeFactory $loopProbeFactory,
    ) {
    }

    public function getLoop(): ILoop
    {
        if (self::$loop === null) {
            self::$loop = $this->createLoop();
        }
        return self::$loop;

The expression return self::loop could return the type null which is incompatible with the type-hinted return AlecRabbit\Spinner\Core\Contract\Loop\Contract\ILoop. Consider adding an additional type-check to rule them out.

    }

    private function createLoop(): ILoop
    {
        return $this->getLoopProbe()->createLoop();
    }

    private function getLoopProbe(): ILoopProbe
    {
        return $this->loopProbeFactory->getProbe();
    }
}