Argv::__construct() - Code Metrics - Inspection of "调整参数方法位置" - luolongfei/freenom - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( e5eb23...42f52b )

by luo

created 2020-03-12 12:53 UTC

Argv::__construct() A

↳ Parent: Argv

Complexity

Conditions	3
Paths	2

Size

Total Lines	16
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	14
c	0
b	0
f	0
dl	0
loc	16
rs	9.7998
cc	3
nc	2
nop	0

<?php
/**
 * 命令行参数
 *
 * @author mybsdc <[email protected]>
 * @date 2020/1/3
 * @time 16:32
 */

namespace Luolongfei\Lib;

class Argv
{
    /**
     * @var Argv
     */
    protected static $instance;

    /**
     * @var array 所有命令行传参
     */
    public $allArgvs = [];

    public function __construct()
    {
        if ($this->get('help') || $this->get('h')) {
            $desc = <<<FLL
Description
Params:
-c: <string> 指定要实例化的类名。默认调用FreeNom类
-m: <string> 指定要调用的方法名（不支持静态方法）。默认调用handle方法
-h: 显示说明

Example: 
$ php run -c=FreeNom -m=handle

FLL;
            echo $desc;
            exit(0);

        }
    }

    /**
     * @return Argv
     */
    public static function instance()
    {
        if (!self::$instance instanceof self) {

            self::$instance = new self();
        }

        return self::$instance;
    }

    /**
     * 获取命令行参数
     *
     * @param string $name
     * @param string $default
     *
     * @return mixed|string
     */
    public function get(string $name, string $default = '')
    {
        if (!$this->allArgvs) {

            $this->setAllArgvs();
        }

        return $this->allArgvs[$name] ?? $default;
    }

    /**
     * 设置命令行所有参数
     *
     * @return array
     */
    public function setAllArgvs()
    {
        global $argv;

        foreach ($argv as $a) { // Windows默认命令行无法正确传入使用引号括住的带空格参数，换个命令行终端就好，Linux不受影响
            if (preg_match('/^-{1,2}(?P<name>\w+)(?:=([\'"]|)(?P<val>[^\n\t\v\f\r\'"]+)\2)?$/i', $a, $m)) {
                $this->allArgvs[$m['name']] = $m['val'] ?? true;
            }
        }

        return $this->allArgvs;
    }
}

1			<?php
2			/**
3			* 命令行参数
4			*
5			* @author mybsdc <[email protected]>
6			* @date 2020/1/3
7			* @time 16:32
8			*/
9
10			namespace Luolongfei\Lib;
11
12			class Argv
13			{
14			/**
15			* @var Argv
16			*/
17			protected static $instance;
18
19			/**
20			* @var array 所有命令行传参
21			*/
22			public $allArgvs = [];
23
24			public function __construct()
25			{
26			if ($this->get('help') \|\| $this->get('h')) {
27			$desc = <<<FLL
28			Description
29			Params:
30			-c: <string> 指定要实例化的类名。默认调用FreeNom类
31			-m: <string> 指定要调用的方法名（不支持静态方法）。默认调用handle方法
32			-h: 显示说明
33
34			Example:
35			$ php run -c=FreeNom -m=handle
36
37			FLL;
38			echo $desc;
39			exit(0);
			0 ignored issues – show Best Practice introduced 2020-03-12 12:55 UTC by Report Bug Copy Issue Report Using `exit` here is not recommended. In general, usage of exit should be done with care and only when running in a scripting context like a CLI script. Loading history...
40			}
41			}
42
43			/**
44			* @return Argv
45			*/
46			public static function instance()
47			{
48			if (!self::$instance instanceof self) {
			0 ignored issues – show introduced 2020-02-01 11:31 UTC by Report Bug Copy Issue Report `self::instance` is always a sub-type of `self`. Loading history...
49			self::$instance = new self();
50			}
51
52			return self::$instance;
53			}
54
55			/**
56			* 获取命令行参数
57			*
58			* @param string $name
59			* @param string $default
60			*
61			* @return mixed\|string
62			*/
63			public function get(string $name, string $default = '')
64			{
65			if (!$this->allArgvs) {
			0 ignored issues – show Bug Best Practice introduced 2020-02-01 11:31 UTC by Report Bug Copy Issue Report The expression `$this->allArgvs` 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. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
66			$this->setAllArgvs();
67			}
68
69			return $this->allArgvs[$name] ?? $default;
70			}
71
72			/**
73			* 设置命令行所有参数
74			*
75			* @return array
76			*/
77			public function setAllArgvs()
78			{
79			global $argv;
80
81			foreach ($argv as $a) { // Windows默认命令行无法正确传入使用引号括住的带空格参数，换个命令行终端就好，Linux不受影响
82			if (preg_match('/^-{1,2}(?P<name>\w+)(?:=([\'"]\|)(?P<val>[^\n\t\v\f\r\'"]+)\2)?$/i', $a, $m)) {
83			$this->allArgvs[$m['name']] = $m['val'] ?? true;
84			}
85			}
86
87			return $this->allArgvs;
88			}
89			}

luolongfei / freenom

Push — master ( e5eb23...42f52b )

Argv::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like