[Resolved] Error Message Won't Show

Randomizer

I run this code to check a mysql query failure but the error message never appears.

$sql_query = mysql_query("SELECT * FROM users WHERE (email = '$email')");

if (!$sql_query)
{
die("<font face='Comic Sans MS'>That e-mail dose not exist!</font>");
}

else { //send password return email }

Help appreciated, many thanx.

bubblenut

Well, if you have a table called users and a field in that table called email then it will never fail. I think what you may be looking for is to check if no results are returned. This is very different from a query failing. To check the number of rows in the result set use [man]mysql_num_rows()[/man]
HTH
Bubble

Randomizer

Dose that mean i would do something like this?

<?php
$x = mysql_num_rows($mysql_query);

if ($x = 0)
{
//error message
}

else 
{
//do this
}

To check if results are empty?

Weedpacket

Originally posted by Randomizer
if ($x = 0)
[/B]

if($x==0)

What if($x=0) does is set $x equal to zero, and then tests to see if the result is nonzero.

I often use a template resembling:

$query='SELECT blah...';
$query_result=sql_query($query);
$query_numrows = sql_num_rows($query_result);
if(!$query_numrows) // Convenient shorthand for $query_numrows==0
{   // Handle case of empty resultset
}
else
{   for($i=0; $i<$query_numrows; $i++)
    {   $row = sql_fetch_array($query_result);
        // Foo
    }
}

Eewwwww... <font> tags!

Frederick

I don't know why everyone does this simple code the long way.

$dblink represent database coonection

$sql = mysql_query("SELECT * FROM users WHERE email='$email' ",$dblink);
if (!mysql_num_rows($sql)) {
echo "No match was found\n";
} else {
$row = mysql_fetch_array($sql);
echo "".$row["email"]."\n";
}

Weedpacket

Originally posted by Frederick
I don't know why everyone does this simple code the long way.

Well, if you're sure you're not going to get more than one result (which I suppose is reasonable in this case), then apart from some intermediate variables which are there for the convenience of the programmer, yours is pretty much equivalent to mine (apart from the blight of using SELECT *).