Shrikeh\Collection\ObjectStorageTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	2
	getStorage()	0	1	?
A	append()	0	4	1

<?php

namespace Shrikeh\Collection;

use Traversable;
use SplObjectStorage;

trait ObjectStorageTrait
{
    /**
     * Force that this trait is used inside an OuterIterator
     */
    use \Shrikeh\Collection\RequiresOuterIteratorTrait;

    public function __construct(Traversable $objects)
    {
        static::testOuterIterator($this);
        parent::__construct(new SplObjectStorage());

        foreach ($objects as $key => $value) {
            $this->append($value, $key);
        }
        $this->getStorage()->rewind();
    }

    /**
     * @return mixed
     */
    abstract protected function getStorage();

    /**
     * We don't want to make this abstract, because then we would defeat the
     * whole point of type hinting on this.
     * Usage within domain collections should be append(SomeDomainThing $thing, $key)
     * so instead we throw an Exception so a dev knows where the problem lies.
     */
    private function append($data, $key)

The parameter $data is not used and could be removed.

The parameter $key is not used and could be removed.

    {
        throw new \LogicException('you must override the append() method');
    }
}