AlecRabbit\Spinner\Core\Widget\Builder\WidgetRevolverBuilder

Complexity

Total Complexity

5

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	build()	0	8	1
A	validate()	0	7	1
A	withStyleRevolver()	0	5	1
A	withCharRevolver()	0	5	1
A	withTolerance()	0	5	1

<?php

declare(strict_types=1);

namespace AlecRabbit\Spinner\Core\Widget\Builder;

use AlecRabbit\Spinner\Core\Contract\ITolerance;
use AlecRabbit\Spinner\Core\Revolver\A\ARevolverBuilder;
use AlecRabbit\Spinner\Core\Revolver\Contract\IFrameRevolver;
use AlecRabbit\Spinner\Core\Widget\Contract\IWidgetRevolver;
use AlecRabbit\Spinner\Core\Widget\Contract\IWidgetRevolverBuilder;
use AlecRabbit\Spinner\Core\Widget\WidgetRevolver;
use AlecRabbit\Spinner\Exception\LogicException;

/**
 * @psalm-suppress PossiblyNullArgument
 */
final class WidgetRevolverBuilder extends ARevolverBuilder implements IWidgetRevolverBuilder
{
    private ?IFrameRevolver $styleRevolver = null;
    private ?IFrameRevolver $charRevolver = null;
    private ?ITolerance $tolerance = null;

    public function build(): IWidgetRevolver
    {
        $this->validate();

        return new WidgetRevolver(
            $this->styleRevolver,

It seems like $this->styleRevolver can also be of type null; however, parameter $style of AlecRabbit\Spinner\Core\Widget\WidgetRevolver::__construct() does only seem to accept AlecRabbit\Spinner\Core\Revolver\Contract\IRevolver, maybe add an additional type check?

            $this->charRevolver,

It seems like $this->charRevolver can also be of type null; however, parameter $character of AlecRabbit\Spinner\Core\Widget\WidgetRevolver::__construct() does only seem to accept AlecRabbit\Spinner\Core\Revolver\Contract\IRevolver, maybe add an additional type check?

            $this->tolerance,

It seems like $this->tolerance can also be of type null; however, parameter $tolerance of AlecRabbit\Spinner\Core\Widget\WidgetRevolver::__construct() does only seem to accept AlecRabbit\Spinner\Core\Contract\ITolerance, maybe add an additional type check?

        );
    }

    protected function validate(): void
    {
        match (true) {
            $this->styleRevolver === null => throw new LogicException('Style revolver is not set.'),
            $this->charRevolver === null => throw new LogicException('Character revolver is not set.'),
            $this->tolerance === null => throw new LogicException('Tolerance is not set.'),
            default => null,
        };
    }

    public function withStyleRevolver(IFrameRevolver $styleRevolver): IWidgetRevolverBuilder
    {
        $clone = clone $this;
        $clone->styleRevolver = $styleRevolver;
        return $clone;
    }

    public function withCharRevolver(IFrameRevolver $charRevolver): IWidgetRevolverBuilder
    {
        $clone = clone $this;
        $clone->charRevolver = $charRevolver;
        return $clone;
    }

    public function withTolerance(ITolerance $tolerance): IWidgetRevolverBuilder
    {
        $clone = clone $this;
        $clone->tolerance = $tolerance;
        return $clone;
    }
}