Question

我被要求写一个小PHP脚本，从一些下拉框中获取一些POSTed输入，这些输入提供了一些可选标准，最后，吐出一个或多个包含唯一代码的字符串变量。

变量 names 的形式为$ thingPlaceType，每个变量都是唯一的。下拉框允许选择：

任何一个＆＃34;事情＆＃34;或所有＆＃34;事物＆＃34;一起
任何一个＆＃34;地点＆＃34;或者所有＆＃34;地方＆＃34;一起
任何一个＆＃34;类型＆＃34;或所有＆＃34;类型＆＃34;一起

我无法在不使用嵌套切换语句的情况下弄清楚如何选择这些代码

switch($_POST['thing'])
{
  case "thing1":
     switch($_POST['place'])
     {
       case "place1":
          switch($_POST['type'])
          {
            case "type1":
               $output = $thing1Place1Type1;
            case "type2":
               $output = $thing1Place1Type2;
            case "alltypes":
               $output = $thing1Place1Type1.$thing1Place1Type2.$thing1PlaceType3;
           }
        case "place2":
        ...
        case "allplaces":
        ...
      }
  case "thing2":
     switch($_POST['place'])
     {
       case "place1":
          switch($_POST['type'])
          {
            case "type1":
               $output = $thing1Place1Type1;
            ...
      ...
  ...
}

似乎代码变成了Arrow Anti-Pattern。我认为我可以使用多维数组做一些事情，或者可能是一个单独的数组，我将值与键匹配。但我觉得抓着稻草，必定会有一些我不知道的东西。是时候将字符串转换为具有属性的正确对象吗？

Answer 1

您需要将代码重新分解为函数。例如： -

switch($_POST['thing'])
{
  case "thing1":
      $result = processThings($thing1);
      break;
  case "thing2":
      $result = processThings($thing2);
      break;
}

function processThings($thing)
{
    //processing code goes here
}

我相信你明白了。如果您愿意，可以在函数中使用更多的开关块，这样可以避免反模式并使代码更易于理解。

Answer 2

如果你想将它们转换为对象..你可以创建它。

  class Object {
        private $thing;
        private $place;
        private $type;

        public function __construct() {
            $this->thing = $_POST['thing'];
            $this->place = $_POST['place'];
            $this->type  = $_POST['type'];

            $this->processThing($this->thing, $this->place, $this->type);
        }

        public function processThing($thing = false, $place = false, $type = false) {
               //noW that you have all the properties you just need just process it
        }

    }

    if(isset($_POST['thing']) && isset($_POST['place']) && isset($_POST['type'])) {
        $object = new Object();
    }

Answer 3

好吧，如果您能找到一种方法来避免人们泄露您的网站 - 可能是通过在目标变量名称上使用前缀，那么您可以这样做：

$variableName = "A prefix_".$_POST['thing'].$_POST['type'].$_POST['place'];

$evaluatedVariable = $$variableName;

这些被称为“变量变量”。可能我会因使用它们而受到抨击，但如果你能负责任地使用它们，我发现它们在过去很有用。

当然，这不会直接适用于你的'alltypes'案例。您可以使用该建议重构函数

如何替换嵌套的switch语句

3 个答案: