Nip\Router\RouteCollections\Traits\ArrayAccessTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Test Coverage

Coverage

20%

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	offsetUnset()	0	3	1
A	offsetSet()	0	3	1
A	offsetExists()	0	3	1
A	offsetGet()	0	3	1

<?php
declare(strict_types=1);

namespace Nip\Router\RouteCollections\Traits;

use Nip\Router\Route\Route;

/**
 * Trait ArrayAccessTrait
 * @package Nip\Router\RouteCollections\Traits
 */
trait ArrayAccessTrait
{
    /**
     * @param mixed $offset
     * @return bool
     */
    public function offsetExists($offset): bool
    {
        return $this->has($offset);

It seems like has() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @param mixed $offset
     * @return Route|null
     */
    public function offsetGet($offset): mixed
    {
        return $this->get($offset);

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @param mixed $offset
     * @param mixed $value
     */
    public function offsetSet($offset, $value): void
    {
        $this->add($offset, $value);

It seems like add() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @param mixed $offset
     */
    public function offsetUnset($offset): void
    {
        $this->remove($offset);

It seems like remove() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}