TomPHP\ContainerConfigurator\ApplicationConfigIterator

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	key()	0	4	1
A	next()	0	8	2
A	rewind()	0	6	1
A	endChildren()	0	4	1

<?php

namespace TomPHP\ContainerConfigurator;

use RecursiveArrayIterator;
use RecursiveIteratorIterator;

final class ApplicationConfigIterator extends RecursiveIteratorIterator
{
    /**
     * @var string[]
     */
    private $path = [];

    /**
     * @var string
     */
    private $separator;

    /**
     * @param ApplicationConfig $config
     */
    public function __construct(ApplicationConfig $config)
    {
        parent::__construct(
            new RecursiveArrayIterator($config->asArray()),
            RecursiveIteratorIterator::SELF_FIRST
        );
        $this->separator = $config->getSeparator();
    }

    public function key()
    {
        return implode($this->separator, array_merge($this->path, [parent::key()]));
    }

    public function next()
    {
        if ($this->hasChildren()) {

The method hasChildren() does not exist on TomPHP\ContainerConfigurator\ApplicationConfigIterator. Did you maybe mean callHasChildren()?

            array_push($this->path, parent::key());

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

        }

        parent::next();
    }

    public function rewind()
    {
        $this->path = [];

        parent::rewind();
    }

    public function endChildren()
    {
        array_pop($this->path);
    }
}