AlecRabbit\Spinner\Core\Builder\TimerBuilder

Complexity

Total Complexity

4

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	validate()	0	6	1
A	withStartTime()	0	5	1
A	build()	0	7	1
A	withTimeFunction()	0	5	1

<?php

declare(strict_types=1);


namespace AlecRabbit\Spinner\Core\Builder;

use AlecRabbit\Spinner\Contract\ITimer;
use AlecRabbit\Spinner\Core\Builder\Contract\ITimerBuilder;
use AlecRabbit\Spinner\Core\Timer;
use AlecRabbit\Spinner\Exception\LogicException;
use Closure;

final class TimerBuilder implements ITimerBuilder
{
    private ?float $startingTime = null;
    private ?Closure $timeFunction = null;

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

        return new Timer(
            timeFunction: $this->timeFunction,

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

            time: $this->startingTime,

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

        );
    }

    private function validate(): void
    {
        match (true) {
            null === $this->startingTime => throw new LogicException('Starting time is not set.'),
            $this->timeFunction === null => throw new LogicException('Time function is not set.'),
            default => null,
        };
    }

    public function withStartTime(float $time): ITimerBuilder
    {
        $clone = clone $this;
        $clone->startingTime = $time;
        return $clone;
    }

    public function withTimeFunction(Closure $timeFunction): ITimerBuilder
    {
        $clone = clone $this;
        $clone->timeFunction = $timeFunction;
        return $clone;
    }
}