巨型elseif链，巨型开关，还是带功能的小开关？

Question

我有一个 9000 行的 PHP 文件，其中包含大约 30 个离散区域，通过 $_POST 变量导航到。所以一个可能是……

elseif (isset($_POST['view_user']) 
     || isset($_POST['edit_user']) 
     || isset($_POST['process_user_status']))

等等。我可能有大约 75 个进入这 30 个区域的点，所有这些都由像上面这样的长 elseif isset 链处理。

我一直在考虑将其更改为更理智的东西。到目前为止我提出的想法：

1）将帖子归结为布尔值，并在 elseif 链中使用它。因此，上面的内容可以归结为 elseif ($area_user)，如果 $ 中的任何一个都将 $area_user 设置为 true上面的_POST已设置。但这并没有真正解决复杂性问题。

2）用用例代替elseif。所以上面的内容会变成……

case (isset($_POST['view_user'])):
case (isset($_POST['edit_user'])):
case (isset($_POST['process_user_status'])):
    do stuff;
    break;

但是，虽然它删除了 elseif 语法，但它只是用一些更易于管理的东西替换它，但可能仍然隐藏了真正的问题。

3）使用函数。因此，在页面顶部，我有一个类似的 switch 语句，但它不是在页面中间直接进入脚本区域，而是调用一个函数，因此它可能调用 UserArea($_POST['whatever']).这样做的优点是可以将所有 $_POST 变量移出脚本的核心部分，并将它们集中到导航和函数调用中。但是，它将需要大量全局函数声明，目前我不需要这样做，因为 elseif 的分支位于全局范围内。

4）使用完整的 MVC 拆分、模板等进行完全重构。很愿意，但目前不是一个选择。很高兴我将模型分开了，但视图和控制器现在必须共存。

当我写这篇文章时，我越来越说服自己相信 3，但我想看看你们的想法。对于这种情况，最佳导航实践应该是什么？

Answer 1

您还有一个选择：处理函数字典。

将所有潜在 $_POST 键（“view_user”等）放入指向处理它们的函数（名称）的关联数组中。然后，代替 ifelse 链，迭代实际 $_POST 键，直到在数组中找到匹配项并调用关联的函数。

现在，您可以轻松地将处理程序函数移动到其他文件中，尽管您可能需要向该动态函数调用添加一些参数。大多数 MVC 框架都使用这种调度（针对路径中的模式，而不是表单或查询字符串数据）。

// Untested, but something like this
$handlers = array(
  'view_user' => 'UserArea',
  'edit_user' => 'UserArea',
  'view_document' => 'DocManagement', // for example
  // etc...
);
foreach ($_POST as $key => $value) {
  if(array_key_exists($key, $handlers)){
    call_user_func($handlers[$key]);
  }
}