AbstractLoader::next() - Code Metrics - Inspection of "Code quality fixes" - jasny/container - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — master ( fb1394...c6b41f )

by Arnold

created 2018-08-27 13:31 UTC

AbstractLoader::next() A

↳ Parent: AbstractLoader

Complexity

Conditions	4
Paths	3

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	4

Importance

Changes

Metric	Value
cc	4
eloc	5
nc	3
nop	0
dl	0
loc	11
ccs	5
cts	5
cp	1
crap	4
rs	10
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace Jasny\Container\Loader;

/**
 * Base class for loader using two iterators.
 */
abstract class AbstractLoader implements \OuterIterator
{
    /**
     * @var \Iterator
     */
    protected $items;

    /**
     * @var \ArrayIterator|null
     */
    protected $entries;


    /**
     * Class constructor.
     *
     * @param \Iterator $items
     */
    public function __construct(\Iterator $items)
    {
        $this->items = $items;
        $this->prepareNext();
    }

    /**
     * Prepare new entries
     *
     * @return void
     */
    abstract protected function prepareNext(): void;

    /**
     * Return the current element
     *
     * @return callable|null
     */
    public function current(): ?callable
    {
        return isset($this->entries) ? $this->entries->current() : null;
        return isset($this->entries) ? $this->entries->/** @scrutinizer ignore-call */ current() : null;
    }

    /**
     * Move forward to next element
     *
     * @return void
     */
    public function next(): void
    {
        if (!isset($this->entries)) {
            // Shouldn't happen if `prepareNext()` works correctly
            return; // @codeCoverageIgnore
        }

        $this->entries->next();

        while (!$this->entries->valid() && $this->items->valid()) {
            $this->prepareNext();
        }
    }

    /**
     * Return the key of the current element
     *
     * @return string|null
     */
    public function key(): ?string
    {
        return isset($this->entries) ? $this->entries->key() : null;
    }

    /**
     * Checks if current position is valid
     *
     * @return bool
     */
    public function valid(): bool
    {
        return isset($this->entries) && $this->entries->valid();
    }

    /**
     * Rewind the iterator to the first element
     *
     * @return void
     */
    public function rewind(): void
    {
        $this->items->rewind();
        $this->entries = null;

        $this->prepareNext();
    }

    /**
     * Returns the inner iterator for the current entry.
     *
     * @return \Iterator
     */
    public function getInnerIterator(): \Iterator
    {
        return $this->items;
    }
}


jasny / container

Push — master ( fb1394...c6b41f )

AbstractLoader::next() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like