Smoren\Containers\Structs\MappedLinkedListIterator

Complexity

Total Complexity

6

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	rewind()	0	3	1
A	valid()	0	3	1
A	__construct()	0	3	1
A	key()	0	3	1
A	next()	0	3	1
A	current()	0	3	1

<?php

namespace Smoren\Containers\Structs;

use Iterator;

/**
 * Class MappedLinkedListIterator
 */
class MappedLinkedListIterator implements Iterator
{
    /**
     * @var MappedLinkedList iterator owner
     */
    protected MappedLinkedList $owner;
    /**
     * @var LinkedListItem|null current position
     */
    protected ?LinkedListItem $position = null;

    /**
     * LinkedListIterator constructor.
     * @param MappedLinkedList $owner iterator owner
     */
    public function __construct(MappedLinkedList $owner)
    {
        $this->owner = $owner;
    }

    /**
     * @inheritDoc
     */
    public function current()
    {
        return $this->position->getData();

The method getData() does not exist on null.

    }

    /**
     * @inheritDoc
     */
    public function next(): void
    {
        $this->position = $this->position->getNext();
    }

    /**
     * @inheritDoc
     */
    public function key(): string
    {
        return $this->position->getExtra();

The expression return $this->position->getExtra() could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }

    /**
     * @inheritDoc
     */
    public function valid(): bool
    {
        return $this->position !== null;
    }

    /**
     * @inheritDoc
     */
    public function rewind(): void
    {
        $this->position = $this->owner->getList()->getFirst();
    }
}