Money\Sums

Complexity

Total Complexity

18

Size/Duplication

Total Lines	70
Duplicated Lines	31.43 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	plus()	8	8	3
A	minus()	8	8	3
A	greater()	3	6	3
A	equal()	3	6	3
B	__toString()	0	26	5

<?php

/**
 * Money sums comparator
 *
 * @author Alexey Krupskiy <admin@inji.ru>
 * @link http://inji.ru/
 * @copyright 2016 Alexey Krupskiy
 * @license https://github.com/injitools/cms-Inji/blob/master/LICENSE
 */

namespace Money;

class Sums extends \Object
{
    public $sums = [];

    public function __construct($sums)
    {
        $this->sums = $sums;
    }

    function plus(Sums $sums)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $newSums = $this->sums;
        foreach ($sums->sums as $currency_id => $sum) {
            $newSums[$currency_id] = isset($newSums[$currency_id]) ? $newSums[$currency_id] + $sum : $sum;
        }
        return new Sums($newSums);
    }

    function minus(Sums $sums)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $newSums = $this->sums;
        foreach ($sums->sums as $currency_id => $sum) {
            $newSums[$currency_id] = isset($newSums[$currency_id]) ? $newSums[$currency_id] - $sum : -$sum;
        }
        return new Sums($newSums);
    }

//Equal, Less and Greater
    public function greater(Sums $sums)
    {
        if (count($this->sums) == count($sums->sums) && isset($sums->sums[key($this->sums)])) {

This code seems to be duplicated across your project.

            return current($this->sums) > current($sums->sums);
        }
    }

    public function equal(Sums $sums)
    {
        if (count($this->sums) == count($sums->sums) && isset($sums->sums[key($this->sums)])) {

This code seems to be duplicated across your project.

            return current($this->sums) == current($sums->sums);
        }
    }

    function __toString()

It is generally recommended to explicitly declare the visibility for methods.

    {
        $string = '';
        $first = true;
        foreach ($this->sums as $currency_id => $sum) {
            if ($first) {
                $first = false;
            } else {
                $string.= '<br />';
            }
            $string.= '<span style="white-space:nowrap;">';
            $string.= number_format($sum, 2, '.', ' ');
            if (\App::$cur->money) {
                $currency = \Money\Currency::get($currency_id);
                if ($currency) {
                    $string.= ' ' . $currency->acronym();
                } else {
                    $string.= ' руб.';
                }
            } else {
                $string.= ' руб.';
            }
            $string.= '</span>';
        }
        return $string;
    }

}