Qiniu\Http\RequestOptions

Complexity

Total Complexity

9

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
B	getCurlOpt()	0	25	8
A	__construct()	0	18	1

<?php

namespace Qiniu\Http;

use Qiniu\Http\Middleware\Middleware;

final class RequestOptions
{

    /**
     * @var int|null
     * http 请求的超时时间，单位：秒，默认：0，不超时
     */
    public $connection_timeout;

    /**
     * @var int|null
     * http 请求的超时时间，单位：毫秒，默认：0，不超时
     */
    public $connection_timeout_ms;

    /**
     * @var int|null
     * http 请求的超时时间，单位：秒，默认：0，不超时
     */
    public $timeout;


    /**
     * @var int|null
     * http 请求的超时时间，单位：毫秒，默认：0，不超时
     */
    public $timeout_ms;

    /**
     * @var string|null
     * 代理URL，默认：空
     */
    public $proxy;

    /**
     * @var int|null
     * 代理鉴权方式，默认：空
     */
    public $proxy_auth;

    /**
     * @var string|null
     * 代理鉴权参数，默认：空
     */
    public $proxy_user_password;

    /**
     * @var array<Middleware>
     */
    public $middlewares;

    public function __construct(
        $connection_timeout = null,
        $connection_timeout_ms = null,
        $timeout = null,
        $timeout_ms = null,
        $middlewares = array(),
        $proxy = null,
        $proxy_auth = null,
        $proxy_user_password = null
    ) {
        $this->connection_timeout = $connection_timeout;
        $this->connection_timeout_ms = $connection_timeout_ms;
        $this->timeout = $timeout;
        $this->timeout_ms = $timeout_ms;
        $this->proxy = $proxy;
        $this->proxy_auth = $proxy_auth;
        $this->proxy_user_password = $proxy_user_password;
        $this->middlewares = $middlewares;
    }

    public function getCurlOpt()
    {
        $result = array();
        if ($this->connection_timeout != null) {

It seems like you are loosely comparing $this->connection_timeout of type integer|null against null; this is ambiguous if the integer can be zero. Consider using a strict comparison !== instead.

            $result[CURLOPT_CONNECTTIMEOUT] = $this->connection_timeout;
        }
        if ($this->connection_timeout_ms != null) {

It seems like you are loosely comparing $this->connection_timeout_ms of type integer|null against null; this is ambiguous if the integer can be zero. Consider using a strict comparison !== instead.

            $result[CURLOPT_CONNECTTIMEOUT_MS] = $this->connection_timeout_ms;
        }
        if ($this->timeout != null) {

It seems like you are loosely comparing $this->timeout of type integer|null against null; this is ambiguous if the integer can be zero. Consider using a strict comparison !== instead.

            $result[CURLOPT_TIMEOUT] = $this->timeout;
        }
        if ($this->timeout_ms != null) {

It seems like you are loosely comparing $this->timeout_ms of type integer|null against null; this is ambiguous if the integer can be zero. Consider using a strict comparison !== instead.

            $result[CURLOPT_TIMEOUT_MS] = $this->timeout_ms;
        }
        if ($this->proxy != null) {

It seems like you are loosely comparing $this->proxy of type null|string against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

            $result[CURLOPT_PROXY] = $this->proxy;
        }
        if ($this->proxy_auth != null) {

It seems like you are loosely comparing $this->proxy_auth of type integer|null against null; this is ambiguous if the integer can be zero. Consider using a strict comparison !== instead.

            $result[CURLOPT_PROXYAUTH] = $this->proxy_auth;
        }
        if ($this->proxy_user_password != null) {

It seems like you are loosely comparing $this->proxy_user_password of type null|string against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

            $result[CURLOPT_PROXYUSERPWD] = $this->proxy_user_password;
        }
        return $result;
    }
}