Biscolab\UpDown\Http\UpDownRequest

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	addParam()	0	6	1
A	getQuery()	0	10	2
A	__construct()	0	6	3

<?php
/**
 * Copyright (c) 2019 - present
 * updown - UpDownRequest.php
 * author: Roberto Belotti - roby.belotti@gmail.com
 * web : robertobelotti.com, github.com/biscolab
 * Initial version created on: 15/2/2019
 * MIT license: https://github.com/biscolab/updown-php/blob/master/LICENSE
 */

namespace Biscolab\UpDown\Http;

use Biscolab\UpDown\Abstracts\AbstractObject;

/**
 * Class UpDownRequest
 * @package Biscolab\UpDown\Http
 */
class UpDownRequest
{

    /**
     * @var array
     */
    private $params = [];

    /**
     * UpDownRequest constructor.
     *
     * @param array $params
     */
    public function __construct(array $params = [])
    {

        if ($params) {

The expression $params of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            foreach ($params as $param_name => $param_value) {
                $this->addParam($param_name, $param_value);
            }
        }
    }

    /**
     * @param string         $param_name
     * @param AbstractObject $param_value
     *
     * @return UpDownRequest
     */
    public function addParam(string $param_name, $param_value): UpDownRequest
    {

        $this->params[$param_name] = $param_value;

        return $this;
    }

    /**
     * @return string
     */
    public function getQuery(): string
    {

        $params = [];

        foreach ($this->params as $param_name => $param_value) {
            $params[$param_name] = (string)$param_value;
        }

        return http_build_query($params);
    }

}