some update for params parse. add 'arrayValues' option

Author: inhere

examples/Controllers/HomeController.php

@@ -611,7 +611,7 @@ public function jsonCommand()
      * a example for use arguments on command
      * @usage home:useArg [arg1=val1 arg2=arg2] [options]
      * @example
-     *  home:useArg status=2 name=john arg0 -s=test --page=23 -d -rf --debug --test=false -a v1 --ab -c -g --cd val -h ''
+     *  home:useArg status=2 name=john arg0 -s=test --page=23 -d -rf --debug --test=false -a v1 --ab -c -g --cd val -h '' -i stat=online
      *  home:useArg status=2 name=john name=tom name=jack arg0 -s=test --page=23 --id=23 --id=154 --id=456  -d -rf --debug --test=false
      */
     public function useArgCommand()

examples/app

@@ -20,7 +20,7 @@ $app->setLogo("
                           /_/   /_/
 ", 'success');
 
-require __DIR__ . '/routes.php';
+require __DIR__ . '/commands.php';
 
 // run
 $app->run();

examples/routes.php renamed to examples/commands.php

@@ -41,13 +41,23 @@ final class CommandLine
      * --long-opt=<value>
      * @link http://php.net/manual/zh/function.getopt.php#83414
      * @param array $params
-     * @param array $noValues List of parameters without values(bool option keys) noVal
-     * @param bool $mergeOpts Whether merge short-opts and long-opts
+     * @param array $config
      * @return array
      */
-    public static function parseByArgv(array $params, array $noValues = [], $mergeOpts = false): array
+    public static function parseByArgv(array $params, array $config = []): array
     {
+        $config = array_merge([
+            // List of parameters without values(bool option keys)
+            'noValues' => [], // ['debug', 'h']
+            // Whether merge short-opts and long-opts
+            'mergeOpts' => false,
+            // list of params allow array.
+            'arrayValues' => [], // ['names', 'status']
+        ], $config);
+
         $args = $sOpts = $lOpts = [];
+        $noValues = array_flip((array)$config['noValues']);
+        $arrayValues = array_flip((array)$config['arrayValues']);
 
         // each() will deprecated at 7.2. so,there use current and next instead it.
         // while (list(,$p) = each($params)) {
@@ -79,7 +89,7 @@ public static function parseByArgv(array $params, array $noValues = [], $mergeOp
                 $nxt = current($params);
 
                 // next elem is value. fix: allow empty string ''
-                if ($val === true && self::nextIsValue($nxt) && !\in_array($opt, $noValues,true)) {
+                if ($val === true && self::nextIsValue($nxt) && !isset($noValues[$opt])) {
                     // list(,$val) = each($params);
                     $val = $nxt;
                     next($params);
@@ -93,10 +103,21 @@ public static function parseByArgv(array $params, array $noValues = [], $mergeOp
                     continue;
                 }
 
+                $val = self::filterBool($val);
+                $isArray = isset($arrayValues[$opt]);
+                
                 if ($isLong) {
-                    $lOpts[$opt] = self::filterBool($val);
+                    if ($isArray) {
+                        $lOpts[$opt][] = $val;
+                    } else {
+                        $lOpts[$opt] = $val;
+                    }
                 } else {
-                    $sOpts[$opt] = self::filterBool($val);
+                    if ($isArray) {
+                        $sOpts[$opt][] = $val;
+                    } else {
+                        $sOpts[$opt] = $val;
+                    }
                 }
 
                 // arguments: param doesn't belong to any option, define it is args
@@ -111,7 +132,7 @@ public static function parseByArgv(array $params, array $noValues = [], $mergeOp
             }
         }
 
-        if ($mergeOpts) {
+        if ($config['mergeOpts']) {
             return [$args, array_merge($sOpts, $lOpts)];
         }