PHPのEOMについて

質問

<?php
function h($a)
{
return htmlspecialchars($_SESSION["$a"], ENT_QUOTES, "UTF-8");
}
$uname = h("uname");
$email = h("email");
$message = h("message");

$to = "akiko.nishizawa@nifty.com";
$title = "（メールフォームより）";
$ext_header = "From: {$email}";
$body = <<<EOM
(Webサイトからのメール）

お名前：{$uname}
メールアドレス：{$email}
メッセージ：{$message}

EOM;

$rc = mb_send_mail($to, $title, $body, $ext_header);
if (!$rc)
exit;
else
$_SESSION = NULL;
?>
<p>メールを送信しました</p>
<table border = "1">
<tr>
<td>お名前</td>
<td width = "300"><?php echo $uname; ?></td>
</tr>
<tr>
<td>メールアドレス</td>
<td width = "300"><?php echo $email; ?></td>
</tr>
<tr>
<td>メッセージ</td>
<td width = "300"><?php echo nl2br($message); ?></td> 　//なぜかここでエラー
</tr>
</table>

シンタックスエラーが出るのは最後から３行目なのですが、
EOMの部分をはずすとシンタックスエラーがなくなります。
ググってもどこが悪いのか全然わかりませんでした。
回答よろしくおねがいします！！

回答No.2

agunuz · ベストアンサー

>//なぜかここでエラー

コピペして試しましたが、syntax errorにはなりません。また

>EOMの部分をはずすとシンタックスエラーがなくなります。

EOM; の行を削除すると（当然ですが）エラーになりました、

なので、提示されたソースでは判断出来ません。最低でもsyntax errorのあとのメッセージも転記してください（それでも「あり得そうなミス」を片っ端から試すしかありませんが・・・）。

（蛇足）
変数名のみを二重引用符で括るのは無駄です。変数名だけを書いてください。二重引用符で括るのは文字列リテラルと混在させるときだけです。

function h($a) {
return htmlspecialchars($_SESSION[$a], ENT_QUOTES, 'UTF-8');
}

また、画面表示はhtmlspecialcharsを通すべきですがメール本文に展開するときはhtmlspecialcharsしてはいけません。なので、

function h($a) {
return htmlspecialchars($a, ENT_QUOTES, 'UTF-8');
}

$uname = isset($_SESSION['name']) ? $_SESSION['name'] : '';

としておいて、

<td width = "300"><?php echo h($uname); ?></td>

などとすべきです。ソースを読んだときに「ブラウザに返すときにエスケープしている」かどうかが一目瞭然でミスが減らせます。

お礼

蛇足がとってもためになりました！
回答が遅くなってごめんなさい！
回答ありがとうございました！

回答No.1

noname#244856

コピペしましたがシンタックスエラーは発生しませんでした。
http://ideone.com/fDpfry

お礼

お礼遅れてすみません！
結局僕のパソコンでやるとなぜかシンタックスエラーになってしまうようです。
回答してくれてありがとうございました！！