Question

我正在研究一个项目，以进一步学习php以及它如何用于与mysql数据库进行交互。该项目是一个论坛，该页面显示一个类别中的所有主题。我想知道我是否有效地处理我的呼叫，如果没有，我如何构建我的查询以使它们更有效率？我知道它的一个小点，一个网站没有在测试之外使用，但我想尽早处理。

<?php
$cid = $_GET['cid'];
$tid = $_GET['tid'];

// starting breadcrumb stuff
$catname = mysql_query("SELECT cat_name FROM categories WHERE id = '".$cid."'");
$rcatname = mysql_fetch_array( $catname );
$topicname = mysql_query("SELECT topic_title FROM topics WHERE id = '".$tid."'");
$rtopicname = mysql_fetch_array( $topicname );
echo "<p style='padding-left:15px;'><a href='/'> Home </a> &raquo; <a href='index.php'> Categories </a> &raquo; <a href='categories.php?cid=".$cid."'> ".$rcatname['cat_name']."</a> &raquo; <a href='#'> ".$rtopicname['topic_title']. "</a></p>";
//end breadcrumb

$sql = "SELECT * FROM topics WHERE cat_id='".$cid."' AND id='".$tid."' LIMIT 1";
$res = mysql_query($sql) or die(mysql_error());
if (mysql_num_rows($res) == 1) {
    echo "<input type='submit' value='Reply' onClick=\"window.location = 'reply.php?cid=".$cid."&tid=".$tid."'\" />";
    echo "<table>";
    if ($_SESSION['user_id']) { echo "<thead><tr><th>Author</th><th>Topic &raquo; ".$rtopicname['topic_title']."</th></thead><hr />"; 
    } else { 
        echo "<tr><td colspan='2'><p>Please log in to add your reply.</p><hr /></td></tr>"; 
    }
    echo "<tbody>";
    while ($row = mysql_fetch_assoc($res)) {
        $sql2 = "SELECT * FROM posts WHERE cat_id='".$cid."' AND topic_id='".$tid."'";
        $res2 = mysql_query($sql2) or die(mysql_error());
        while ($row2 = mysql_fetch_assoc($res2)) {
            echo "<tr><td width='200' valign='top'>by ".$row2['post_creator']." <hr /> Posted on:<br />".$row2['post_date']."<hr /></td><td valign='top'>".$row2['post_content']."</td></tr>";
        }
        $old_views = $row['topic_views'];
        $new_views = $old_views + 1;
        $sql3 = "UPDATE topics SET topic_views='".$new_views."' WHERE cat_id='".$cid."' AND id='".$tid."' LIMIT 1";
        $res3 = mysql_query($sql3) or die(mysql_error());
        echo "</tbody></table>";
    }
} else {
    echo "<p>This topic does not exist.</p>";
  }
?>

谢谢你们！

Answer 1

看起来像一个经典的(n+1)查询错误可能会导致潜伏死亡。您使用一次往返获得一个密钥，然后循环结果以获得基于它的n个值。如果第一个结果集很大，那么您将进行大量的网络往返。

您可以通过JOIN一次性将其全部带回来，并节省大量的网络延迟。

Answer 2

这些陈述本身相当简单，所以你所知道的进一步优化它们并不多。但是，如果您创建一些业务对象并在一次调用中将数据缓存到它们中，然后从业务对象访问数据，那么它可能会更快。

换句话说，1000行的1次SQL调用比单行的1000次调用要快得多。

Answer 3

以下是我编写上述代码时要做的一些额外事情：

当您知道要使用的列时，切勿在{{1}}语句中使用*。
执行查询时始终使用SELECT。
一旦结果集达到其目的，取消设置结果集。
在查询中使用某些替换时，使用or die(mysql_error())来逃避注入。

这些数据库调用可以优化吗？

3 个答案: