$rslt = mysql_query("SELECT * FROM comments WHERE user = 1");
$i = 1;
if ($row = mysql_fetch_assoc($rslt)) {
echo '<p>'. $i . ' '. $row['comment'] . '</p>';
$i++;
while ($row = mysql_fetch_assoc($rslt)){
echo '<div class="border"></div>';
echo '<p>'. $i . ' ' . $row['comment'] . '</p>';
$i++;
} // end while
} // end if
Avoids needing to know the number of rows.
Executes the if statement only once instead of each loop.
The HTML and php were kind of messy and inconsistent, so I just assumed the whole block was within php tags. Obviously, open and close the php tag as you see fit.
Bu büyük ölçüde bir tarz konudur, ama sık sık ve yaygın olarak yürütülecek sql deyimi dize tutmak için kullanılan değişken adı $ sql, biraz yanıltıcı olduğuna karar verdi, ben bu nedenle $ rslt için değişken adını değiştirdi.