if no result display "this" if result display "that"

Jamez

Hi Guys,

i have dynamic urls querying a Db to search for product reviews.

Most products dont have reviews yet, so i want to make the code echo "no reviews yet" if the result is blank...

I already figured out how to echo the results if there was any, here is the code:

<?
$d = ($_GET["d"]);

include 'admin/connect.php';

mysql_select_db("acne");
$query = "select * from prods where id = '$d'";

$result = mysql_query($query);
$num_results = mysql_num_rows($result);

while ($row = mysql_fetch_array($result))
{
  $id=$row["id"];

echo '<html><head>';
echo "<title>".($row["title"])." Reviews</title>";
$strType = $row["cat"];
$strFileName = str_replace(" ", "_", $strType);

include 'temps/head.php';

echo '<a href="products.php"><b>Products</b></a> <b>»</b> <a href="cat/'.$strFileName.'.html"><b>'.$strType.'</b></a> <b>»</b> '.($row["name"]);
}


mysql_select_db("acne");
$query = "select * from reviews where pid = '$d'";

$result = mysql_query($query);
$num_results = mysql_num_rows($result);


while ($row = mysql_fetch_array($result))
{
  $id=$row["id"];

echo '<div align="center"><table class="acn">';
echo '<tr><td>'.$row["rev"].'</td></tr>';
echo '<tr><td>Review by: '.$row["name"].' from '.$row[loc].'</td></tr>';
echo '<tr><td><a href="review.php?r='.($row["pid"]).'">Write Your Own Review</a></td></tr>';
echo "</table></div>";
}
include 'temps/foot.php'; 
?>

The second query from the table 'reviews' is the one i need to display the "if no results" message....

Anyone help? 🙂

Jamez

i tried adding a

if (num_results = 0) {echo 'no reviews';}

else
{code to display results}

I just changed the while () line to the else line.. however it didnt work.

Come on guys! Help me out pleeeeeeeeez 🙁

LarsLai

hi jamez!
i had just a short look at your script and i do not know if there is something else wrong,
but here you go with the if-statement:

if ($num_results == 0)

TheDefender

Originally posted by Jamez
Come on guys! Help me out pleeeeeeeeez 🙁

Just an FYI Jamez... This kind of stuff doesn't get you help any faster, and comes across whiney. The == LarsLai gave you should resolve your issue, as it is a comparison operator, and the single = is used to set a value.

You should also think about what time you originally posted... It's not a hotbed of adtivity at that time, so you have to be patient sometimes...

Jamez

Originally posted by TheDefender
Just an FYI Jamez... This kind of stuff doesn't get you help any faster, and comes across whiney.

You should also think about what time you originally posted... It's not a hotbed of adtivity at that time, so you have to be patient sometimes...

There were 6 or 7 other members viewing this forum when i posted and i was asking them.

I'm sorry it bothers you that I add a little character and emotion to my post rather than act like a robot 😉

I'm pretty sure the forum is here for helping people with questions and problems (like i had) so I don't see any reason why you need to add these remarks in.

I'm sorry if i have a sense of urgency when im in the middle of coding something and i get a little error i just cant seem to figure out and want the answer why its still fresh in my mind 😉

TheDefender

I'm pretty sure the forum is here for helping people with questions and problems (like i had) so I don't see any reason why you need to add these remarks in.

You are exactly right... The forum is here for that very reason... And as far as my interjection was concerned, it was merely an FYI so you could ascertain some sort of understanding as to why the responses didn't come as quick as you want them to. If you don't want to take constructive ettiquette criticism (which is just as much fair game as code critique and help), without getting personal, then I am sorry you feel that way, but it was only meant as an FYI. This way, you can have it in your head, the next time you seek help, that just because 6 or 7 others are online, it doesn't mean you are going to get your answer right away, because sometimes we have to take people's code, and put it into our own IDEs to find the less obvious problems, so the responses can be a bit slow... especially at that time of day. You just gotta be patient.