foreachがうまく動作しない...？

Question

沖縄しか表示されないのですが、なぜでしょうか...

<?php
class Pref {
	static function getPrefs(){
		$prefset = array('北海道','青森県','岩手県','宮城県','秋田県','山形県','福島県','茨城県','栃木県','群馬県','埼玉県','千葉県','東京都','神奈川県','新潟県','富山県','石川県','福井県','山梨県','長野県','岐阜県','静岡県','愛知県','三重県','滋賀県','京都府','大阪府','兵庫県','奈良県','和歌山県','鳥取県','島根県','岡山県','広島県','山口県','徳島県','香川県','愛媛県','高知県','福岡県','佐賀県','長崎県','熊本県','大分県','宮崎県','鹿児島県','沖縄県');
		foreach ($prefset as $prefs) {
			$initial = $prefs == '東京都' ? ' selected' : '';
			$options = "<option value=".$prefs.$initial.">".$prefs."</option>";
		}
		return $options;
	}
}
?>

<td>都道府県</td>
<td colspan="2">
	<select style="width: 173px" name="pref">
	<?php echo Pref::getPrefs(); ?>
	</select>
</td>

どこがおかしいのか見つけられないです。
method内でechoすると全部表示されます。
ただ、methodは処理だけしたいので、
呼び出し時にechoしたいなと思い上記の記述にしてみたら沖縄しか表示されないんです。

上記と下記はなにが違うのでしょうか？

↓下記は正常に表示される。

<?php
class Pref {
	static function getPrefs(){
		$prefset = array('北海道','青森県','岩手県','宮城県','秋田県','山形県','福島県','茨城県','栃木県','群馬県','埼玉県','千葉県','東京都','神奈川県','新潟県','富山県','石川県','福井県','山梨県','長野県','岐阜県','静岡県','愛知県','三重県','滋賀県','京都府','大阪府','兵庫県','奈良県','和歌山県','鳥取県','島根県','岡山県','広島県','山口県','徳島県','香川県','愛媛県','高知県','福岡県','佐賀県','長崎県','熊本県','大分県','宮崎県','鹿児島県','沖縄県');
		foreach ($prefset as $prefs) {
			$initial = $prefs == '東京都' ? ' selected' : '';
			echo "<option value=".$prefs.$initial.">".$prefs."</option>";
		}
	}
}
?>
<td>都道府県</td>
<td colspan="2">
	<select style="width: 173px" name="pref">
	<?php Pref::getPrefs(); ?>
	</select>
</td>

Answer 1

提示されたコードだと、$optionsに何度も再代入しているので、最後の沖縄だけが表示されているのだと思います。

<?php
class Pref {
    static function getPrefs(){
        $prefset = array('北海道','青森県','岩手県','宮城県','秋田県','山形県','福島県','茨城県','栃木県','群馬県','埼玉県','千葉県','東京都','神奈川県','新潟県','富山県','石川県','福井県','山梨県','長野県','岐阜県','静岡県','愛知県','三重県','滋賀県','京都府','大阪府','兵庫県','奈良県','和歌山県','鳥取県','島根県','岡山県','広島県','山口県','徳島県','香川県','愛媛県','高知県','福岡県','佐賀県','長崎県','熊本県','大分県','宮崎県','鹿児島県','沖縄県');
        $options = '';
        foreach ($prefset as $prefs) {
            $initial = $prefs == '東京都' ? ' selected' : '';
            $options .= "<option value=".$prefs.$initial.">".$prefs."</option>";
        }
        return $options;
    }
}
?>

上記のコードはどうでしょうか。 .= 演算子を使って再代入せずに文字列を結合しています。

Comments

[earnest_gay]

単純なことに気づけず...

ありがとうございました！