Dimtrovich\DbDumper\Compressor\GzipCompressor

Complexity

Total Complexity

9

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	open()	0	8	2
A	write()	0	9	2
A	__construct()	0	4	2
A	close()	0	3	1
A	read()	0	10	2

<?php

/**
 * This file is part of dimtrovich/db-dumper".
 *
 * (c) 2024 Dimitri Sitchet Tomkeu <devcode.dst@gmail.com>
 *
 * For the full copyright and license information, please view
 * the LICENSE file that was distributed with this source code.
 */

namespace Dimtrovich\DbDumper\Compressor;

use Dimtrovich\DbDumper\Exceptions\Exception;

class GzipCompressor extends Factory
{
    /**
     * @var false|resource
     */
    private $handler;

    public function __construct()
    {
        if (! function_exists('gzopen')) {
            throw Exception::compressionDriverMissing('gzip');
        }
    }

    /**
     * {@inheritDoc}
     */
    public function open(string $filename, string $mode = 'wb'): bool
    {
        $this->handler = gzopen($filename, $mode);
        if (false === $this->handler) {
            throw Exception::fileNotWritable($filename);
        }

        return true;
    }

    /**
     * {@inheritDoc}
     */
    public function write(string $data): int
    {
        $bytesWritten = gzwrite($this->handler, $data);

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

        if (false === $bytesWritten) {
            throw Exception::failledToWrite();
        }

        return $bytesWritten;
    }

    /**
     * {@inheritDoc}
     */
    public function read(): string
    {
        $content = '';

        while (! gzeof($this->handler)) {

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

            // Read buffer-size bytes
            $content .= gzread($this->handler, 4096); // read 4kb at a time

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

        }

        return $content;
    }

    /**
     * {@inheritDoc}
     */
    public function close(): bool
    {
        return gzclose($this->handler);

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

    }
}