VideoGamesRecords\CoreBundle\ValueObject\GameStatus

Complexity

Total Complexity

8

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	inArray()	0	3	1
A	getIndex()	0	3	1
A	getStatusChoices()	0	5	1
A	__toString()	0	3	1
A	isInactive()	0	3	1
A	__construct()	0	5	1
A	isActive()	0	3	1
A	getValue()	0	3	1

<?php

namespace VideoGamesRecords\CoreBundle\ValueObject;

use Webmozart\Assert\Assert;

class GameStatus
{
    const STATUS_ACTIVE = 'ACTIF';
    const STATUS_INACTIVE = 'INACTIF';

    public const VALUES = [
        self::STATUS_ACTIVE,
        self::STATUS_INACTIVE,
    ];

    private string $value;

    public function __construct(string $value)
    {
        self::inArray($value);

        $this->value = $value;
    }

    public static function inArray(string $value): void
    {
        Assert::inArray($value, self::VALUES);
    }

    public function getValue(): string
    {
        return $this->value;
    }

    public function getIndex(): int
    {
        return array_search($this->value, self::VALUES);

The expression return array_search($this->value, self::VALUES) could return the type string which is incompatible with the type-hinted return integer. Consider adding an additional type-check to rule them out.

    }

    public function __toString(): string
    {
        return $this->value;
    }

    public function isActive(): bool
    {
        return self::STATUS_ACTIVE === $this->value;
    }

    public function isInactive(): bool
    {
        return self::STATUS_INACTIVE === $this->value;
    }

    public static function getStatusChoices(): array
    {
        return [
            self::STATUS_ACTIVE => self::STATUS_ACTIVE,
            self::STATUS_INACTIVE => self::STATUS_INACTIVE,
        ];
    }
}