SpareParts\Enum\Set\SetTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	51
Duplicated Lines	13.73 %

Coupling/Cohesion

Components	2
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

4 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

<?php
namespace SpareParts\Enum\Set;

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

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

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

    /**
     * @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;
    }
}