w3l\Holt45\Get

Complexity

Total Complexity

6

Size/Duplication

Total Lines	61
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	chkGet()	4	4	1
A	assignFromGet()	4	4	1
A	chkGetAll()	15	15	4

<?php
/**
 * Get.php
 */
namespace w3l\Holt45;

/**
 * Check/assign from superglobal $_GET
 */
trait Get

This class seems to be duplicated in your project.

{
    /**
     * Check $_GET
     *
     * Example:
     * ```php
     * if(chkGet("s") == "a") instead of if(isset($_GET["s"]) && $_GET["s"] == "a")
     * ```
     *
     * @param string $getKey Get-key.
     * @return bool|string
     */
    public static function chkGet($getKey)
    {
        return filter_input(INPUT_GET, $getKey);
    }

    /**
     * Assign value from $_GET
     *
     * Example:
     * ```php
     * $var = assignFromGet("a") instead of $var = ((!empty($_GET["s"])) ? $_GET["s"] : "");
     * ```
     *
     * @param string $getKey Get-key.
     * @return string
     */
    public static function assignFromGet($getKey)
    {
        return (string)filter_input(INPUT_GET, $getKey);
    }

    /**
     * Check if multiple $_GET-keys are not empty
     *
     * Example:
     * ```php
     * if(chkGetAll(array("a","b"))) instead of if(!empty($_GET["a"]) && !empty($_GET["b"]))
     * ```
     *
     * @param array $keys Get-keys.

Consider making the type for parameter $keys a bit more specific; maybe use array[].

     * @return bool
     */
    public static function chkGetAll(...$keys)
    {
        // If first value is array, then create array from first value
        if ((array)$keys[0] === $keys[0]) {
            $keys = $keys[0];
        }

        foreach ($keys as $key) {
            $val = filter_input(INPUT_GET, $key);
            if (empty($val)) {
                return false;
            }
        }
        return true;
    }
}