Spatie\CodeOutline\Elements\Line

Complexity

Total Complexity

7

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes	1
Bugs	1	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	make()	0	9	1
A	getMaximumCharacterPositionDensity()	0	5	2
A	merge()	0	9	2
A	__construct()	0	5	1
A	offsetGet()	0	3	1

<?php

namespace Spatie\CodeOutline\Elements;

use Spatie\Typed\Collection;
use Spatie\Typed\T;

class Line extends Collection
{
    public function __construct(int ...$values)
    {
        parent::__construct(T::int());

        $this->set($values);
    }

    public static function make(int $indentationCount, int $characterCount): self
    {
        $indentationValues = array_fill(0, $indentationCount, -1);

        $characterValues = array_fill(0, $characterCount, 1);

        $lineValues = array_merge($indentationValues, $characterValues);

        return new self(...$lineValues);
    }

    public function merge(self $line): self
    {
        $mergedLine = clone $this;

        foreach ($line as $position => $value) {
            $mergedLine[$position] = ($this[$position] ?? 0) + $value;
        }

        return $mergedLine;
    }

    public function getMaximumCharacterPositionDensity(): int
    {
        return $this->count()
            ? max($this->data)
            : 0;
    }

    public function offsetGet($offset): int
    {
        return parent::offsetGet($offset);

The expression return parent::offsetGet($offset) could return the type null which is incompatible with the type-hinted return integer. Consider adding an additional type-check to rule them out.

    }
}