Jabe\Engine\Impl\Batch\DeploymentMappings

Complexity

Total Complexity

7

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getOverallIdCount()	0	3	1
A	get()	0	6	2
A	of()	0	5	1
A	remove()	0	8	2
A	add()	0	5	1

<?php

namespace Jabe\Engine\Impl\Batch;

class DeploymentMappings extends \ArrayObject
{
    protected $overallIdCount;

    public static function of(DeploymentMapping $mapping): DeploymentMappings
    {
        $mappings = new DeploymentMappings();
        $mappings->add($mapping);
        return $mappings;
    }

    public function add(DeploymentMapping $mapping): bool
    {
        $this->overallIdCount += $mapping->getCount();
        $this[] = $mapping;
        return true;
    }

    public function get(int $mappingIndex)

The parameter $mappingIndex is not used and could be removed.

    {
        if (isset($this[$index])) {

The variable $index seems to be never defined.

            return $this[$index];
        } else {
            throw new \Exception(sprintf("Undefined array key %d", $index));
        }
    }

    public function remove(int $mappingIndex): bool
    {
        if (isset($this[$mappingIndex])) {
            $this->overallIdCount -= $this[$mappingIndex]->getCount();
            unset($this[$mappingIndex]);
            return true;
        } else {
            return false;
        }
    }

    public function getOverallIdCount(): int
    {
        return $this->overallIdCount;
    }
}