SpareParts\Enum\Set\SetTrait

Complexity

Total Complexity

11

Size/Duplication

Total Lines	71
Duplicated Lines	9.86 %

Coupling/Cohesion

Components	2
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getIterator()	0	4	1
A	count()	0	4	1
A	contains()	0	7	2
A	setEnumClass()	7	7	3
A	__construct()	0	15	4

<?php
namespace SpareParts\Enum\Set;

use SpareParts\Enum\Enum;
use SpareParts\Enum\Exception\EnumSetMustContainEnumsException;
use SpareParts\Enum\Exception\InvalidEnumClassException;

trait SetTrait
{
    /**
     * @var Enum[]
     */
    protected $set = [];

    /**
     * @var string
     */
    protected $enumClass;

    /**
     * @param string $enumClass
     * @param Enum[] $set
     */
    public function __construct($enumClass, $set = [])
    {
        // if enum class was sent, we must make sure it is valid
        $this->setEnumClass($enumClass);

        if (!is_array($set)) {
            throw new EnumSetMustContainEnumsException(sprintf("Enum set must be initialized with array of enums."));
        }
        foreach ($set as $enum) {
            if (!($enum instanceof $this->enumClass)) {
                throw new EnumSetMustContainEnumsException(sprintf("Expected %s, got %s", $this->enumClass, get_class($enum)));
            }
            $this->set[(string) $enum] = $enum;
        }
    }

    /**
     * @return \ArrayIterator
     */
    public function getIterator()
    {
        return new \ArrayIterator(array_values($this->set));
    }

    /**
     * @return int
     */
    public function count()
    {
        return count($this->set);
    }

    /**
     * @param Enum $value
     * @return bool
     */
    public function contains(Enum $value)
    {
        if (in_array($value, $this->set)) {
            return true;
        }
        return false;
    }

    /**
     * @param string $enumClass
     */
    protected function setEnumClass($enumClass)

This method seems to be duplicated in your project.

    {
        if (!is_null($enumClass) && !is_subclass_of($enumClass, Enum::class)) {

Due to PHP Bug #53727, is_subclass_of might return inconsistent results on some PHP versions if \SpareParts\Enum\Enum::class can be an interface. If so, you could instead use ReflectionClass::implementsInterface.

            throw new InvalidEnumClassException("Class ${enumClass} does not implement Enum (as it should). Maybe you forgot to specify correct enum class in constructor?");
        }
        $this->enumClass = $enumClass;
    }
}