webman框架源码修改及性能优化（二）

webman-framework/src/App.php几处性能优化（建议官方修改）

不是鸡蛋里挑骨头，webman本身就是出于对性能的追求，所以应要该精于求精，改进任何不太合理的地方。

1. guessControllerAction(...)这个方法，在foreach循环中用了array_merge，array_merge这个方法性能并不是很好，另一个问题是进行了两次foreach循环，实际只需要一次，看对比图。
   

修改后的代码

protected static function parseControllerAction(string $path)
 {
        // $path = str_replace('-', '', $path); //这个方法里注释掉这一行，为了路径更规范，比如some-other，如果不注释掉这行，就变成someother了
        //....
}

protected static function guessControllerAction($pathExplode, $action, $suffix, $classPrefix)
    {
        array_unshift($pathExplode, 'app');
        $map = [];
        $map2 = [];
        foreach ($pathExplode as $index => $section) {
            $tmp = $pathExplode;
            array_splice($tmp, $index + 1, 0, 'controller');
            $map2[] = trim("$classPrefix\\" . implode('\\', $tmp), '\\') . '\\Index';
            $controller = array_pop($tmp);
            if ($controller === 'controller') {
                continue;
            }
            $controller = static::kebab2camel($controller);
            $map[] = trim("$classPrefix\\" . implode('\\', $tmp), '\\') . '\\' . $controller;
        }

        $map = array_merge($map, $map2);

        foreach ($map as $controllerClass) {
            $controllerClass .= $suffix;
            if ($controllerAction = static::getControllerAction($controllerClass, $action)) {
                return $controllerAction;
            }
        }
        return false;
    }

    public static function kebab2camel($name)
    {
        $name = preg_replace_callback('/-([a-z]+)/', function($m) {
            return ucfirst($m[1]);
        }, $name);
        return ucfirst($name);
    }

2. 【强烈建议官方修改】getController(...)这个方法，两次进行scandir遍历目录和文件，webman可能是考虑了各种奇葩目录和文件命名方式，但规范就是规范，这个限定，规范后代码写起来更简洁，性能更好。
   

修改后的getController(...)：

protected static function getController(string $controllerClass)
    {
        if (class_exists($controllerClass)) {
            return (new ReflectionClass($controllerClass))->name;
        }

        $basePath = '';
        $file = '';
        if (0 === strpos($controllerClass, 'plugin\\')) {
            $basePath = BASE_PATH . '/plugin';
            $file = $basePath . str_replace('\\', '/', substr($controllerClass, 6)) . '.php';
        } else {
            $basePath = static::$appPath;
            $file = $basePath . str_replace('\\', '/', substr($controllerClass, 3)) . '.php';
        }

        if (is_file($file)) {
            require_once $file;
            if (class_exists($controllerClass, false)) {
                return (new ReflectionClass($controllerClass))->name;
            }
        }

        return false;
    }