Thumb::make() - Code Metrics - Inspection of "Thumb class and first example" - robsonvleite/cropper - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Branch — master (aa60db)

by Robson
created 2018-09-25 01:42 UTC
Thumb::make() C

↳ Parent: Thumb
Complexity

Conditions	13
Paths	45
Size

Total Lines	48
Code Lines	29
Duplication

Lines	0
Ratio	0 %
Importance

Changes
Metric	Value
cc	13
eloc	29
nc	45
nop	3
dl	0
loc	48
rs	6.6166
c	0
b	0
f	0
How to fix Complexity

<?php

namespace CoffeeCode\Cropper;

/**
 * Class CoffeeCode Thumb
 *
 * @author Robson V. Leite <https://github.com/robsonvleite>
 * @package CoffeeCode\Cropper
 */
class Thumb
{
    /** @var string */
    private $cachePath;

    /** @var array */
    private $cacheSize;

    /** @var string */
    private $imagePath;

    /** @var string */
    private $imageName;

    /** @var string */
    private $imageMime;

    /** @var string */
    private $imageInfo;

    /**
     * Allow jpg and png to thumb and cache generate
     *
     * @var array allowed media types
     */
    private static $allowedExt = ['image/jpeg', "image/png"];

    /**
     * @param string $cachePath
     * @param int $jpgQuality
     * @param int $pngCompressor
     */
    public function __construct(string $cachePath, int $jpgQuality = 75, int $pngCompressor = 5)
    {
        $this->cachePath = $cachePath;
        $this->cacheSize = [$jpgQuality, $pngCompressor];
    }

    /**
     * Make an thumb image
     *
     * @param string $imagePath
     * @param int $width
     * @param int|null $height
     * @return string
     * @throws \Exception
     */
    function make(string $imagePath, int $width, int $height = null): string

    {
        if (!file_exists($imagePath)) {
            throw new \Exception("Image not found");
        }

        $this->imagePath = $imagePath;
        $this->imageMime = mime_content_type($this->imagePath);
        $this->imageInfo = pathinfo($this->imagePath);
        $this->imageName = md5($this->imageInfo['basename']) . "x{$width}{$height}" . ($this->imageMime == "image/jpeg" ? ".jpg" : ".png");

        if (!in_array($this->imageMime, self::$allowedExt)) {
            throw new \Exception("Not a valid JPG or PNG image");
        }

        if (!file_exists($this->cachePath) || !is_dir($this->cachePath)) {
            if (!mkdir($this->cachePath, 0755)) {
                throw new \Exception("Could not create cache folder");
            }
        }

        if (file_exists("{$this->cachePath}/{$this->imageName}") && is_file("{$this->cachePath}/{$this->imageName}")) {
            return "{$this->cachePath}/{$this->imageName}";
        }

        list($src_w, $src_h) = getimagesize($this->imagePath);
        $height = ($height ?? ($width * $src_h) / $src_w);

        $src_x = 0;
        $src_y = 0;

        $cmp_x = $src_w / $width;
        $cmp_y = $src_h / $height;

        if ($cmp_x > $cmp_y) {
            $src_w = round($src_w / $cmp_x * $cmp_y);
            $src_x = round(($src_w - ($src_w / $cmp_x * $cmp_y))); //2
        } elseif ($cmp_y > $cmp_x) {
            $src_h = round($src_h / $cmp_y * $cmp_x);
            $src_y = round(($src_h - ($src_h / $cmp_y * $cmp_x))); //2
        }

        if ($this->imageMime == "image/jpeg") {
            return $this->fromJpg($width, $height, $src_x, $src_y, $src_w, $src_h);
            return $this->fromJpg($width, $height, /** @scrutinizer ignore-type */ $src_x, $src_y, $src_w, $src_h);
        }

        if ($this->imageMime == "image/png") {
            return $this->fromPng($width, $height, $src_x, $src_y, $src_w, $src_h);
            return $this->fromPng($width, $height, /** @scrutinizer ignore-type */ $src_x, $src_y, $src_w, $src_h);
        }
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

class MyClass implements ReturnsInt {
    public function returnsIntHinted(): int
    {
        if (foo()) {
            return 123;
        }
        // here: null is implicitly returned
    }
}
    }

    /**
     * Clear cache
     *
     * @param string|null $imageName
     * @example $t->flush("images/image.jpg"); clear image name and variations size
     * @example $t->flush(); clear all image cache folder
     */
    public function flush(string $imageName = null): void
    {
        $scan = scandir($this->cachePath);
        $name = ($imageName ? md5(pathinfo($imageName)['basename']) : null);

        foreach ($scan as $file) {
            $file = "{$this->cachePath}/{$file}";
            if ($imageName && strpos($file, $name) && is_file($file)) {
                echo "clear {$file}<br>";
                unlink($file);
            }

            if (!$name && is_file($file)) {
                unlink($file);
            }
        }
    }

    /**
     * @param int $width
     * @param int $height
     * @param int $src_x
     * @param int $src_y
     * @param int $src_w
     * @param int $src_h
     * @return string
     */
    private function fromJpg(int $width, int $height, int $src_x, int $src_y, int $src_w, int $src_h): string
    {
        $thumb = imagecreatetruecolor($width, $height);
        $source = imagecreatefromjpeg($this->imagePath);
        imagecopyresized($thumb, $source, 0, 0, $src_x, $src_y, $width, $height, $src_w, $src_h);
        imagejpeg($thumb, "{$this->cachePath}/{$this->imageName}", $this->cacheSize[0]);

        imagedestroy($thumb);
        imagedestroy($source);

        return "{$this->cachePath}/{$this->imageName}";
    }

    /**
     * @param int $width
     * @param int $height
     * @param int $src_x
     * @param int $src_y
     * @param int $src_w
     * @param int $src_h
     * @return string
     */
    private function fromPng(int $width, int $height, int $src_x, int $src_y, int $src_w, int $src_h): string
    {
        $thumb = imagecreatetruecolor($width, $height);
        $source = imagecreatefrompng($this->imagePath);

        imagealphablending($thumb, false);
        imagesavealpha($thumb, true);
        imagecopyresized($thumb, $source, 0, 0, $src_x, $src_y, $width, $height, $src_w, $src_h);
        imagepng($thumb, "{$this->cachePath}/{$this->imageName}", $this->cacheSize[1]);

        imagedestroy($thumb);
        imagedestroy($source);

        return "{$this->cachePath}/{$this->imageName}";
    }
}
robsonvleite / cropper

Branch — master (aa60db)

Thumb::make() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like
