使用php进行有效的表单处理

Question

我正在处理某个项目，我想在多个模板中处理来自表单的请求。问题是如何在处理脚本中调用适当的函数。虽然，我已经编码了一段时间，但我仍然想不出比在隐藏字段中使用变量更好的方法：

if ($_POST['somehiddenfield'] == 1) {
   some_function_1();//doesnt matter if its a function or a method
}

if ($_POST['somehiddenfield'] == 2) {
   $mainclass->somemethod();
}
//goes on indefinitely

我还想将所有内容保存在一个处理程序文件中，在该文件中调用我的主类。那么有没有比使用if ... else更有效的方法呢？

Answer 1

我会做以下事情：

仍然有一个隐藏字段，但让它包含类似于表单名称的内容

<input type="hidden" name="formName" value="post">

然后你可以在消费 php 脚本中做类似的事情：

<?php
// whatever class you use... this is just a simple dummy
class FormsProcessor {
      public function post($params) {
          echo "processing post form";
      }
}

$formName = "post";  // would be $formName = filter_input(INPUT_POST, $_POST['formName'],FILTER_SANITIZE_STRING,FILTER_FLAG_STRIP_HIGH);
// BUT BE SURE TO SANITIZE THE INPUT!!!

$params = [];  // dummy
$formsProcessor = new FormsProcessor();
                 // here's the trick.
$formsProcessor->{$formName}($params);
// to be even safer you could check first if this method_exists() 
// and/or if it's in a list of allowed methods.

请注意，此类中不得有任何其他用户不应调用的方法。你可以通过真正组合两部分的方法名称来解决这个问题：

$methodName = $formName."Processor";
//....
$formsProcessor->{$methodName}();`

Answer 2

我会保留一个包含所有可能选项的硬编码 key=>value 数组。传递隐藏的输入字段，检查您的帖子值和硬编码选项的键之间是否有任何交集，并将任何匹配的值调用为函数。

$map = [
    'yourHiddenField' => 'myFunctionName',
    'anotherHiddenField' => 'myOtherFunctionName',
    'yourOtherHiddenField' => 'yetAnotherfunctionName',
];

$intersection = array_intersect(array_keys($map), array_keys($_POST));

foreach ($intersection as $key) {
    $this->{$map[$key])();
}

此代码未经测试。

编辑： 请注意允许运行任何输入而不预定义应允许运行的功能。

即使经过消毒也可能有多危险的示例：

class Test {
    public $i = 1;

    function __construct(){
        $this->i++;
    }
}

$formVariable = '__construct';

$t = new Test();

$t->{$formVariable}();

echo $t->i;