TRex\Collection\CollectionFilterTrait

Complexity

Total Complexity

11

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

85.19%

Importance

Changes	5
Bugs	1	Features	4

4 Methods

Rating	Name	Duplication	Size	Complexity
A	filter()	0	13	4
A	each()	0	9	2
A	extract()	0	6	2
A	rand()	0	11	3

<?php
namespace TRex\Collection;

trait CollectionFilterTrait
{
    /**
     * @param callable|null $filter
     * @param int $flag (only for php version >= 5.6)
     * @return $this

Should the return type not be object?

     */
    public function filter(callable $filter = null, $flag = 0)
    {
        if (defined('HHVM_VERSION')) {
            if (version_compare(HHVM_VERSION, '3.8.1', '>=')) {
                return new $this(array_filter((array)$this, $filter, $flag));
            }
        } else {
            if (version_compare(phpversion(), '5.6.0', '>=')) {
                return new $this(array_filter((array)$this, $filter, $flag));
            }
        }
        return new $this(array_filter((array)$this, $filter));
    }

    /**
     * Executes the callback for every value.
     * Returns a collection with the result of each callback call.
     *
     * @param callable $callback
     * @param array $args
     * @return $this

Should the return type not be object?

     */
    public function each(callable $callback, array $args = [])
    {
        return new $this(array_map(
            function ($value) use ($callback, $args) {
                return call_user_func($callback, $value, $args) ?: $value;
            },
            (array)$this
        ));
    }

    /**
     * Extracts the sequence of elements.
     * Starts at index $startIndex and stop after $length keys.
     *
     * @param int $startIndex
     * @param int $length
     * @param bool $areKeysPreserved
     * @return mixed

Consider making the return type a bit more specific; maybe use object.

     */
    public function extract($startIndex, $length = 0, $areKeysPreserved = true)
    {
        $collection = (array)$this;
        $length = $length ?: count($collection);
        return new $this(array_slice($collection, $startIndex, $length, $areKeysPreserved));
    }

    /**
     * @param int $length
     * @return $this

Should the return type not be object?

     */
    public function rand($length = 1)
    {
        $keys = (array)array_rand((array)$this, $length);
        $result = new $this;
        foreach ($this as $key => $value) {

The expression $this of type this<TRex\Collection\CollectionFilterTrait> is not traversable.

            if (in_array($key, $keys)) {
                $result[$key] = $value;
            }
        }
        return $result;
    }
}