0

有谁知道编写以下内容的更有效方法。我现在的加载时间超过一分钟。这是我在更长的代码集之后想出的。

$max = $i + 25;
while ($i < $max)
  {
    $memberno = $memberid;
    if($pulledid!=$memberid)
      {
        $pulledid = mysql_result($result99,$i,"id_member");
        $sql_statement = "Select vinyl_fortrade.ImageID,vinyl_fortrade.MemberID from vinyl_fortrade where exists (Select ImageID from vinyl_wanted WHERE MemberID = $memberno AND vinyl_wanted.ImageID = vinyl_fortrade.ImageID) AND vinyl_fortrade.MemberID=$pulledid";
        $result=mysql_query($sql_statement)or die(mysql_error());    
        $num=mysql_numrows($result);
        if($num>0)
          {
            $sql_statement2 = "Select vinyl_wanted.ImageID, vinyl_wanted.MemberID from vinyl_wanted where exists (Select ImageID from vinyl_fortrade WHERE MemberID = $memberno AND vinyl_wanted.ImageID = vinyl_fortrade.ImageID) AND vinyl_wanted.MemberID=$pulledid";
            $result2=mysql_query($sql_statement2)or die(mysql_error());                
            $num2=mysql_numrows($result2);
            $membername = mysql_result($result99,$i,"smf_members.member_name");
            $userid2 = mysql_result($result99,$i,"smf_members.id_member");
            if ($num2 > 0)
              {
                $lastlogged = mysql_result($result99,$i,"smf_members.last_login");
                echo '<td><a href="http://vinylmationconnection.com/forum/index.php?action=pm;sa=send;    u='.$userid.'"style="color: #FFF">'.$membername.'</a><br>Last Logged In:<br>';
                echo date('Y-m-d', $lastlogged);
                echo ':</td>';
                echo '<td>';
                $n = 0;
                $m = 0;
                while ($n < $num)
                  {
                    $title=mysql_result($result,$n,"vinyl_fortrade.ImageID");
                    $sql_statementj = "Select * from vinyl_figures WHERE ImageID = '$title'";
                    $resultj=mysql_query($sql_statementj)or die(mysql_error());  
                    $seriess=mysql_result($resultj,0,"vinyl_figures.Series");
                    echo '<img  src="http://vinylmationconnection.com/images/p'.$seriess.'/'.$title.'.jpg">';
                    $n++;
                    if($n%4=="0")
                      echo "<br>";
                  }
                echo '</td><td>';
                while ($m < $num2)
                  {
                    $title=mysql_result($result2,$m,"vinyl_wanted.ImageID");
                    $sql_statementj = "Select Series from vinyl_figures WHERE ImageID = '$title'";
                    $resultj=mysql_query($sql_statementj)or die(mysql_error());  
                    $seriess=mysql_result($resultj,0,"vinyl_figures.Series");
                    echo '<img src="http://vinylmationconnection.com/images/p'.$seriess.'/'.$title.'.jpg">';
                    $m++;
                    if($m%4=="0")
                      echo "<br>";
                  }                              
                echo '</td> </tr>';
              }
          }
      }
    $i++;
  }

请让我知道这些代码是否可以自我解释。

4

4 回答 4

2 
$sql_statementj = "Select * from vinyl_figures WHERE ImageID = '$title'";
$sql_statementj = "Select Series from vinyl_figures WHERE ImageID = '$title'";

在这两个语句中，如果我的代码正确，您似乎会对同一个文件发出 mysql 请求。在循环之外提出这些请求，这应该做一些工作。

于 2012-12-17T16:37:37.380 回答
0

编写一个存储过程（也许一个简单的查询/视图就足够了）来完成所有这些查询并观察数据库索引。

于 2012-12-17T16:36:49.547 回答
0

WHERE EXISTS 子句几乎可以肯定是罪魁祸首，应该对其进行重构。慢查询的一般准则...

  1. 使用 EXPLAIN SELECT 查看查询实际在做什么
  2. 为 WHERE、JOIN、ORDER、GROUP 中使用的所有列添加索引
  3. 避免 SELECT * 而是只选择您需要的列
  4. 将 LIMIT 添加到不需要完整表扫描的所有查询。
于 2012-12-17T16:38:17.053 回答
0

除了已经提到的答案，我建议您在将页面发送到浏览器之前使用 mod_deflate 压缩页面，并且应该会产生一些效果。如果您在本地环境中，也可能由于机器负载而导致执行缓慢。

而且我认为LIMIT $i, 25 在查询中使用可能会丢失第一个，我认为它会产生相同的效果。当您使用简单的 foreach 获取所有内容并完成时，我建议您在单个查询中尽可能多地推送，这会增加负载，但应该会减少加载时间。

我不是 100% 的专业人士，但我想提供帮助。

于 2012-12-17T16:46:59.600 回答