我可以进一步优化这个 PHP 脚本以从 MySQL 数据库生成动态导航吗？

Question

我正在开发一个新的 CMS 以用于重复项目。 基本上，这段代码连接到服务器，获取所有表的名称并使用它们生成简单的导航。 它非常适合我的需要，但我只是想知道是否可以进一步优化此代码片段并使其更加简单。 也许制作一个可以自定义格式的类？ 等等。我试图使其尽可能“简单”。

我唯一想解释的是，它会检查表名称是否不“包含”，这是我的 CMS 使用的默认表，用于了解要在前端显示哪些数据。

   <?php

              echo '<div class="dynamic_nav_head">Navigation</div>';
              echo '<div class="dynamic_nav">';
                include('data.php');
                $tables = mysql_list_tables($database);
                  while (list($table) = mysql_fetch_row($tables)) { 
                            if($table!='includes'){
                      echo "<div class='cat'>".ucwords($table)."</div>";
                              echo "<div class='cat_item'>";
                                echo "<a href='?page=read&section=".$table."'>View " . ucwords($table) . "</a>";
                              echo "</div>";
                      echo "<div class='cat_item'>";
                                echo "<a href='?page=add&section=".$table."'>Add New ". ucwords($table) ."</a>";
                              echo "</div>";
                            }  // End If not in Includes.
                  } // End While
              echo '</div>';
 ?>

关于如何使这段代码更精简、更干净、更快速，有什么建议吗？ 提前致谢！

编辑： MySQL 版本：4.1.22

Answer 1

我建议您访问 http://refactormycode.com/

        echo '<div class="dynamic_nav_head">Navigation</div><div class="dynamic_nav">';  // on less echo
        include('data.php');
        $tables = mysql_list_tables($database);
          while (list($table) = mysql_fetch_row($tables)) {     
                    if($table!='includes'){
                          $ucTable= ucwords($table); // just one function call
                           //  just one echo;
                           // you where also using quotes and double quotes backwards

                           echo '<div class="cat">'.$ucTable.'</div><div class="cat_item"><a href="?page=read§ion='.$table.'">View ' .$ucTable.'</a></div><div class="cat_item"><a href="?page=add§ion='.$table.'">Add New '. $ucTable .'</a></div>';
                    }  // End If not in Includes.
          } // End While
      echo '</div>';

Answer 2

你怎么知道代码很慢？ 您的分析器对代码有何评价？ 哪个语句会减慢速度？ 你在什么平台？ mysql是什么版本的？ 该目录中有多少张表？ 您是否遭受过早优化的困扰？

Answer 3

代码还不错。 可读性的一项改进是从数据库构建一个数组，而不是在“while”构造中使用 mysql_fetch_row。 这还允许您在进入循环之前过滤掉不需要的名称。 您还可以将 ucwords 方法映射到数组上，从而将其从 while 循环构造中取出。 关于双引号的观点是有效的，但为了可读性，我会保留单独的 echo 语句，因为这实际上不会产生明显的差异。

希望有帮助。