selectedが効かなくなった

Question

下記記述でselected2000で表示されるのですが

$options = "";
	for($i = 1950;$i <= $now;$i++){
		$initial = $i == '2000' ? ' selected' : '';
		$options .= "<option value=".$i.$initial.">".$i."年"."</option>";
	}

下記記述だとselectedが効いていないのか1950からの表示になってしまいます。

$options = "";
	for($i = 1950;$i <= $now;$i++){
		$initial = $i == '2000' ? ' selected' : '';
		$options .= "<option value=".$i.$initial.'年'.">".$i."年"."</option>";
	}

んーどうすればよいでしょうか？

Answer 1

var_dump($options)してみましょう．そもそもHTMLの属性をくくっていない点で，元の実装も不適切ですが…

前者

<option value=1999>1999年</option>
<option value=2000 selected>2000年</option>
<option value=2001>2001年</option>

後者

<option value=1999年>1999年</option>
<option value=2000 selected年>2000年</option>
<option value=2001年>2001年</option>

なお，アプリーケーション設計としては「年」はvalueに含めないほうが普通だと思います．私ならこう書きます．

$options = '';
for ($i = 1950; $i <= $now; ++$i) {
    $selected = $i === 2000 ? ' selected' : '';
    $options .= sprintf('<option value="%1$s"%2$s>%1$s年</option>', $i, $selected);
}

Answer 2

ミス質問です。

$options = "";
    for($i = 1950;$i <= $now;$i++){
        $initial = $i == '2000' ? ' selected' : '';
        $options .= "<option value=".$i.'年'.$initial.">".$i."年"."</option>";
    }

でした...

Comments

[mpyw]

どちらにせよHTML属性を括らないのはやや不適切なので，ダブルクオートで括るようにしましょう．

[earnest_gay]

ありがとうございます！