Kontrolio\Rules\Core\Length

Complexity

Total Complexity

17

Size/Duplication

Total Lines	92
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	20	8
B	isValid()	0	31	9

<?php

namespace Kontrolio\Rules\Core;

use InvalidArgumentException;
use LogicException;
use Kontrolio\Rules\AbstractRule;

/**
 * Length validation rule.
 *
 * @package Kontrolio\Rules\Core
 */
class Length extends AbstractRule
{
    /**
     * Minimum length.
     *
     * @var int
     */
    private $min;

    /**
     * Maximum length.
     *
     * @var int
     */
    private $max;

    /**
     * Charset.
     *
     * @var string
     */
    private $charset;

    /**
     * Length validation rule constructor.
     *
     * @param int $min
     * @param int $max
     * @param string $charset
     * @throws InvalidArgumentException
     * @throws LogicException
     */
    public function __construct(
        $min = null,
        $max = null,
        $charset = 'UTF-8'
    ) {
        if ($min === null && $max === null) {
            throw new InvalidArgumentException('Either option "min" or "max" must be given.');
        }

        if ($min !== null && $max !== null && $min > $max) {
            throw new LogicException('"Min" option cannot be greater that "max".');
        }

        if ($max !== null && $max < $min) {
            throw new LogicException('"Max" option cannot be less that "min".');
        }

        $this->min = $min;
        $this->max = $max;
        $this->charset = $charset;
    }

    /**
     * Validates input.
     *
     * @param mixed $input
     *
     * @return bool
     */
    public function isValid($input = null)
    {
        if ($input === null || $input === '') {
            return false;
        }

        $input = (string) $input;

        if (!$invalidCharset = !@mb_check_encoding($input, $this->charset)) {
            $length = mb_strlen($input, $this->charset);
        }

        if ($invalidCharset) {
            $this->violations[] = 'charset';

            return false;
        }

        if ($this->max !== null && $length > $this->max) {

The variable $length does not seem to be defined for all execution paths leading up to this point.

            $this->violations[] = 'max';

            return false;
        }

        if ($this->min !== null && $length < $this->min) {
            $this->violations[] = 'min';

            return false;
        }

        return true;
    }
}