Bypass Select statement when empty.

[cyprus]

How can I bypass the code below if my $fields and $where have not been executed yet? The first time the page appears its empty. Thanks

$username="root";
$password="";
$database="";
mysql_connect(localhost,$username);
$query = "SELECT $fields FROM ORDERS $where";
@mysql_select_db($database) or die( "Unable to select database");     
$result = mysql_query($query) or die('Problem with query: ' . $query . '<br>'. mysql_error());
$numrows=mysql_numrows($result);

[sford999]

You could use an if statement

[code]<?php

$username="root";
$password="";
$database="";
mysql_connect(localhost,$username);

if($fields == "" || $where == "")
{
// Show the page when nothing is defined for $fields and $where
}
else
{
$query = "SELECT $fields FROM ORDERS $where";
@mysql_select_db($database) or die( "Unable to select database");   
$result = mysql_query($query) or die('Problem with query: ' . $query . '
'. mysql_error());
$numrows=mysql_numrows($result);
}

?>[/code]

[cyprus]

Many many thanks, works a treat.

[JustinK101]

Just curious, what does the @ mean in front of the mysql_select_db() call?

[markbett]

surpresses sql errors

[JustinK101]

HUmm, so whats the point if he has the || die(mysql_error()) call. Inst it going to hit the or clause so whats the point of surpressing sql errors?

[markbett]

The mysql_select_db( ) function uses the @ to suppress the default error message the ordie that is listed is not well thought out as it returns what you surpressed....

[cyprus]

Thanks for the comments on @, I just picked up my colleages line of code. Should I remove @, as I don't want to suppress seing any errors. Thanks

[Jenk]

Only very few circumstances warrant the use of the error suppresor.

[cyprus]

Thanks, I will remove it. Regards