2011-12-27 190 views
2

嗨,大家好我一直在从SQL表1中提取行1.我两年前写了这段代码,并意识到它是多么的可怕无效。我想通过输入一个简单的for-loop到自动化来加速编码。使用for循环加快PHP代码输入

$maxRows_dd1 = 10; 
$pageNum_dd1 = 0; 
if (isset($_GET['pageNum_dd1'])) { 
    $pageNum_dd1 = $_GET['pageNum_dd1']; 
} 
$startRow_dd1 = $pageNum_dd1 * $maxRows_dd1; 

$maxRows_dd2 = 10; 
$pageNum_dd2 = 1; 
if (isset($_GET['pageNum_dd2'])) { 
    $pageNum_dd2 = $_GET['pageNum_dd2']; 
} 
$startRow_dd2 = $pageNum_dd2 * $maxRows_dd2; 

$maxRows_dd3 = 10; 
$pageNum_dd3 = 2; 
if (isset($_GET['pageNum_dd3'])) { 
    $pageNum_dd3 = $_GET['pageNum_dd3']; 
} 
$startRow_dd3 = $pageNum_dd3 * $maxRows_dd3; 

... dd4到dd99介于两者之间!

$maxRows_dd100 = 10; 
$pageNum_dd100 = 99; 
if (isset($_GET['pageNum_dd99'])) { 
    $pageNum_dd32 = $_GET['pageNum_dd99']; 
} 
$startRow_dd99 = $pageNum_dd99 * $maxRows_dd99; 

对应于:

mysql_select_db($database_rent, $rent); 
$query_dd1 = "SELECT * FROM rent"; 
$query_limit_dd1 = sprintf("%s LIMIT %d, %d", $query_dd1, $startRow_dd1, $maxRows_dd1); 
$dd1 = mysql_query($query_limit_dd1, $rent) or die(mysql_error()); 
$row_dd1 = mysql_fetch_assoc($dd1); 

if (isset($_GET['totalRows_dd1'])) { 
    $totalRows_dd1 = $_GET['totalRows_dd1']; 
} else { 
    $all_dd1 = mysql_query($query_dd1); 
    $totalRows_dd1 = mysql_num_rows($all_dd1); 
} 
$totalPages_dd1 = ceil($totalRows_dd1/$maxRows_dd1)-1; 

mysql_select_db($database_rent, $rent); 
$query_dd2 = "SELECT * FROM rent"; 
$query_limit_dd2 = sprintf("%s LIMIT %d, %d", $query_dd2, $startRow_dd2, $maxRows_dd2); 
$dd2 = mysql_query($query_limit_dd2, $rent) or die(mysql_error()); 
$row_dd2 = mysql_fetch_assoc($dd2); 

if (isset($_GET['totalRows_dd2'])) { 
    $totalRows_dd2 = $_GET['totalRows_dd2']; 
} else { 
    $all_dd2 = mysql_query($query_dd2); 
    $totalRows_dd2 = mysql_num_rows($all_dd2); 
} 
$totalPages_dd2 = ceil($totalRows_dd2/$maxRows_dd2)-1; 

mysql_select_db($database_rent, $rent); 
$query_dd3 = "SELECT * FROM rent"; 
$query_limit_dd3 = sprintf("%s LIMIT %d, %d", $query_dd3, $startRow_dd3, $maxRows_dd3); 
$dd3 = mysql_query($query_limit_dd3, $rent) or die(mysql_error()); 
$row_dd3 = mysql_fetch_assoc($dd3); 

if (isset($_GET['totalRows_dd3'])) { 
    $totalRows_dd3 = $_GET['totalRows_dd3']; 
} else { 
    $all_dd3 = mysql_query($query_dd3); 
    $totalRows_dd3 = mysql_num_rows($all_dd3); 
} 
$totalPages_dd3 = ceil($totalRows_dd3/$maxRows_dd3)-1; 

...一路DD100!

我如何使用for循环来加速从dd1到dd100的每个代码块的所有代码?

+3

Nooo我的眼睛! ;) – PeeHaa 2011-12-27 17:44:54

+0

我的经理让我更新他的系统,我刚刚意识到我会浪费多少时间来尝试添加100个dd实例! – methuselah 2011-12-27 17:45:45

+0

您对从数据库中检索的这些数据实际上做了什么?在页面或其他东西上显示它?为什么你需要这些单独的结果?你不能只查询一次数据库吗? – DaveRandom 2011-12-27 17:46:27

回答

1

这是一种更有效的代码,你做了什么上面做的正是:

<?php 

    // You only need to do these once as they are the same throughout 
    mysql_select_db($database_rent, $rent); 
    $maxRows = 10; 
    // This code gets the total number of rows in the database 
    $totalRowsAll = mysql_fetch_assoc(mysql_query("SELECT count(*) AS total FROM rent", $rent)); 
    $totalRowsAll = (int) $totalRowsAll['total']; 

?> 
<div class="tab_container"> 
    <div id="tab1" class="tab_content"> 
     <table width="100%" border="0" cellspacing="5" cellpadding="5" id="1"> 
<?php 

    for ($i = 0; $i < 100; $i++) { 

    // Calcluate value for this iteration and query database 
    $pageNum = (isset($_GET['pageNum_dd'.($i + 1)])) ? (int) $_GET['pageNum_dd'.($i + 1)] : $i; 
    $startRow = $pageNum * $maxRows; 
    $query = "SELECT * FROM rent LIMIT $startRow, $maxRows"; 
    $result = mysql_query($query, $rent) or die(mysql_error($rent)); 
    $totalRows = (isset($_GET['totalRows_dd1'])) ? (int) $_GET['totalRows_dd1'] : $totalRowsAll; 
    ${'totalPages_dd'.($i + 1)} = ceil($totalRows/$maxRows) - 1; 

    // Now print this row 
?> 
     <tr height="100px" align="center"> 
<?php 

    while ($row = mysql_fetch_assoc($query)) { 

?> 

      <td style="background-color: <?php echo $row['colour']; ?>;" onclick="window.location='pay.php?id=<?php echo $row['dNo']; ?>&user=<?php echo $username; ?>'" onmouseover="this.style.cursor='pointer'"> 
       <form action="pay.php?id=<?php echo $row['dNo']; ?>&user=<?php echo $username; ?>" method="post"> 
       <input type="hidden" id="<?php echo $row['dNo']; ?>"> 
       <input type="hidden" value="<?php echo $username; ?>"> 
       <button type="submit" class="link" id="t<?php echo $row['dNo']; ?>"> 
        <span><?php echo $row['dNo']; ?></span> 
       </button> 
       </form> 
      </td> 
<?php 

    } // End while 

?> 
     </tr> 
<?php 

    } // End for 

?> 
     </table> 
    </div> 
</div> 

... 然而

我相当肯定,这可以在一个单一的来概括查询以获得所需的所有结果,这将会更加高效并大大减少数据库负载。但由于每行使用了$_GET['pageNum_dd*']$_GET['totalPages_dd*']变量,所以我不能100%确定这一点,而且我不知道如何在不知道更多产生什么的情况下完成这些操作。您需要检查这些可以传递的参数是否真的必要/有用。实际上,它们可能是由于行长度不同而造成的,每行的单元数量不等 - 这可能不是您想要的。

对于变量$totalPages_dd*也是如此,这些变量在以下分配但从未在任何地方使用过。它们可能没有用处,分配它们可能毫无意义。

+0

嗨,感谢您的帮助 - 我有20个选项卡,即'

'每个都有10行5行 - 代码能够输出吗?下面是它应该是什么样子的图像(但是一直到951-1000) - http://i.imgur.com/JXM0d.jpg – methuselah 2011-12-28 15:55:14