Thinktomorrow\Vine\Branch\BranchCollection

Complexity

Total Complexity

19

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

86.84%

Importance

Changes	3
Bugs	0	Features	0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	hasBranches()	0	4	2
A	push()	0	6	1
A	__get()	0	6	2
A	validateBranches()	0	10	3
A	offsetSet()	0	7	2
A	offsetExists()	0	3	1
A	offsetUnset()	0	3	1
A	offsetGet()	0	3	2
A	count()	0	4	1
A	getIterator()	0	4	1
A	toArray()	0	4	1
A	flatten()	0	4	1

<?php

namespace Thinktomorrow\Vine\Branch;

use ArrayAccess;
use ArrayIterator;
use Countable;
use IteratorAggregate;
use Thinktomorrow\Vine\VineException;

class BranchCollection implements ArrayAccess, Countable, IteratorAggregate
{
    private $branches;

    public function __construct(array $branches = [])
    {
        self::validateBranches($branches);

        $this->branches = $branches;
    }

    public function hasBranches()
    {
        return ($this->branches && count($this->branches) > 0);

The expression $this->branches of type array 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.

    }

    public function push(Branch $branch)
    {
        $this->branches[] = $branch;

        return $this;
    }

    public function __get($key)
    {
        if(isset($this->$key)) return $this->$key;

        throw VineException::unknownBranchCollectionAttribute($key);
    }

    public static function validateBranches($branches)
    {
        foreach($branches as $branch)
        {
            if(!$branch instanceof Branch)
            {
                throw VineException::requiredAsBranch($branch);
            }
        }
    }

    public function offsetSet($offset, $value) {
        if (is_null($offset)) {
            $this->branches[] = $value;
        } else {
            $this->branches[$offset] = $value;
        }
    }

    public function offsetExists($offset) {
        return isset($this->branches[$offset]);
    }

    public function offsetUnset($offset) {
        unset($this->branches[$offset]);
    }

    public function offsetGet($offset) {
        return isset($this->branches[$offset]) ? $this->branches[$offset] : null;
    }

    public function count()
    {
        return count($this->branches);
    }

    /**
     * Iterator for the branches
     *
     * @return \ArrayIterator
     */
    public function getIterator()
    {
        return new ArrayIterator($this->branches);
    }

    public function toArray()
    {
        return BuildArray::fromCollection($this);
    }

    public function flatten()
    {
        // return a flattened array of all branch items
    }
}