このPHP記述を修正願います

質問

にわかプログラマーです。
PHP記述に詳しい方、ひとつ教えてください。

■現在、下記PHP記述があるとして、これをHTML出力すると

echo "<tr><td>日付：$date</td></tr><tr><td><select name=\"ayear\">";
for($i = 2002; $i <= $last_year; $i++){
echo "<option value=" . $i . (($i == $year) ? ' selected' : '') . ">" . $i . "</option>";
}

■こう出力されてしまいます。
<option value=2013 selected>2013</option>

これを
■下記のように出力できるようにしたい
<option value="2013" "selected">2013</option>

（HTML記述として""で閉じられていないのを解決したい）

上記PHPを修正したいのですが、どなたか記述を修正していただけないでしょうか。

不躾な質問で申し訳ないですがよろしくお願い致します。

回答No.3

agunuz · ベストアンサー

にわかだろうが何だろうが「プログラマ」だというなら、マニュアルを読みましょう。

http://www.php.net/manual/ja/language.types.string.php

echo '<option value="' . $i . '"' . (($i == $year) ? ' selected="selected"' : '') . '>' . $i . '</option>';

お礼

すいません。ちょっと時間がなくて皆様に甘えました。
ばっちりです！
本当にありがとうございました。

回答No.5

yambejp

効率は悪いですが、こんな風にした方が見やすいかもしれません

<?PHP
$last_year=date("Y",strtotime("-1 year"));
$year="2010";
for($i = 2002; $i <= $last_year; $i++){
$selected =($i == $year) ? ' selected="selected"' : '';
echo htmlspecialchars("<option value=\"{$i}\"{$selected}>{$i}</option>\n");
}
?>

おそらく$yearは外部からくる文字列だと思いますので
本来であれば数値と文字列の比較を厳密にする処理を入れた方がいいのですが
今回のケースであれば「$i == $year」で問題ないでしょう
（===で比較するとマッチしない）

回答No.4

noname#244856

細かい訂正ですが失礼します。

<?=$date?>

に関して、「?>」直後の改行コードは削除されてしまうので正しいインデントで表示したければ

<?=$date . "\n"?>

や

<?=$date . PHP_EOL?>

とする必要があります。

回答No.2

noname#244856

そもそもHTMLタグをechoでそのまま出力するのは見にくいので避けたほうがいいです。
制御構造に関する別構文とecho短縮構文を使うのが一般的です。
http://php.net/manual/ja/control-structures.alternative-syntax.php
http://php.net/manual/ja/function.echo.php

<tr>
__<td>
____日付:<?=$date?>
__</td>
</tr>
<tr>
__<td>
____<select name="ayear">
<?php for ($i = 2002, $this_year = idate('Y'); $i < $this_year; $i++): ?>
<?php $selected = $i === $year ? ' selected' : ''; ?>
______<option value="<?=$i?>"<?=$selected?>><?=$i?></option>
<?php endfor; ?>
____</select>
__</td>
</tr>

ちなみにHTMLでは「selected」で正しい記述です。
XHTMLの場合は「selected="selected"」とする必要があります。

回答No.1

play_with_you

文字列中ではエスケープすればいいので
"<option value=\"" . $i . …
とするか、シングルクォートを使って
'<option value="' . $i . …
とするか。
ちなみに、「"selected"」じゃなくて「selected="selected"」です。