Gielfeldt\Iterators\CombineIterator

Complexity

Total Complexity

3

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	key()	0	5	1
A	current()	0	5	1

<?php

namespace Gielfeldt\Iterators;

class CombineIterator extends TraversableIterator
{
    const NEED_ANY = \MultipleIterator::MIT_NEED_ANY;
    const NEED_ALL = \MultipleIterator::MIT_NEED_ALL;

    public function __construct(\Traversable $keysIterator, \Traversable $valuesIterator, int $flags = self::NEED_ANY)
    {
        $iterator = new \MultipleIterator($flags | \MultipleIterator::MIT_KEYS_ASSOC);
        $iterator->attachIterator($keysIterator, 'key');
        $iterator->attachIterator($valuesIterator, 'value');
        parent::__construct($iterator);
    }

    public function key()
    {
        $current = parent::current();

It seems like you call parent on a different method (current() instead of key()). Are you sure this is correct? If so, you might want to change this to $this->current().

        return $current['key'] ?? null;
    }

    public function current()
    {
        $current = parent::current();
        return $current['value'] ?? null;
    }
}