データベースからのsql記述及び実行と、その表示の仕方がわかりません。

Question

データベースから最新の予約IDを取得してから、その予約ID+1の値を新規予約のIDに設定する機能を実装中に以下のエラーメッセージが発生しました。

###発生している問題・エラーメッセージ
Parse error: syntax error, unexpected '' (T_ENCAPSED_AND_WHITESPACE), expecting identifier (T_STRING) or variable (T_VARIABLE) or number (T_NUM_STRING) in C:\xampp\htdocs\taiikukan\reserve_scr.php on line 31

###該当のソースコード
 
コード
``` function disp_form() {
    $pdo = new PDO('mysql:host=localhost;
    dbname=taiikukandb;charset=utf8','dbuser','abc123');
    $pdo -> setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
    $sql = "select * FROM yoyaku_tbl where reserveID(max)";
    $stmt = $pdo -> query($sql);
    while ($row = $stmt -> fetch(PDO::FETCH_ASSOC)){

    print <<< FRM1
    <form action="./reserve_scr.php" method="get">
    <table border=0">
    <tr>
       <td>予約番号</td>
       <td>
      ($row['reserveID']);


    </td></tr>

###試したこと
様々なサイトでデータベースからの情報取得及びその表示方法を調べていろいろ試しましたが、
うまく表示できていません。
###補足情報(言語/FW/ツール等のバージョンなど)
PHPで記述しており、コーディングはAtomで行っております。

Comments

[m.ts10806]

プログラムコードは```で囲ってください。（わからなければ質問編集画面でコード部分を選択し<code>ボタンを押してください）　また、若干質問テンプレート部分の文言が残っているようなので質問内容を精査してください。

Answer 1

エラーについて

ヒアドキュメント内における連想配列の展開の仕方ではないでしょうか？

($row['reserveID']);

↓

{$row['reserveID']}

ヒアドキュメントの閉じ部分も含めたソース全体をご提示いただいたわけではないので
他にもあるかもしれませんが・・ひとまず。

SQLについて

既に指摘があるかとは思います、おそらくそのSQLでは予約ID+1は取得できません。
SQLのエラーがでそうです。

select * FROM yoyaku_tbl where reserveID(max)

提示ソースのようにreserveID(max)と書いてしまうとreserveID()という関数にmaxという引数を渡していることになります。
「そんな関数ないよ（＆引数maxに対して「そんなカラムないよ」）」ってSQLのエラーが出るんじゃないでしょうか。

単に最大値+1をとりたいだけならselect max(reserveID)+1 FROM yoyaku_tblだけで良いはず。また、別名つけておくととりやすくなります。
例：select max(reserveID)+1 as new_reserve_id FROM yoyaku_tbl

$row = $stmt->fetch(PDO::FETCH_ASSOC); //SQLにより最大値+1の情報のみ1行1カラムのみが取得されている

print <<< FRM1
<form action="./reserve_scr.php" method="get">
<table border="0">
<tr>
<td>予約番号</td>
<td>
{$row["new_reserve_id"]}
FRM1;

あとyambejpさんのご指摘のようにhtmlエスケープをかけておくと良いでしょう。
※ご提示ソース内のhtmlに記述ミスがあったので勝手に修正してます。比較して探してみてください。

プログラミング言語からSQLを用いてDB問い合わせを行う際は、
実際にDBに対してツールなどで直接SQLを実行して、問題なく意図通りに動くかどうかを確認されたほうが良いと思います。
mysqlならphpmyadmin、postgreSQLならpgadminなど、ですね。

Comments

[m.ts10806]

;がついていたので()も何かの名残なのかなと思って勝手に削除してしまいましたが、文字列として"reserveID"を()で囲いたいのであれば残しておいてください。

[Tsukumo_Aoi]

ありがとうございました！
無事解決できました！
回答感謝いたします！

[m.ts10806]

SQL部分についても回答しました。ご確認ください。
※これでタイトル部分の疑問も含めて全て解決できるか分かりませんが、ご参考まで。

[m.ts10806]

ただ、reserveIDが整数型であることが前提のSQLなので、整数型ではない場合（英数混在など）はSQLで頑張ってとるのではなく、order でreserveID降順ソートにlimit 1とかつけて1件だけ取得して、PHP側で加工して最大値＋１とすると良いです。

[Tsukumo_Aoi]

整数型にしてあるので大丈夫かと思います。
親切丁寧にありがとうございました！

Answer 2

print <<< FRM1
<form action="./reserve_scr.php" method="get">
<table border=0">
<tr>
<td>予約番号</td>
<td>
({$row['reserveID']});

</td></tr>
```ですね。連想配列を埋め込むときは`{ }`で囲わないとおかしくなります。

Comments

[Tsukumo_Aoi]

ありがとうございます！
ひとまずこれについては解決できました！
回答感謝します！

Answer 3

where reserveID(max)

は、SQLの書式的にあやしいですが大丈夫でしょうか？

また、reserveIDのデータ型にもよりますが

while ($row = $stmt -> fetch(PDO::FETCH_ASSOC)){ 
$reserveID=htmlspecialchars($row['reserveID']);

print <<< FRM1
・・・・
{$reserveID}

・・・・
FRM1;
}

のような処理をするよう心がけた方が良いと思います

Comments

[Tsukumo_Aoi]

SQL間違ってました・・・(笑)
アドバイスありがとうございます！