bytic / Collections

src/Lazy/Traits/CheckLoadedFunctionsTrait.php

<?php

namespace Nip\Collections\Lazy\Traits;

use Nip\Collections\CollectionInterface;

/**
 * Trait CheckLoadedFunctionsTrait
 * @package Nip\Collections\Lazy\Traits
 * @internal
 */
trait CheckLoadedFunctionsTrait
{
    /**
     * @inheritDoc
     */
    public function offsetSet($key, $value)
    {
        $this->load();

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

        parent::offsetSet($key, $value);
    }

    /**
     * @inheritDoc
     */
    public function offsetGet($key)
    {
        $this->load();
        return parent::offsetGet($key);
    }

    /**
     * @inheritDoc
     */
    public function offsetExists($key)
    {
        $this->load();
        return parent::offsetExists($key);
    }

    /**
     * @inheritDoc
     */
    public function offsetUnset($key)
    {
        $this->load();
        parent::offsetUnset($key);
    }

    /**
     * @inheritDoc
     */
    public function filter(callable $callback = null): CollectionInterface
    {
        $this->load();
        return parent::filter($callback);
    }


    /**
     * @inheritDoc
     */
    public function count()
    {
        $this->load();
        return parent::count();
    }
}