Gielfeldt\Iterators\DiffIterator

Complexity

Total Complexity

9

Size/Duplication

Total Lines	42
Duplicated Lines	16.67 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	2
A	setDiff()	0	4	1
A	accept()	7	11	4
A	diffCurrent()	0	4	1
A	diffKey()	0	4	1

<?php

namespace Gielfeldt\Iterators;

class DiffIterator extends \FilterIterator
{
    protected $iterators;
    protected $callback;

    public function __construct(\Traversable ...$iterators)
    {
        $innerIterator = array_shift($iterators);
        parent::__construct($innerIterator);
        foreach ($iterators as $iterator) {
            $this->iterators[] = new CachingIterator($iterator);
        }
        $this->callback = \Closure::fromCallable([static::class, 'diffCurrent']);
    }

    public function setDiff(callable $callback)
    {
        $this->callback = \Closure::fromCallable($callback);
    }

    public function accept()
    {
        foreach ($this->iterators as $iterator) {

This code seems to be duplicated across your project.

            foreach ($iterator as $key => $value) {
                if (($this->callback)($this->getInnerIterator(), $key, $value)) {
                    return false;
                }
            }
        }
        return true;
    }

    public static function diffCurrent($iterator, $key, $value)

The parameter $key is not used and could be removed.

    {
        return $iterator->current() == $value;
    }

    public static function diffKey($iterator, $key, $value)

The parameter $value is not used and could be removed.

    {
        return $iterator->key() == $key;
    }
}