前端控制器将参数链接到应用逻辑的最佳方法？

Question

我正在使用Front Controller设计模式构建应用程序,并且只有一个页面index.php,所有用户请求都通过该页面作为参数传递(与常规设计中的不同页面/控制器相对).

如何将这些参数连接到应用程序逻辑？

例如,我有两种不同的行为:

index.php?action=userLogin&username=admin&password=qwerty //process user login

index.php?action=displayUsersTable //show registered users

目前我有一个array系统接受的所有操作(以及预期的参数),我将actionURL中的param与key此数组的比较进行比较,然后检查此操作所需的参数.

//1 = optional, 2=required
$systemActions = [
      "userLogin" => [
            "login" => 2,
            "password" => 2
                     ],
      "displayUsersTable" => []
                 ];

显然,随着系统的发展,这将成为一个怪物阵列.

是否有更好的方法将发送到前端控制器的参数绑定到系统操作？

Answer 1

由于代码是“固定的”（即不是从数据库驱动），因此不需要注入数组（以及它所需的所有处理/内存开销。所以是的，它可以改进。

但有很多选择，具体取决于项目的增长程度。

最简单

最简单的是简单的“if”语句或开关。为了简单起见，我会从这里开始。

更复杂

你说其他项目有不同的页面/控制器 - 但这是有原因的。当您要求改进时，特别是如果您期望项目发展到您正在寻求优化的程度，那么您确实应该考虑这些原因（并分成文件）。

另一方面，您可以将所有调用拆分为文件/类并自动加载文件/类。

这样您只执行您需要的代码（较小的文件大小），非常模块化并且易于协作。如果添加新操作，则无需修改索引或数组 - 只需修改正在处理的操作文件。

示例（从我目前正在使用这种方法进行的项目中得到极大简化）：

1) 创建一个“baseAction”基类，所有操作都将从其扩展。您可以添加常用功能，例如清理/预处理参数、日志记录、验证标头等。

abstract class baseAction {
    protected $aExpectedParams = [];
    protected $aParams = [];
    protected $validParams = true;

    function __construct() {
        foreach (self::$aExpectedParams as $name=>$aParam) {
            if (isset($_GET[$name]))
                if ($aParam['type'] == 'string') {
                    self::$aParams[$name] = $_GET[$name];
                } elseif ($aParam['type'] == 'int') {
                    self::$aParams[$name] = (int)$_GET[$name];
                }
            } elseif ($aParam['required']) {
                self::$validParams = false;                
            }
        }
    }

    // This is the called function
    abstract function execute();

}

2) 通过扩展基本 Action 来创建“action”类。将它们保存在单独的文件中（以便其他人可以在项目上进行协作而不会干扰）。

// put in 'actions/userLogin.php
class userLogin extends baseAction {
    protected $aExpectedParams = [
            'login' => ['type' => 'string', 'required' => true]
            'password' => ['type' => 'string', 'required' => true]  // NOTE: you should never actually pass password unencrypted through "get" as they'll get stuck in user logs!
                    ];

    public function execute() {
        // Do Whatever
    }
}

。

// put in 'actions/displayUsersTable.php
class displayUsersTable extends baseAction {
    public function execute() {
        // Do Whatever
    }
}

3) 创建一个自动加载器来提取这些单独的文件。

function myAutoloader($className) {
    if (file_exists(__DIR__ . '/actions/' . $className . '.php')) {
        require_once(__DIR__ . '/actions/' . $className . '.php');
    }
}
spl_autoload_register ('myAutoloader');

4）然后你的index.php就干净了

$action = $_GET['action'] ?? '';
if (strlen($action) > 0 && class_exists($action) && method_exists($action, 'execute')) {
    $oAction = new $action();
    $oAction->execute();
} else {
    // Oopsie
}

（最后一个片段的注释：“class_exists”触发自动加载器。“method_exists”是检查某人是否没有请求公共 php 类，例如“object”；如果你更安全，你应该命名这个或添加额外的验证。这只是一个例子！）