BoneMvc\Module\Unicorn\Collection\UnicornCollection

Complexity

Total Complexity

14

Size/Duplication

Total Lines	101
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	update()	0	9	2
A	append()	0	4	1
A	current()	0	4	1
A	findKey()	0	12	3
A	findById()	0	12	3
A	toArray()	0	14	2
A	jsonSerialize()	0	4	1
A	__toString()	0	4	1

<?php

declare(strict_types=1);

namespace BoneMvc\Module\Unicorn\Collection;

use BoneMvc\Module\Unicorn\Entity\Unicorn;
use Doctrine\Common\Collections\ArrayCollection;
use JsonSerializable;
use LogicException;

class UnicornCollection extends ArrayCollection implements JsonSerializable
{
    /**
     * @param Unicorn $unicorn
     * @return $this
     * @throws LogicException
     */
    public function update(Unicorn $unicorn): UnicornCollection
    {
        $key = $this->findKey($unicorn);
        if($key) {

The expression $key of type integer|null is loosely compared to true; this is ambiguous if the integer can be zero. You might want to explicitly use !== null instead.

            $this->offsetSet($key,$unicorn);
            return $this;
        }
        throw new LogicException('Unicorn was not in the collection.');
    }

    /**
     * @param Unicorn $unicorn
     */
    public function append(Unicorn $unicorn): void
    {
        $this->add($unicorn);
    }

    /**
     * @return Unicorn|null
     */
    public function current(): ?Unicorn
    {
        return parent::current();
    }

    /**
     * @param Unicorn $unicorn
     * @return int|null
     */
    public function findKey(Unicorn $unicorn): ?int
    {
        $it = $this->getIterator();
        $it->rewind();
        while($it->valid()) {
            if($it->current()->getId() == $unicorn->getId()) {
                return $it->key();
            }
            $it->next();
        }
        return null;
    }

    /**
     * @param int $id
     * @return Unicorn|null
     */
    public function findById(int $id): ?Unicorn
    {
        $it = $this->getIterator();
        $it->rewind();
        while($it->valid()) {
            if($it->current()->getId() == $id) {
                return $it->current();
            }
            $it->next();
        }
        return null;
    }

    /**
     * @return array
     */
    public function toArray(): array
    {
        $collection = [];
        $it = $this->getIterator();
        $it->rewind();
        while($it->valid()) {
            /** @var Unicorn $row */
            $row = $it->current();
            $collection[] = $row->toArray();
            $it->next();
        }

        return $collection;
    }

    /**
     * @return string
     */
    public function jsonSerialize(): string
    {
        return \json_encode($this->toArray());
    }

    /**
     * @return string
     */
    public function __toString(): string
    {
        return $this->jsonSerialize();
    }
}