Is there an easier way to do this?

rajend3

I have a PHP script that takes a variable and depending on the value will select the appropriate option in the select box so it is the default option when the page loads. Right now the code to do this looks like this:

if($code=="AB") {
     echo "<option value=\"AB\" selected=\"selected\">Alberta</option>";
}
else {
     echo "<option value=\"AB\">Alberta</option>";
}
if($code=="BC") {
     echo "<option value=\"BC\" selected=\"selected\">British Columbia</option>";
else {
     echo "<option value=\"BC\">British Columbia</option>";
}

...

The code continues like this for all provinces. Is there a better way to do this?

NogDog

$provinces = array(
   'AB' => 'Alberta',
   'BC' => 'British Columbia'
);
foreach($provinces as $key => $name)
{
   printf(
      "<option value='%s'%s>%s</option>\n",
      $key,
      ($code == $key) ? " selected='selected'" : '',
      $name
   );
}

jaql

$code = 'AB';

$province = array(
	'AB' => 'Alberta',
	'BC' => 'British Columbia',
	'MB' => 'Manitoba',
	'NB' => 'New Brunswick',
	'NL' => 'Newfoundland and Labrador',
	'NS' => 'Nova Scotia',
	'ON' => 'Ontario',
	'PE' => 'Prince Edward Island',
	'QC' => 'Quebec',
	'SK' => 'Saskatchewan',
);

foreach ($province as $abbr => $name) {
  if ($code == $abbr) {
    print "<option value='$abbr' selected='selected'>$name</option>\n";
  }
  else {
    print "<option value='$abbr'>$name</option>\n";
  }
}

jaql

NogDog;10930953 wrote:

$provinces = array(
   'AB' => 'Alberta',
   'BC' => 'British Columbia'
);[/QUOTE]

Beat me to it :P

And anyway, for your second parameter in your printf() statement, I think you meant:

[code=php]
($code == $key) ? " selected='selected'" : ''

NogDog

jaql;10930955 wrote:
Beat me to it 😛

And anyway, for your second parameter in your printf() statement, I think you meant:
($code == $key) ? " selected='selected'" : ''

You are correct, I changed variable names mid-stream...fixing now.

rajend3

Thanks for the help guys.