Ianriizky\CodingInterview\Factorial\Factorial

Complexity

Total Complexity

8

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	resultWithRecursive()	0	9	2
A	resultWithLoop()	0	13	3
A	__construct()	0	2	1
A	resultWithTailRecursive()	0	10	2

<?php

namespace Ianriizky\CodingInterview\Factorial;

class Factorial
{
    /**
     * Temporary value for the recursive value.
     *
     * @var int
     */
    protected int $result = 1;

    /**
     * Create a new instance class.
     *
     * @param  int  $initialN
     * @return void
     */
    public function __construct(protected int $initialN)
    {
        //
    }

    /**
     * Return result of the factorial using loop way.
     *
     * @return int
     */
    public function resultWithLoop(): int
    {
        if ($this->initialN <= 1) {
            return 1;
        }

        $result = 1;

        for ($n = $this->initialN; $n >= 1; $n--) {
            $result *= $n;
        }

        return $result;
    }

    /**
     * Return result of the factorial using recursive way.
     *
     * @return int
     */
    public function resultWithRecursive(): int
    {
        if ($this->initialN <= 1) {
            return 1;
        }

        $n = $this->initialN;

        return $n * $this->resultWithRecursive($this->initialN = $n - 1);

The call to Ianriizky\CodingInterview\Factorial\Factorial::resultWithRecursive() has too many arguments starting with $this->initialN = $n - 1.

    }

    /**
     * Return result of the factorial using tail recursive way.
     *
     * @return int
     */
    public function resultWithTailRecursive(): int
    {
        if ($this->initialN <= 0) {
            return $this->result;
        }

        $this->result *= $this->initialN;
        $this->initialN--;

        return $this->resultWithTailRecursive();
    }
}