AlecRabbit\Spinner\Core\Revolver\FrameRevolverBuilder

Complexity

Total Complexity

5

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	withTolerance()	0	5	1
A	withInterval()	0	5	1
A	withFrameCollection()	0	5	1
A	validate()	0	7	1
A	build()	0	9	1

<?php

declare(strict_types=1);


namespace AlecRabbit\Spinner\Core\Revolver;

use AlecRabbit\Spinner\Contract\IInterval;
use AlecRabbit\Spinner\Core\Contract\IFrameCollection;
use AlecRabbit\Spinner\Core\Revolver\Contract\IFrameRevolver;
use AlecRabbit\Spinner\Core\Revolver\Contract\IFrameRevolverBuilder;
use AlecRabbit\Spinner\Exception\LogicException;

final class FrameRevolverBuilder implements IFrameRevolverBuilder
{
    private ?IFrameCollection $frames = null;
    private ?IInterval $interval = null;
    private ?int $tolerance = null;

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

        return
            new FrameCollectionRevolver(
                $this->frames,

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

                $this->interval,

It seems like $this->interval can also be of type null; however, parameter $interval of AlecRabbit\Spinner\Core\Revolver\FrameCollectionRevolver::__construct() does only seem to accept AlecRabbit\Spinner\Contract\IInterval, 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\Revolver\FrameCollectionRevolver::__construct() does only seem to accept integer, maybe add an additional type check?

            );
    }

    private function validate(): void
    {
        match (true) {
            $this->frames === null => throw new LogicException('Frame collection is not set.'),
            $this->interval === null => throw new LogicException('Interval is not set.'),
            $this->tolerance === null => throw new LogicException('Tolerance is not set.'),
            default => null,
        };
    }

    public function withFrameCollection(IFrameCollection $frames): IFrameRevolverBuilder
    {
        $clone = clone $this;
        $clone->frames = $frames;
        return $clone;
    }

    public function withInterval(IInterval $interval): IFrameRevolverBuilder
    {
        $clone = clone $this;
        $clone->interval = $interval;
        return $clone;
    }

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