formでselectを使って、選ばれて内容によってphpのif文を動かしたい

質問させていただきます。

現在phpの勉強中です。

勉強で飲食店をイメージしたページを作っています。

formでselectを使って、選ばれて内容によってphpのif文を動かしたいのですがエラーが返ってきます。
if文をなくして1つだけにす### ヘディングのテキストると動くのですが・・・

いろいろ試してみたのですが解決できないので皆さんのお知恵をお貸いただけないでしょうか。

下記がソースコードです。

<form id="menuInput" name="menuInput" method="post" action="menu-category.php">

<dl>
    <dt>
    <label for="kind">種類</label>
    </dt>
    <dd>
    <select name="kind">
    <option value="魚">魚</option>
    <option value="肉">肉</option>
    <option value="野菜">野菜</option>
    <option value="ドリンク">ドリンク</option>
    </select>
    </dd>
</dl>

<input type="submit" value="検索">

</form>

<?php
require('dbconnect.php');
?>


<div id="menu_list">

<table class="menu_list">
    <tr>
	<th>商品管理ID</ht>
	<th>カテゴリー</ht>
	<th>商品名</ht>
	<th>価格</ht>
	<th>備考</ht>
    </tr>

<?php

$category = $_POST['kind'];


if($category == "魚"){
$recodeSet = $mysqli->query('SELECT * FROM menu WHERE kind = "魚" ORDER BY id');
while($table = mysqli_fetch_assoc($recodeSet)){

}elseif($category == "肉"){
$recodeSet = $mysqli->query('SELECT * FROM menu WHERE kind = "肉" ORDER BY id');
while($table = mysqli_fetch_assoc($recodeSet)){

}elseif($category == "野菜"){
$recodeSet = $mysqli->query('SELECT * FROM menu WHERE kind = "野菜" ORDER BY id');
while($table = mysqli_fetch_assoc($recodeSet)){

}elseif($category == "ドリンク"){
$recodeSet = $mysqli->query('SELECT * FROM menu WHERE kind = "ドリンク" ORDER BY id');
while($table = mysqli_fetch_assoc($recodeSet)){

?>
<tr>
    <td><?php echo(htmlspecialchars($table['id'])); ?></td>
    <td><?php echo(htmlspecialchars($table['kind'])); ?></td>
    <td><?php echo(htmlspecialchars($table['name'])); ?></td>
    <td><?php echo(htmlspecialchars($table['price'])); ?></td>
    <td><?php echo(htmlspecialchars($table['remarks'])); ?></td>
</tr>

<?php

} else {
	echo "表示されることはないはず";
}

?>

</table>

退会済みユーザー

2018/06/28 02:18

シンタックスエラーか

Orlofsky

2018/06/28 02:19

表示されたエラーメッセージをそのまま載せましょう。エラーメッセージでググれば自力で解決できるかも？

m.ts10806

2018/06/28 02:39

PHP部分はインデントきちんとつけてください。全てベタで書いているとカッコの閉じ忘れなどに気づきにくくなります。　あと・・・こんなに沢山ifもwhileも書く必要なさそうなコードです。

m.ts10806

2018/06/28 02:40

「検索条件と検索結果」としたいのでしたら1画面におさめることもご検討ください。検索条件をかえたいときにいちいち画面を戻らなきゃいけません。

spcl

2018/06/29 09:15

asahina1979様シンタックスエラーでした。

spcl

2018/06/29 09:16

Orlofsky様　次回よりエラーメッセージも記載します。ご指摘ありがとうございます。

spcl

2018/06/29 09:17

mts10806様　もっと綺麗にコードを書くように意識します。あたお教えいただきました通り同じ画面に収まるよう工夫してみます。

Orlofsky

2018/06/29 09:21

今回もエラーメッセージを載せると同じ問題にぶつかった人がこの質問を見つけたら役に立つかも？

行動規範の内容に同意します

回答2件

ベストアンサー

whileの閉じタグが抜けているような気がします。。
入力チェックは省いていますがselectの値によってSQLを切り替えるのであれば以下だけでも行けそうですが。

PHP
1$category = $_POST['kind'];
2$recodeSet = $mysqli->query("SELECT * FROM menu WHERE kind = '".$category."' ORDER BY id");

投稿2018/06/28 02:25

編集2018/06/28 03:21

mayoi_maimai

総合スコア1583

spcl

2018/06/29 09:10

ご回答ありがとうございます。 mayoi_maimai様のおっしゃるとおりのソースコードでやりたかったことが実現できました！ sql文の中にも変数を使用できるのは知らなかったので勉強になりました！

行動規範の内容に同意します

この手の処理はprepareで

PHP
1$category = filter_input(INPUT_POST,'kind');
2
3$stmt = $mysqli->prepare('SELECT * FROM menu WHERE kind =? ORDER BY id');
4$stmt->bind_param('s', $category);
5$stmt->execute();