HustleWorks\Chute\ImageValidator

Complexity

Total Complexity

11

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	validate()	0	12	2
A	_checkWidth()	0	8	3
A	_checkHeight()	0	8	3
A	_checkType()	0	6	3

<?php

namespace HustleWorks\Chute;

use HustleWorks\Chute\DTO\ImageFile;
use HustleWorks\Chute\DTO\ImageRuleConfiguration;
use HustleWorks\Chute\Contracts\ImageValidatorInterface;

class ImageValidator implements ImageValidatorInterface
{
    private $reasons_to_fail = [];

    /**
     * @param ImageFile              $file
     * @param ImageRuleConfiguration $rules
     * @return \HustleWorks\Chute\ServiceResponse
     */
    public function validate(ImageFile $file, ImageRuleConfiguration $rules): ServiceResponse
    {
        $this->_checkWidth($file, $rules);
        $this->_checkHeight($file, $rules);
        $this->_checkType($file, $rules);

        return new StandardServiceResponse(
            !sizeof($this->reasons_to_fail),
            [],
            sizeof($this->reasons_to_fail) ? "Message was invalid for the following reasons: " . implode(', ', $this->reasons_to_fail) : "Valid image."
        );
    }

    private function _checkWidth(ImageFile $file, ImageRuleConfiguration $rules)
    {
        if ($file->width < $rules->min_width) {
            $this->reasons_to_fail[] = 'image too narrow';
        } elseif ($file->width > ($rules->max_width ?? PHP_INT_MAX)) {
            $this->reasons_to_fail[] = 'image too wide';
        }
    }

    private function _checkHeight(ImageFile $file, ImageRuleConfiguration $rules)
    {
        if ($file->height < $rules->min_height) {
            $this->reasons_to_fail[] = 'image too short';
        } elseif ($file->height > ($rules->max_height ?? PHP_INT_MAX)) {
            $this->reasons_to_fail[] = 'image too tall';
        }
    }

    private function _checkType(ImageFile $file, ImageRuleConfiguration $rules)
    {
        if ($rules->mime_types and !in_array($file->mime_type, $rules->mime_types)) {

Using logical operators such as and instead of && is generally not recommended.

            $this->reasons_to_fail[] = 'file type not allowed';
        }
    }

}