w3l\Holt45\Post

Complexity

Total Complexity

6

Size/Duplication

Total Lines	61
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	chkPost()	4	4	1
A	assignFromPost()	4	4	1
A	chkPostAll()	15	15	4

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

/**
 * Check/assign from superglobal $_POST
 */
trait Post

This class seems to be duplicated in your project.

{
    /**
     * Check $_POST
     *
     * Example:
     * ```php
     * if(chkPost("s") == "a") instead of if(isset($_POST["s"]) && $_POST["s"] == "a")
     * ```
     *
     * @param string $postKey Post-key.
     * @return bool|string
     */
    public static function chkPost($postKey)
    {
        return filter_input(INPUT_POST, $postKey);
    }

    /**
     * Assign value from $_POST
     *
     * Example:
     * ```php
     * $var = assignFromPost("a") instead of $var = ((!empty($_POST["s"])) ? $_POST["s"] : "");
     * ```
     *
     * @param string $postKey Post-key.
     * @return string
     */
    public static function assignFromPost($postKey)
    {
        return (string)filter_input(INPUT_POST, $postKey);
    }

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

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

     * @return bool
     */
    public static function chkPostAll(...$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_POST, $key);
            if (empty($val)) {
                return false;
            }
        }
        return true;
    }
}