Pipes\Pipe

Complexity

Total Complexity

5

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	18
Bugs	2	Features	9

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	12	4
A	getIterator()	0	4	1

<?php

namespace Pipes;

use IteratorAggregate;
use Pipes\Iterator\PlainArrayIterator;

class Pipe implements IteratorAggregate
{
    use PipenessTrait;

    protected $var;

    public function __construct(&$var = null)
    {
        if (is_array($var)) {
            $this->var = new PlainArrayIterator($var);
        }
        if ($var instanceof \Traversable) {
            $this->var = $var;
        }
        if (!func_num_args()) {
            $this->var = [];
        }
    }

    /**
     * This method is implemented just because it's required by the
     * \IteratorAggregate interface.
     *
     * Returns an instance of the last array/Traversable of the chain
     * Don't use this method: it won't return a Pipe instance:
     * no more chaining magic.
     * It may very well be a plain array.
     *
     * If you need an iterator, use toIterator() instead.
     *
     * @return array|\Traversable
     */
    public function getIterator()
    {
        return $this->var;

The expression $this->var; of type Traversable|array adds the type array to the return on line 42 which is incompatible with the return type declared by the interface IteratorAggregate::getIterator of type Traversable.

    }
}