GenericCollections\Abstracts\AbstractCollection

Complexity

Total Complexity

20

Size/Duplication

Total Lines	80
Duplicated Lines	8.75 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes	5
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	add()	7	15	4
A	contains()	0	4	1
A	remove()	0	12	3
A	retainAll()	0	14	4
A	removeAll()	0	13	4
A	removeIf()	0	12	4

<?php namespace GenericCollections\Abstracts;

use GenericCollections\Interfaces\CollectionInterface;
use GenericCollections\Internal\DataArray;
use GenericCollections\Traits\CollectionMethods;

/**
 * This is a partial implementation of CollectionInterface
 * The main Collection object extends this class but types methods
 *
 * Use this class to implement your own collection class.
 * You will need to implement the getElementType method on your concrete case.
 *
 * @package GenericCollections\Abstracts
 */
abstract class AbstractCollection extends DataArray implements CollectionInterface
{
    use CollectionMethods;
    
    public function add($element)
    {
        if ($this->optionUniqueValues() && $this->contains($element)) {
            return false;
        }
        if (! $this->checkElementType($element)) {

This code seems to be duplicated across your project.

            throw new \InvalidArgumentException(
                'Invalid element type;'
                . ' the collection ' . get_class($this)
                . ' was expecting a ' . $this->getElementType() . ' type'
            );
        }
        $this->data[] = $element;
        return true;
    }

    public function contains($element)
    {
        return in_array($element, $this->data, $this->optionComparisonIsIdentical());
    }

    public function remove($element)
    {
        $index = array_search($element, $this->data, $this->optionComparisonIsIdentical());
        if (false === $index) {
            return false;
        }
        unset($this->data[$index]);
        if ($index < count($this->data)) {
            $this->data = array_values($this->data);
        }
        return true;
    }

    public function retainAll(array $elements)
    {
        $changed = false;
        foreach ($this->data as $index => $element) {
            if (! in_array($element, $elements, $this->optionComparisonIsIdentical())) {
                unset($this->data[$index]);
                $changed = true;
            }
        }
        if ($changed) {
            $this->data = array_values($this->data);
        }
        return $changed;
    }

    public function removeAll(array $elements)
    {
        $changed = false;
        foreach ($elements as $element) {
            do {
                $removed = $this->remove($element);
                if ($removed) {
                    $changed = true;
                }
            } while ($removed);
        }
        return $changed;
    }

    public function removeIf(callable $callable)
    {
        $changed = false;
        foreach ($this->data as $element) {
            if (true === call_user_func($callable, $element)) {
                if ($this->remove($element)) {
                    $changed = true;
                }
            }
        }
        return $changed;
    }
}