Graze\DataNode\NodeCollection

Complexity

Total Complexity

20

Size/Duplication

Total Lines	96
Duplicated Lines	29.17 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

72.97%

Importance

Changes	2
Bugs	0	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	add()	0	7	2
A	apply()	0	11	4
A	__clone()	0	6	2
A	__toString()	0	4	1
A	getClone()	0	4	1
B	first()	14	14	5
B	last()	14	14	5

<?php

namespace Graze\DataNode;

use Graze\DataStructure\Collection\Collection;
use InvalidArgumentException;

/**
 * Class NodeCollection
 *
 * A Collection of DataNodes that can be acted upon by a flow
 *
 * @package Graze\DataFlow\Node
 */
class NodeCollection extends Collection implements NodeCollectionInterface
{
    /**
     * {@inheritdoc}
     */
    public function add($value)
    {
        if (!($value instanceof NodeInterface)) {
            throw new InvalidArgumentException("The specified value does not implement NodeInterface");
        }
        return parent::add($value);
    }

    /**
     * {@inheritdoc}
     */
    public function apply($fn)
    {
        foreach ($this->items as &$item) {
            $out = call_user_func($fn, $item);
            if (isset($out) && ($out instanceof NodeInterface)) {
                $item = $out;
            }
        }

        return $this;
    }

    /**
     * On clone, clone all flows too
     */
    public function __clone()
    {
        foreach ($this->items as &$item) {
            $item = clone $item;
        }
    }

    /**
     * @return string
     */
    public function __toString()
    {
        return "NodeCollection";
    }

    /**
     * @inheritdoc
     */
    public function getClone()
    {
        return clone $this;
    }

    /**
     * @param callable   $fn

Should the type for parameter $fn not be null|callable?

     * @param mixed|null $default
     *
     * @return NodeInterface|null
     */
    public function first(callable $fn = null, $default = null)

This method seems to be duplicated in your project.

    {
        if (is_null($fn)) {
            return count($this->items) > 0 ? reset($this->items) : $default;
        }

        foreach ($this->getIterator() as $value) {
            if (call_user_func($fn, $value)) {
                return $value;
            }
        }

        return $default;
    }

    /**
     * @param callable   $fn

Should the type for parameter $fn not be null|callable?

     * @param mixed|null $default
     *
     * @return NodeInterface|null
     */
    public function last(callable $fn = null, $default = null)

This method seems to be duplicated in your project.

    {
        if (is_null($fn)) {
            return count($this->items) > 0 ? end($this->items) : $default;
        }

        foreach (array_reverse($this->items) as $value) {
            if (call_user_func($fn, $value)) {
                return $value;
            }
        }

        return $default;
    }
}