PHP循环优化

Question

我有一个 php 方法，它使用从属性中检索的数据创建一个 HTML 表。 我最关心的是我的应用程序的性能，因为我要处理大量数据。

public function getHTML() {

    $phpObj =  json_decode($this->data); // array(object, object, object, ....);

    $table = "<table><tbody>\n";

    if (count($phpObj->query->results->row) > 0) {
        $row = $phpObj->query->results->row;

         foreach ($row as $value) {
            $table .= "<tr>\n";
            foreach ($value as $key => $val) { // concerned about loop inside loop
                $table .= "<td>" . $value->$key . "</td>"; 
            }
            $table .= "\n</tr>\n";
        }   

        $table .= "</tbody></table>";
        return $table;
    }
    else {
        return 'HTML table not created.';
    }       
}

有没有更有效的方式来遍历数组和对象而不在循环中创建循环？

Answer 1

不要连接并返回值，而是立即回显它。不太干净，但性能会更有趣，因为字符串会立即输出到管理更有效的输出缓冲区。

循环中的循环通常是遍历二维数组的最佳方式。

Answer 2

字符串连接是成本密集型的​​。您可以通过使用数组来减少重复字符串连接的数量：

public function getHTML() {
    $phpObj =  json_decode($this->data);
    if (count($phpObj->query->results->row) > 0) {
        $rows = array();
        foreach ($phpObj->query->results->row as $row) {
            $cells = array();
            $rows[] = "<td>" . implode("</td><td>", $row) . "</td>";
        }
        return "<table><tbody>\n<tr>\n" .
               implode("\n<tr>\n<tr>\n", $rows) .
               "\n</tr>\n</tbody></table>";
    } else {
        return 'HTML table not created.';
    }
}

您也可以使用anonymous functions（自 PHP 5.3 起可用）：

public function getHTML() {
    $phpObj =  json_decode($this->data);
    if (count($phpObj->query->results->row) > 0) {
        return "<table><tbody>\n<tr>\n" .
               implode("\n<tr>\n<tr>\n", array_map(function($cells) { return "<td>".implode("</td><td>", $cells)."</td>"; }, $phpObj->query->results->row)) .
               "\n</tr>\n</tbody></table>";
    } else {
        return 'HTML table not created.';
    }
}

Answer 3

更新 Shrapnel 上校正确地指出，奇怪的是，字符串连接实际上在 php 中相对较快。

正如文森特所说，不要运行一堆串联，那会杀了你。您有两个选项可以加快脚本速度：

1. 立即回显。
2. 将您的行存储在一个数组中，并在最后加入该数组。

两个例子：

<?php

$mylines = array();
foreach ($row as $value) {
    $mylines[] = "<tr>\n";
    foreach ($value as $key => $val) { // concerned about loop inside loop
        $mylines[] = "<td>" . $value->$key . "</td>"; 
    }
    $mylines[] = "\n</tr>\n";
}
return implode('', $mylines);

Answer 4

您可以将 HTML 构建移动到前端，并通过 AJAX 和 javascript 将 JSON 数据发送给用户。

这也可以让您一次只发送数据片段（取决于您的 html 布局），因此可以在需要时动态轮询它们（例如 google/bing 图像搜索）。

我知道我没有直接回答这个问题。那是因为你拥有的代码可能是最快的（在 PHP 中），没有做一些愚蠢的小优化，只会使代码更难阅读/维护（可能只会节省几个 %）。

编辑：再次查看后，我敢打赌您的代码实际上只是从 JSON 中的外部源轮询数据。您可以通过让前端 javascript 执行 HTTP 命中并处理那里的数据来完全删除此代码。这完全消除了运行此 PHP 代码的需要。

编辑 2：在阅读了您关于这是禁用 javascript 的后备的评论后，我查看了您当前正在执行的代码。它似乎可以翻译成implode。

//declared this functions somewhere
function tr_build( $row_value )
{
    $tablerow .= "<tr>\n";
    if( $row_value ) {
        $tablerow .= "<td>".implode( "</td><td>", $row_value )."</td>";
    }
    $tablerow .= "\n</tr>\n";
    return $tablerow;
}

//this replaces the double loop
if( $row ) {
    $tablerows = "<tr>\n".implode( "\n</tr>\n<tr>\n", array_map( "tr_build", $row ) )."\n</tr>\n"
} else {
    $tablerows = "";
}

Answer 5

你为什么要为此烦恼？

        $table .= "<tr>\n";
        foreach ($value as $key => $val) { // concerned about loop inside loop
            $table .= "<td>" . $value->$key . "</td>"; 
        }
        $table .= "\n</tr>\n";

你从来没有真正使用过$val，那为什么还要有这个循环呢？

        $table .= "<table><td>";
        $table .= implode("</td><td>", array_keys($value));
        $table .= "</td></table>";