Nip\Collections\Traits\OperationsOnItemsTrait::valueAdd()

Complexity

Conditions	4
Paths	5

Size

Total Lines	16
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	4.128

Importance

Changes	1
Bugs	0	Features	0

<?php
    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));
    }