if/elseif help

[michaelowen]

hello,

I am having a syntax problem with my if/elseif statement.

What i need is to display the variable depending on 3 circumstances, and i get the AND circumstance to work. What am i doing wrong?

---

if ($state)
{
$specs = "event_state = '".$state."'";
}

elseif ($city)
{
$specs = "event_city = '".$city."'";
};

elseif (($city) && ($state))
{
$specs = "event_city = '".$city."' AND event_state = '".$state."'";
};

---

the first two work, but not the third part.......help

thanks,

-Michael

[LordShryku]

Because they never get there. It computes to true before then. You may want to move the third check to the first position, then it will be checked first. You should also use [man]isset[/man] or [man] empty[/man] for those checks, otherwise php throws notices of undefined variables.

if(isset($city) && isset($state)) {
   $specs = "event_city = '".$city."' AND event_state = '".$state."'";
}
elseif(isset($state)) {
   $specs = "event_state = '".$state."'";
}
elseif(isset($city)) {
   $specs = "event_city = '".$city."'";
}

Switch may be an option too....

switch(TRUE) {
   case (isset($city) && isset($state)):
      $specs = "event_city = '".$city."' AND event_state = '".$state."'";
   break;
   case (isset($state)):
      $specs = "event_state = '".$state."'";
   break;
   case (isset($city)):
      $specs = "event_city = '".$city."'";
   break;
   default:
      // do nothing
   break;
}

[Drakla]

Get those semicolons away from those braces for a start, they're redundant and causing grief. You don't need the brackets in the last bit elseif ($city && $state ) would be fine.

But the last condition will never ever be met - if it didn't match $state in the first part then it won't match it in the last part, same with $city

[michaelowen]

Yes, that was my question. I have this running and it works fine, but it sends the EMPTY part as well to the statement. How do i get it to send JUST the variable chosen. Something like this?

---

if(isset($state) && isset($city) == "") {
$specs = "event_state = '".$state."'";
}
elseif(isset($city) && isset($state) == "") {
$specs = "event_city = '".$city."'";
}
elseif(isset($city) && isset($state)) {
$specs = "event_city = '".$city."' AND event_state = '".$state."'";
}

$run_query = "SELECT * FROM calendar where $specs";

for example, if state is the only part chosen, then it still sends city =''

thanks.

-michael