InShore\Bookwhen\Responses\Concerns\ArrayAccessible

Complexity

Total Complexity

4

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

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

<?php

declare(strict_types=1);

namespace InShore\Bookwhen\Responses\Concerns;

use BadMethodCallException;

/**
 * @template TArray of array
 *
 * @mixin Response<TArray>
 */
trait ArrayAccessible
{
    /**
     * {@inheritDoc}
     */
    public function offsetExists(mixed $offset): bool
    {
        return array_key_exists($offset, $this->toArray());

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

    }

    /**
     * {@inheritDoc}
     */
    public function offsetGet(mixed $offset): mixed
    {
        return $this->toArray()[$offset];
    }

    /**
     * {@inheritDoc}
     */
    public function offsetSet(mixed $offset, mixed $value): never

The parameter $offset is not used and could be removed.

The parameter $value is not used and could be removed.

    {
        throw new BadMethodCallException('Cannot set response attributes.');
    }

    /**
     * {@inheritDoc}
     */
    public function offsetUnset(mixed $offset): never

The parameter $offset is not used and could be removed.

    {
        throw new BadMethodCallException('Cannot unset response attributes.');
    }
}