bytic / container

src/Traits/ContainerArrayAccessTrait.php

<?php

namespace Nip\Container\Traits;

/**
 * Class ContainerPersistenceTrait
 * @package Nip\Container
 */
trait ContainerArrayAccessTrait
{
    /**
     * Determine if a given offset exists.
     *
     * @param string $key
     * @return bool
     */
    public function offsetExists($key): bool
    {
        return $this->has($key);

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

    }

    /**
     * Get the value at a given offset.
     *
     * @param string $key
     * @return mixed
     */
    public function offsetGet($key): mixed
    {
        return $this->get($key);

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

    }

    /**
     * Set the value at a given offset.
     *
     * @param string $key
     * @param mixed $value
     * @return void
     */
    public function offsetSet($key, $value): void
    {
        $this->add($key, $value);

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

    }

    /**
     * Unset the value at a given offset.
     *
     * @param string $key
     * @return void
     */
    public function offsetUnset($key): void
    {
        $this->remove($key);

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

    }

    /**
     * Dynamically access container services.
     *
     * @param string $key
     * @return mixed
     */
    public function __get($key)
    {
        return $this[$key];
    }

    /**
     * Dynamically set container services.
     *
     * @param string $key
     * @param mixed $value
     * @return void
     */
    public function __set($key, $value)
    {
        $this[$key] = $value;
    }
}