Nip\Collections\Traits\OperationsTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

13.33%

Importance

Changes	1
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	count()	0	4	1
A	size()	0	8	2
A	isEmpty()	0	4	1
A	isNotEmpty()	0	4	1
A	clear()	0	7	1

<?php

namespace Nip\Collections\Traits;

use Traversable;

/**
 * Class OperationsTrait
 * @package Nip\Collections\Traits
 */
trait OperationsTrait
{

    /**
     * Returns the number of parameters.
     *
     * @return int The number of parameters
     */
    public function count()
    {
        return count($this->items);

The property items does not exist. Did you maybe forget to declare it?

    }

    /**
     * Returns number of items in $collection.
     *
     * @return int
     */
    public function size()
    {
        $result = 0;
        foreach ($this as $value) {

The expression $this of type this<Nip\Collections\Traits\OperationsTrait> is not traversable.

            $result++;
        }
        return $result;
    }

    /**
     * @return bool
     */
    public function isEmpty()
    {
        return $this->count() < 1;
    }

    /**
     * @return bool
     */
    public function isNotEmpty()
    {
        return !$this->isEmpty();
    }

    /**
     * @return $this
     */
    public function clear()
    {
        $this->rewind();

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

        $this->items = [];

        return $this;
    }
}