Kata\Rover\Domain\Model\Orientation

Complexity

Total Complexity

26

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

72.41%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	NORTH()	0	4	1
A	SOUTH()	0	4	1
A	EAST()	0	4	1
A	WEST()	0	4	1
A	isNorth()	0	4	1
A	isSouth()	0	4	1
A	isEast()	0	4	1
A	isWest()	0	4	1
C	turn()	0	22	17

<?php

namespace Kata\Rover\Domain\Model;

class Orientation

Since you have declared the constructor as private, maybe you should also declare the class as final.

{
    private $orientation;

    private function __construct(string $orientation)
    {
        $this->orientation = $orientation;
    }

    public static function NORTH(): Orientation
    {
        return new self('N');
    }

    public static function SOUTH(): Orientation
    {
        return new self('S');
    }

    public static function EAST(): Orientation
    {
        return new self('E');
    }

    public static function WEST(): Orientation
    {
        return new self('W');
    }

    public function isNorth(): bool
    {
        return $this->orientation === 'N';
    }

    public function isSouth(): bool
    {
        return $this->orientation === 'S';
    }

    public function isEast(): bool
    {
        return $this->orientation === 'E';
    }

    public function isWest(): bool
    {
        return $this->orientation === 'W';
    }

    public function turn(Movement $movement): Orientation
    {
        if (($movement->isLeft() && $this->isEast()) || ($movement->isRight() && $this->isWest()))
        {
            return self::NORTH();
        }

        if (($movement->isLeft() && $this->isWest()) || ($movement->isRight() && $this->isEast()))
        {
            return self::SOUTH();
        }

        if (($movement->isLeft() && $this->isSouth()) || ($movement->isRight() && $this->isNorth()))
        {
            return self::EAST();
        }

        if (($movement->isLeft() && $this->isNorth()) || ($movement->isRight() && $this->isSouth()))
        {
            return self::WEST();
        }
    }
}