[Resolved] appending variables to the WHERE part of a query

etoast

hi everyone
i have been fiddling with this query for ages but i'm going wrong somewhere. could someone cast a fresh pair of slightly better trained eyes over it please and advise what i am doing wrong. i am trying to run a query where if people select "Any" for a dropdown it is left out of the WHERE part of the query:

$result = "SELECT * FROM table WHERE price_id='".$price_id."'"; 
if ($location !== Any){ 
$result .= " AND location='".$location."'"; 
} 
if ($type !== Any){ 
$result .= " AND type='".$type."'"; 
}

while($row = mysql_fetch_array($result)) { 
...etc...

thank you!

nemonoman

if ( $type <> 'Any' )

put quotes around string 'Any'

etoast

thanks but that didn't change anything. the page prints without an error but it is blank. i can echo the $result before the WHERE and the mysql query shows correctly. but if i put it within the

while($row = mysql_fetch_array($result)) {

... it doesn't show anything. which makes me think that this bit isn't working. but i don't see why.

the complicated version that does work and that i am trying to simplify has a query that looks like this:

$result = mysql_query("SELECT * FROM table WHERE price_id='".$price_id."'") or die (mysql_error());

do i need that mysql_query part in front of the SELECT and if so how do i add it to the appending version in my post above?

thanks

nemonoman

sorry, I didn't realize what I was dealing with:

$query= "SELECT * FROM table WHERE price_id='$priceid'";
if ($location <>'Any'){
$query.= " AND location='$location'";
}
if ($type <>'Any'){
$query.= " AND type='$type'";
}
$result=mysql_query($query);
while($row = mysql_fetch_array($result)) {

etc...
}

PS, you'll have to connect to the database before you run the query.

PPS: you'll have to have a database before you connect.

PPPS. You'll have to turn on your computer too.

etoast

I NEED A COMPUTER!!?

As in you didn't realise you were dealing with a simpleton? 🙂
I knew about the db connection etc, I just didn't want to bore you with the parts of the code I was ok with.

The good news is you are indeed a genius the query works and I thank you for your wisdom.