CSFCloud\TempFiles\TempFile

Complexity

Total Complexity

11

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getId()	0	2	1
A	__construct()	0	3	1
A	getPath()	0	2	1
A	delete()	0	2	1
A	getText()	0	2	1
B	getLastLines()	0	23	5
A	__toString()	0	2	1

<?php

namespace CSFCloud\TempFiles;

class TempFile {

    private $id;
    private $full_path;

    public function __construct(string $id, string $path) {
        $this->id = $id;
        $this->full_path = $path;
    }

    public function getId() : string {
        return $this->id;
    }

    public function getPath() : string {
        return $this->full_path;
    }

    public function __toString() : string {
        return $this->full_path;
    }

    public function getText() : string {
        return file_get_contents($this->full_path);
    }

    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;
    }

    public function delete() {
        unlink($this->full_path);
    }

}