AlecRabbit\Spinner\Core\Factory\DriverLinkerFactory

Complexity

Total Complexity

4

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getDriverLinker()	0	6	2
A	__construct()	0	4	1
A	createLinker()	0	6	1

<?php

declare(strict_types=1);


namespace AlecRabbit\Spinner\Core\Factory;

use AlecRabbit\Spinner\Core\Contract\IDriverLinker;
use AlecRabbit\Spinner\Core\Contract\ISettingsProvider;
use AlecRabbit\Spinner\Core\Contract\Loop\Contract\ILoop;
use AlecRabbit\Spinner\Core\DriverLinker;
use AlecRabbit\Spinner\Core\Factory\Contract\IDriverLinkerFactory;

final class DriverLinkerFactory implements IDriverLinkerFactory
{
    private static ?IDriverLinker $driverLinker = null;

    public function __construct(
        protected ILoop $loop,
        protected ISettingsProvider $settingsProvider,
    ) {
    }

    public function getDriverLinker(): IDriverLinker
    {
        if (self::$driverLinker === null) {
            self::$driverLinker = $this->createLinker();
        }
        return self::$driverLinker;

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

    }

    private function createLinker(): IDriverLinker
    {
        return
            new DriverLinker(
                $this->loop,
                $this->settingsProvider->getDriverSettings()->getOptionLinker(),
            );
    }
}