Pipes\Filter\EachTrait

Complexity

Total Complexity

2

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	7
Bugs	2	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	each()	0	11	2

<?php

namespace Pipes\Filter;

trait EachTrait
{
    /**
     * Applies a callback to each element.
     * (internally uses \CallbackFilterIterator).
     *
     * The passed callback will be invoked with the following argument:
     *
     *      - $value (iterator's current())
     *
     * If the second parameter is true the following arguments will be added
     * to the call:
     *
     *      - $key (iterator's key())
     *      - $iterator (the iterator itself)
     *
     * @param callable $______callback a PHP callable (closure, string or array)
     * @param bool     $______allArgs  if true key and iterator will be passed to the callback
     *
     * @return \Pipes\Pipe
     */
    public function each(callable $______callback, $______allArgs = false)
    {
        return $this->chainWith(

It seems like chainWith() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            new \CallbackFilterIterator($this->getIterator(),

It seems like getIterator() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                function () use ($______callback, $______allArgs
                ) {
                    call_user_func_array($______callback, $______allArgs ? func_get_args() : [func_get_arg(0)]);

                    return true;
                }));
    }
}