Spatie\CodeOutline\Elements\Line

Complexity

Total Complexity

9

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	make()	0	9	1
A	__construct()	0	5	1
A	getMaximumCharacterPositionDensity()	0	5	2
A	merge()	0	17	4
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(Line $line): Line
    {
        $mergedLine = new Line();

        $largestLine = $this->count() > $line->count()
            ? $this
            : $line;

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

        if (! $largestLine->count()) {
            return new EmptyLine();
        }

        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.

    }
}