ComicVine\Api\Filters\FilterValidation

Complexity

Total Complexity

37

Size/Duplication

Total Lines	103
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	4
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
B	isIntAndBetween()	0	14	5
A	isKeyAndValueAre()	0	10	3
A	isParamOfTypeMultiple()	0	8	1
A	isParamOfTypeSingle()	0	8	2
A	isParamAValue()	0	10	3

<?php

namespace ComicVine\Api\Filters;

/**
 * Trait FilterCheck
 *
 * @package grzgajda/comicvine-api
 * @author  Grzegorz Gajda <grz.gajda@outlook.com>
 */
trait FilterValidation
{
    /**
     * Check if input is integer and stay
     * in range between $min and $max.
     *
     * @param mixed      $input Input field
     * @param int        $min   Min range of input
     * @param int|string $max   Max range of input
     *
     * @return bool
     */
    public function isIntAndBetween($input, $min, $max = "")
    {
        if (is_int($input) === false) {
            return false;
        }

        if ($input < $min
            || ($input > $max && is_int($max) === true)
        ) {
            return false;
        }

        return true;
    }

    /**
     * Check if key or value is not of any described types.
     *
     * @param string       $key         Value of key
     * @param string       $keyParams   Not-Types of key
     * @param string       $value       Value of value
     * @param string|array $valueParams Not-Types of value
     *
     * @return bool
     */
    public function isKeyAndValueAre($key, $keyParams, $value, $valueParams)
    {
        if ($this->isParamOfTypeSingle($key, $keyParams) === false
            || $this->isParamOfTypeSingle($value, $valueParams) === false

It seems like $valueParams defined by parameter $valueParams on line 48 can also be of type array; however, ComicVine\Api\Filters\FilterValidation::isParamOfTypeSingle() does only seem to accept string, maybe add an additional type check?

        ) {
            return false;
        }

        return true;
    }

    /**
     * Check if value is any of these types.
     *
     * @param string $param Value to check
     * @param array  $types List of types
     *
     * @return bool
     */
    public function isParamOfTypeMultiple($param, $types)
    {
        array_walk($types, function (&$type) use ($param) {
            $type = $this->isParamOfTypeSingle($param, $type);
        });

        return in_array(true, $types);
    }

    /**
     * Check if value is not that type.
     *
     * @param string $param Value
     * @param string $type  Type
     *
     * @return bool
     */
    public function isParamOfTypeSingle($param, $type)
    {
        if (is_array($type) === true) {
            return $this->isParamOfTypeMultiple($param, $type);
        }

        return call_user_func("is_$type", $param);
    }

    /**
     * Check if value has a proper value and key a proper type.
     *
     * @param string $key      Key of value
     * @param string $keyParam Key possible types
     * @param string $param    Param value
     * @param array  $string   Param possible values.
     *
     * @return boolean
     */
    public function isParamAValue($key, $keyParam, $param, array $string)
    {
        if ($this->isParamOfTypeSingle($key, $keyParam) === true
            && in_array($param, $string) === true
        ) {
            return true;
        }

        return false;
    }
}