Assimtech\Money\Accountant

Complexity

Total Complexity

10

Size/Duplication

Total Lines	107
Duplicated Lines	16.82 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	checkCurrenciesMatch()	0	10	2
A	add()	9	9	1
A	subtract()	9	9	1
A	multiply()	0	7	1
A	divide()	0	7	1
A	sum()	0	20	4

<?php

namespace Assimtech\Money;

use InvalidArgumentException;

class Accountant
{
    /**
     * @param Money $money1
     * @param Money $money2
     * @throws InvalidArgumentException
     */
    protected function checkCurrenciesMatch(Money $money1, Money $money2)
    {
        if ($money1->getCurrency()->getCode() !== $money2->getCurrency()->getCode()) {
            throw new InvalidArgumentException(sprintf(
                'Cannot work with monies of differing currencies (%s, %s)',
                $money1,
                $money2
            ));
        }
    }

    /**
     * $money1 + $money2
     *
     * @param Money $money1
     * @param Money $money2
     * @return Money
     */
    public function add(Money $money1, Money $money2)

This method seems to be duplicated in your project.

    {
        $this->checkCurrenciesMatch($money1, $money2);

        return new Money(
            $money1->getAmount() + $money2->getAmount(),
            $money1->getCurrency()
        );
    }

    /**
     * $money1 - $money2
     *
     * @param Money $money1
     * @param Money $money2
     * @return Money
     */
    public function subtract(Money $money1, Money $money2)

This method seems to be duplicated in your project.

    {
        $this->checkCurrenciesMatch($money1, $money2);

        return new Money(
            $money1->getAmount() - $money2->getAmount(),
            $money1->getCurrency()
        );
    }

    /**
     * $money * $fraction
     *
     * @param Money $money
     * @param float|integer $fraction
     * @return Money
     */
    public function multiply(Money $money, $fraction)
    {
        return new Money(
            $money->getAmount() * $fraction,
            $money->getCurrency()
        );
    }

    /**
     * $money / $fraction
     *
     * @param Money $money
     * @param float|integer $fraction
     * @return Money
     */
    public function divide(Money $money, $fraction)
    {
        return new Money(
            $money->getAmount() / $fraction,
            $money->getCurrency()
        );
    }

    /**
     * @param Money[] $monies
     * @return Money|null
     */
    public function sum(array $monies)
    {
        $totalMoney = null;

        foreach ($monies as $money) {
            if (!$money instanceof Money) {
                throw new InvalidArgumentException('$monies must be an array of Money');
            }

            if ($totalMoney === null) {
                $totalMoney = clone $money;

                continue;
            }

            $totalMoney = $this->add($totalMoney, $money);
        }

        return $totalMoney;
    }
}