Nip\Collections\Traits\OperationsOnItemsTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Test Coverage

Coverage

87.5%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	valueSubtract()	0	9	2
A	valueAdd()	0	16	4
A	each()	0	9	3

<?php

namespace Nip\Collections\Traits;

use Exception;

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

    /**
     * Execute a callback over each item.
     *
     * @param  callable  $callback
     * @return $this
     */
    public function each(callable $callback)
    {
        foreach ($this as $key => $item) {
            if ($callback($item, $key) === false) {
                break;
            }
        }

        return $this;
    }

    /**
     * @param $key
     * @param $value
     *
     * @throws Exception
     */
    public function valueAdd($key, $value)
    {
        if (!$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?

            $this->set($key, $value);

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

            return;
        }

        $currentValue = $this->get($key, 0);

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

        $currentValue = $currentValue === null ? 0 : $currentValue;

        if (!is_numeric($currentValue)) {
            throw new Exception("Current value for key {$key} is not numeric. [{$value}] provided. ");
        }

        $this->set($key, ($currentValue + $value));
    }

    /**
     * @param $key
     * @param $value
     *
     * @throws Exception
     */
    public function valueSubtract($key, $value)
    {
        $currentValue = $this->get($key, 0);

        if (!is_numeric($currentValue)) {
            throw new Exception("Current value for key {$key} is not numeric. [{$value}] provided. ");
        }

        $this->set($key, ($currentValue - $value));
    }
}