Respect\Validation\ResultIterator

Complexity

Total Complexity

10

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	current()	0	4	1
A	next()	0	4	1
A	key()	0	6	1
A	valid()	0	6	2
A	rewind()	0	4	1
A	hasChildren()	0	6	1
A	getChildren()	0	4	1
A	count()	0	4	1

<?php


namespace Respect\Validation;

use Countable;
use RecursiveIterator;

final class ResultIterator implements RecursiveIterator, Countable
{
    /**
     * @var Result[]
     */
    private $children;

    public function __construct(Result $result)
    {
        $this->children = $result->getChildren();
    }

    public function current()
    {
        return current($this->children);
    }

    public function next()
    {
        return next($this->children);
    }

    public function key()
    {
        $current = current($this->children);

        return $current->getProperties()['reference'] ?? key($this->children);
    }

    public function valid(): bool
    {
        $key = key($this->children);

        return ($key !== null && $key !== false);
    }

    public function rewind()
    {
        reset($this->children);
    }

    public function hasChildren(): bool
    {
        $children = $this->current()->getChildren();

        return count($children) > 0;
    }

    public function getChildren(): ResultIterator
    {
        return new self($this->current());

It seems like $this->current() targeting Respect\Validation\ResultIterator::current() can also be of type false; however, Respect\Validation\ResultIterator::__construct() does only seem to accept object<Respect\Validation\Result>, did you maybe forget to handle an error condition?

    }

    public function count(): int
    {
        return count($this->children);
    }
}