CSFCloud\TempFiles\TempFile::getLastLines()

Complexity

Conditions	5
Paths	8

Size

Total Lines	23
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function getLastLines(int $lines = 10, $buffer = 4096) : string {
        $f = fopen($this->full_path, "rb");
        fseek($f, -1, SEEK_END);

It seems like $f can also be of type false; however, parameter $handle of fseek() does only seem to accept resource, maybe add an additional type check?

        if(fread($f, 1) != "\n") {

It seems like $f can also be of type false; however, parameter $handle of fread() does only seem to accept resource, maybe add an additional type check?

            $lines -= 1;
        }
        $output = '';
        $chunk = '';

The assignment to $chunk is dead and can be removed.

        while(ftell($f) > 0 && $lines >= 0) {

It seems like $f can also be of type false; however, parameter $handle of ftell() does only seem to accept resource, maybe add an additional type check?

            $seek = min(ftell($f), $buffer);
            fseek($f, -$seek, SEEK_CUR);
            $output = ($chunk = fread($f, $seek)).$output;
            fseek($f, -mb_strlen($chunk, '8bit'), SEEK_CUR);
            $lines -= substr_count($chunk, "\n");
        }

        while($lines++ < 0) {
            $output = substr($output, strpos($output, "\n") + 1);
        }
    
        fclose($f);

It seems like $f can also be of type false; however, parameter $handle of fclose() does only seem to accept resource, maybe add an additional type check?

        return $output;
    }
}