Apie\ValueObjects\StringEnumTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	validValue()	0	4	1
A	getValidValues()	0	4	1
A	sanitizeValue()	0	5	1
A	getLookupTable()	0	13	4

<?php


namespace Apie\ValueObjects;

use ReflectionClass;

trait StringEnumTrait
{
    use StringTrait;

    /**
     * @var string[]
     */
    private static $lookupTable;

    final protected function validValue(string $value): bool
    {
        $values = self::getLookupTable();
        return isset($values[$value]);
    }

    final protected function sanitizeValue(string $value): string
    {
        $values = self::getLookupTable();
        assert(isset($values[$value]));
        return $values[$value];
    }

    private static function getLookupTable(): array
    {
        if (!self::$lookupTable) {

The expression self::lookupTable of type string[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            $values = self::getValidValues();
            self::$lookupTable = [];
            foreach ($values as $value) {
                self::$lookupTable[$value] = $value;
            }
            foreach ($values as $key => $value) {
                self::$lookupTable[$key] = $value;
            }
        }
        return self::$lookupTable;
    }

    final public static function getValidValues(): array
    {
        $reflectionClass = new ReflectionClass(__CLASS__);
        return $reflectionClass->getConstants();
    }
}