Nip\Collections\Lazy\Traits\CheckLoadedFunctionsTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Test Coverage

Coverage

33.33%

Importance

Changes	1
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	offsetExists()	0	4	1
A	filter()	0	4	1
A	offsetUnset()	0	4	1
A	offsetGet()	0	4	1
A	count()	0	4	1
A	offsetSet()	0	4	1

<?php

namespace Nip\Collections\Lazy\Traits;

/**
 * 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?

        return 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();
        return parent::offsetUnset($key);
    }

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


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