functionで定義してるセレクトボックスにselectedがつけられない

Question

下記のようなクラスがあったとして、

class Date {
	static function getYear($yearget){
		$now = date("Y");
		$options = "";

		for($i = 1950;$i <= $now;$i++){
			if($yearget === $i){
			$selected = ' selected';
		}
			$options .= "<option value=".$i.$selected.">".$i."年"."</option>";
		}
		return $options;
	}
}

このように使っているのですが

<select class="year" name="year">
	<?php echo Date::getYear(filter_input(INPUT_POST, 'year')); ?>
</select>

どうも意図した表示がされません。

簡単なプロフィール登録ページなのですが、
内容修正するときにまたいちから入力したり選択したりする手間を省くための記述なのですが、セレクトボックスは1950から始まりますが1980を選択すると1980が残ったままの仕組みを作りたいのですが、どうもselectedが付かないのです。

どこがおかしいのか教えていただけると助かります。。。

Answer 1

$selectedが中途半端にしか与えられてないですね
こうしてみてください

    static function getYear($yearget){
      $now = date("Y");
      $options = "";
      for($i = 1950;$i <= $now;$i++){
        $selected =($yearget == $i)?' selected':'';
        $options .= "<option value='{$i}'{$selected}>{$i}年</option>";
      }
      return $options;
    }

Comments

[earnest_gay]

＝＝＝と＝＝に慣れておらず使いどころを間違っていました...

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

Answer 2

filter_input(INPUT_POST, 'year')で得られるのは文字列ですが、$iは数値です。
そして、===で比較すると、文字列と数値は一致しないはずです。
==で調べる、もしくは(int)filter_input(INPUT_POST, 'year')とintにキャストしてあげましょう。

それと、$selectedがifに当てはまらなかった時なにも定義していないため、elseで空文字にする等の対処をしたほうがよいでしょう。

Comments

[earnest_gay]

ありがとうございます。

＝＝＝と＝＝がなかなか慣れないです汗

最初は
$year = $yearget ＝== $i ? ' selected' : '';

にしてダメだったので
if($yearget === $i){
$selected = ' selected';
}
にしてみたんですが、falseの処理もしておらず違う型同士で比較していたためでした。

最初の
$year = $yearget ＝== $i ? ' selected' : '';
を
$year = $yearget ＝= $i ? ' selected' : '';
にするだけでよかったんですね...

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

Answer 3

<?php
// ファイルの先頭に書いてください
header('Content-type: text/plain');
?>

<select class="year" name="year">
    <?php echo Date::getYear(filter_input(INPUT_POST, 'year')); ?>
</select>

これで、意図した通りのHTMLが出力されているかどうかを確認しましょう。

どうもselectedが付かないのです。

ついていますね。ブラウザで右クリック「ソースの確認」で見てみれば？

<?php
if (filter_input_array(INPUT_POST)) {
    header('Content-type: text/plain');
}
class Date
{

	public static function getYear($yearget)
	{
		$now = date("Y");
		$options = "";

		for ($i = 1950; $i <= $now; $i++) {
			if ($yearget == $i) {
				$selected = ' selected';
			} else {
				$selected = '';
			}
			$options .= "<option value=\"" . $i . "\"" . $selected . ">" . $i . "年" . "</option>";
		}
		return $options;
	}

}

echo Date::getYear("2014");

Comments

[earnest_gay]

ありがとうございます。
まず、＝＝＝でキャッチできてませんでした。

[退会済みユーザー]

＝＝＝ じゃなくて === ね