Stu\Module\Trade\Lib\DealAccountWrapper

Complexity

Total Complexity

8

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	isOverStorage()	0	4	1
A	getStorageSum()	0	7	1
A	getId()	0	4	1
A	getStorageCapacity()	0	4	1
A	getStorage()	0	9	2
A	getShip()	0	4	1
A	__construct()	0	5	1

<?php

declare(strict_types=1);

namespace Stu\Module\Trade\Lib;

use Override;

The type Override was not found. Maybe you did not declare it correctly or list all dependencies?

use Stu\Orm\Entity\ShipInterface;
use Stu\Orm\Entity\StorageInterface;
use Stu\Orm\Entity\TradePostInterface;
use Stu\Orm\Repository\StorageRepositoryInterface;

final class DealAccountWrapper implements DealAccountWrapperInterface
{
    /** @var array<StorageInterface> */
    private ?array $storage = null;

    public function __construct(
        private StorageRepositoryInterface $storageRepository,
        private TradePostInterface $tradePost,
        private int $userId
    ) {
    }

    #[Override]
    public function getId(): int
    {
        return $this->tradePost->getId();
    }

    #[Override]
    public function getShip(): ShipInterface
    {
        return $this->tradePost->getShip();
    }

    /** @return array<StorageInterface> */
    private function getStorage(): array
    {
        if ($this->storage === null) {
            $this->storage = $this->storageRepository->getByTradePostAndUser(
                $this->tradePost->getId(),
                $this->userId
            );
        }
        return $this->storage;

The expression return $this->storage could return the type null which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

    }

    #[Override]
    public function getStorageSum(): int
    {
        return array_reduce(
            $this->getStorage(),
            fn (int $value, StorageInterface $storage): int => $value + $storage->getAmount(),
            0
        );
    }

    #[Override]
    public function isOverStorage(): bool
    {
        return $this->getStorageSum() > $this->tradePost->getStorage();
    }

    #[Override]
    public function getStorageCapacity(): int
    {
        return $this->tradePost->getStorage();
    }
}