Shoman4eg\Nalog\DTO\IncomeServiceItem

Complexity

Total Complexity

6

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getAmount()	0	3	1
A	getQuantity()	0	3	1
A	jsonSerialize()	0	6	1
A	__construct()	0	5	1
A	getTotalAmount()	0	4	1
A	getName()	0	3	1

<?php
declare(strict_types=1);

namespace Shoman4eg\Nalog\DTO;

use Brick\Math\BigDecimal;

/**
 * @author Artem Dubinin <artem@dubinin.me>
 */
final class IncomeServiceItem implements \JsonSerializable
{
    private string $name;

    /** @var float|int|string */
    private $amount;

    /** @var float|int */
    private $quantity;

    /**
     * @param float|int|string $amount
     * @param float|int        $quantity
     */
    public function __construct(string $name, $amount, $quantity)
    {
        $this->name = $name;
        $this->amount = $amount;
        $this->quantity = $quantity;
    }

    public function jsonSerialize(): array
    {
        return [
            'name' => $this->name,
            'amount' => $this->amount,
            'quantity' => $this->quantity,
        ];
    }

    public function getName(): string
    {
        return $this->name;
    }

    /**
     * @return float|int|string
     */
    public function getAmount()
    {
        return $this->amount;
    }

    /**
     * @return float|int
     */
    public function getQuantity()
    {
        return $this->quantity;
    }

    public function getTotalAmount(): BigDecimal
    {
        return BigDecimal::of($this->amount)

The expression return Brick\Math\BigDecimal::of($this->amount)->multipliedBy($this->quantity) could return the type Brick\Math\BigInteger|Brick\Math\BigRational which is incompatible with the type-hinted return Brick\Math\BigDecimal. Consider adding an additional type-check to rule them out.

            ->multipliedBy($this->quantity)
        ;
    }
}