优化这段 PHP 代码？

Question

我目前正在开发一个包含许多数据源的混搭。为了在一页上显示用户想要的所有提要，我当前正在使用 if 语句与 MySQL 数据库进行交叉检查，如下所示：

if($var["type"]=="weather") 

$var 是调用 < code>mysqli_fetch_array

然后包括与下面的功能（例如天气）相关的代码，然后是另一个提要的另一个“if”语句，依此类推。问题是会有很多提要，并且拥有所有这些“if”语句将是缓慢且多余的。

有什么办法可以优化这个 PHP 代码吗？

Answer 1

另一种解决方案可能是使用关联数组将“类型”映射到自定义函数。

例如（伪代码）

function handle_wheater_logic() {
   // ... your code goes here
}

function handle_news_logic() {
  // .. your code goes here
}


$customFunctions = array("wheater" => "handle_wheater_logic", "news" => "handle_news_logic");

while ($row = mysql_fetch_...) {
    call_user_func ($customFunctions[$row["type"]])
}

这将消除使用大量 if 语句的需要。您也可以在配置文件中进行“类型到函数”映射，或者只存储自定义函数的名称以调用数据库表中的每个“类型” - 这取决于您。

当然，您也可以将参数传递给自定义函数。只需查看 call_user_func[_array] 的文档即可。

Answer 2

试试这个：

$methods = array(
    "weather" => function() {
        // code
    },

    "otheroption" => function() {
    }
);

只需使用 $var["type"] 作为数组中的索引来获取函数：

$methods[$var["type"]]();

显然，为了更好的可读性，您可以做类似的事情：

$methods = array(
    "weather" => "wheater_function",

    "otheroption" => "other_function"
);

然后以这种方式调用函数：

call_user_func($methods[$var["type"]]);

为了更加面向对象，我们显然可以在数组中存储实现特定接口的对象，或者存储重新定义 __call() 魔术方法的对象并像函数一样使用它。

Answer 3

您可以使用 switch 语句。

Answer 4

消除大量 if 语句和仅检查一个条件的大量 switch 语句的一个好解决方案是实现一种设计模式，例如策略模式。

这样您就可以将每种类型的代码分开，从而更容易概览和管理。

这是实现的示例 http://blogs.microsoft.co.il/blogs/gilf/archive/2009/11/22/applying-strategy-pattern-instead-of-using-switch-statements.aspx

即使您不会严格执行此操作，它也会为您提供一些关于如何优雅地解决此问题的想法。

Answer 5

创建一个将函数与每种提要类型关联起来的数组：

$actions = array("weather" => "getWeather",
                 "news"    => "getNews");

然后使用 call_user_func 调用正确的函数：

call_user_func($actions[$var["type"]]);

Answer 6

多态性的救援。

inteface FeedInterface {
  public function retrieve($params);
}

class FeedWeather implements FeedInterface {

  public function retrieve($params) {
    //retrieve logic for weather feed
  }

}

class FeedSports implements FeedInterface {

  public function retrieve($params) {
    //retrieve logic for sports feed
  }

}

通过使用 PHP 类自动加载，上述每个声明都可以位于单独的文件中，也可能具有命名空间。那么您的提要检索代码可能如下所示：

$class = 'Feed'.$var["type"];
$feed = new $class;

$feed->retrieve($params);

这过于简化，并且需要一些额外的代码来进行错误处理、发现不存在的类等，但这个想法应该很清晰。

Answer 7

使用 If 语句或 Switch 语句将比您关心的更快。它可能看起来很难看并且维护起来很麻烦，但它会很快。