Too many if() statements - this feels wrong

[jake66]

This sure looks like a lot of if() statements, but it does exactly what I want it to do.

Is this too much, or is there a way to condense it?

if (isset($HTTP_GET_VARS['category'])){
echo'You may also like <a href="' . tep_href_link(SIMILAR) . '">these</a>';
}else{
if (isset($HTTP_GET_VARS['manid']) && $themanquer['similar_country'] !==''){
if ($themanquer['similar_country'] =='CAN'){
$location='Canadian';
}elseif ($themanquer['similar_country'] =='FIN'){
$location='Finnish';
}elseif ($themanquer['similar_country'] =='GER'){
$location='German';
}elseif ($themanquer['similar_country'] =='NOR'){
$location='Norwegian';
}elseif ($themanquer['similar_country'] =='SWE'){
$location='Swedish';
}elseif ($themanquer['similar_country'] =='USA'){
$location='American';
}
echo'Can\'t find it? Try more <a href="' . tep_href_link('loc-'.$themanquer['similar_country']) . '-' . $themanquer['similar_type'].'">'.$location.' ' .$themanquer['similar_type'].'</a>, ';
echo'and <a href="/type-'.$themanquer['similar_type'].'">'.$themanquer['similar_type'],'</a><br />';
}else{
echo'Try other <a href="/type-'.$themanquer['similar_type'].'">'.$themanquer['similar_type'],'</a> stuff.<br/>';
}
}

[jatar_k]

you could try

switch() [php.net]

[adb64]

Or try this

Define an array somewhere at the top of your PHP file:

$Countries = array(
 'CAN' => 'Canadian', 
 'FIN' => 'Finnish', 
 'GER' => 'German', 
 'NOR' => 'Norwegian', 
 'SWE' => 'Swedish', 
 'USA' => 'American');

And where you now have all the if() statements:

if (isset($Countries[$themanquer['similar_country']]))
{
 $location = $Countries[$themanquer['similar_country']]; 
}