Hi there, hoping you can help me, I know it's a reasonably simple question and I probably should know the answer myself, I can't however seem to figure it out, I know I need an IF statement.

I need to basically tell the script to display - "No Results Found" in the instance that no results are indeed found...

Code as follows:

PHP Code:

<?
if ($songid == "") {
$sql="SELECT * FROM artists WHERE name LIKE '%$artist%' ORDER BY name ASC";
$result=mysql_query($sql);
$ncount=0;
while ($myrow=mysql_fetch_array($result)) {
if ($ncount<1)
{
$ncount=$ncount+1;
?>

<? if(isset($myrow['name'])) echo $myrow['name']; ?>

so basically if no artist is found in the directory, it displays the nothing found statement.. maybe I'm tired haha..

you must be tired, that's not even valid php code

basically I think you want something like this for your while loop

PHP Code:

while ( $myrow = mysql_fetch_array($result)){
$ncount++;// this counts how many rows we see
if(isset($myrow['name'])) echo $myrow['name'];
}
if(!$ncount) echo 'No Results Found'; 


you could also do it without actually counting the number of rows if you wanted (though there's not much difference).

PHP Code:

while ( $myrow = mysql_fetch_array($result)){
$ncount == true; //at least one row
if(isset($myrow['name'])) echo $myrow['name'];
}
if(!$ncount) echo 'No Results Found'; 


Oh yeah, you can use mysql_num_rows($result) as well.

just place that check BEFORE the while loop.

Use mysql_num_rows($result). No point having an isset() function call for every table row.

Originally Posted by stoot98

Use mysql_num_rows($result). No point having an isset() function call for every table row.

Yes there is.
An isset() call is way faster than an mysql_num_rows().
If performance is crucial and the site is slashdotted or something similar, it can make the difference between a timeout and a page.

it's not a matter of choosing between isset and mysql_num_rows though

isset doesn't really need to be there though. if there are any rows, and any of their names are "name" then it'll be there.

Hi Guys,
Thank's for your responses, really appreciate it!

I have the following code now which is operational, however, I was wondering if someone could let me know how I could loop the output so I could style it?

PHP Code:

<?

$result = mysql_query("select * from comments WHERE tag = '$headline' and approved = 'YES' order by Name ASC");
while ( $myrow = mysql_fetch_array($result)){
$ncount++;
if(isset($myrow['comment'])) echo $myrow['comment'];
if(isset($myrow['name'])) echo $myrow['name'];
if(isset($myrow['date'])) echo $myrow['date'];
}
if(!$ncount) echo 'No Results Found'; 
?>

Thanks guys,
Andy

Hi!
I got it, just used the following:

PHP Code:

<?

$result = mysql_query("select * from comments WHERE tag = '$headline' and approved = 'YES' order by Name ASC");
while ( $myrow = mysql_fetch_array($result))

{
$ncount++;
 $comment=$myrow["comment"];
 $name=$myrow["name"];
 $date=$myrow["date"];
echo "$name<BR>$date<BR>$comment<BR><BR>";

}
if(!$ncount) echo 'Noone has added a comment yet, will you be the first?'; 
?>

Is that the cleanest way?
Thanks,
Andy

Originally Posted by tripy

Yes there is.
An isset() call is way faster than an mysql_num_rows().
If performance is crucial and the site is slashdotted or something similar, it can make the difference between a timeout and a page.

Sure it is faster if youre calling them once each. I doubt itll be a lot faster when you are calling isset() 3000 times Vs 1 num_rows() with a big result set. Bottom line is there is no point introducing extra code branches when one will do.