https://www.webmaster-forums.net/crss/node/1037794 en https://www.webmaster-forums.net/serverside-scripting/sql-and-php-problem#comment-1216441 <p>True, but I believe mysql_fetch_array moans if the result is broken due to invalid SQL, whereas mysql_num_rows just returns '0'. At least, I think so... I was advised to do it this way by a senior developer I know when we were working together on a project - it may just be that I've inherited a bad habit!</p> Wed, 14 Mar 2007 20:23:59 +0000 andy206uk comment 1216441 at https://www.webmaster-forums.net https://www.webmaster-forums.net/serverside-scripting/sql-and-php-problem#comment-1216440 <p>Doesn't the while loop mean that if there's zero results, the loop will get execute zero times?</p> Wed, 14 Mar 2007 19:50:32 +0000 timjpriebe comment 1216440 at https://www.webmaster-forums.net https://www.webmaster-forums.net/serverside-scripting/sql-and-php-problem#comment-1216432 <p>personally, I prefer to check if their is a result first, before trying to run through it:</p> <p><div class="codeblock"><code>if(mysql_num_rows($result) &gt; 0) {<br /><br />&nbsp;&nbsp;&nbsp; while($row = mysql_fetch_array($result)) {<br /><br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; //****whatever<br /><br />&nbsp;&nbsp; }<br /><br />}</code></div>'</p> Wed, 14 Mar 2007 18:56:44 +0000 andy206uk comment 1216432 at https://www.webmaster-forums.net https://www.webmaster-forums.net/serverside-scripting/sql-and-php-problem#comment-1216408 <p>You haven't actually executed the query. You've just put it into a string is all. For the second block of code, you'll need to do something more like this:</p> <p><div class="codeblock"><code>$result = mysql_query($sql);<br /><br />while($row = mysql_fetch_array( $result )) {</code></div>'</p> Wed, 14 Mar 2007 12:55:38 +0000 timjpriebe comment 1216408 at https://www.webmaster-forums.net