exussum12\CoverageChecker\Loaders\PhpMnd::parseLines()

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function parseLines(): array
    {
        while (($line = fgets($this->file)) !== false) {

It seems like $this->file can also be of type boolean; however, parameter $handle of fgets() does only seem to accept resource, maybe add an additional type check?

            $matches = [];
            $pattern = "/^(?<filename>[^:]+):(?<lineNo>[0-9]+)\. (?<message>.+)/";
            if (preg_match($pattern, $line, $matches)) {
                $this->invalidLines
                    [$matches['filename']]
                    [$matches['lineNo']][] = $matches['message'];
            }
        }

        return array_keys($this->invalidLines);
    }