PHP、MYSQLで検索を行った結果のデータの昇順、降順

search.php
1<form action="./index.php" method="post">
2
3<fieldset>
4<legend>名前</legend>
5 <select name="name">
6     <option value="">なし</option>
7     <option value="ビーグル">ビーグル</option>
8     <option value="柴犬">柴犬</option>
9 </select>
10</fieldset>
11<br>
12
13<fieldset>
14<legend>性別</legend>
15<select name="gender">
16    <option value="">なし</option>
17    <option value="男の子">男の子</option>
18    <option value="女の子">女の子</option>
19</select>
20</fieldset>
21<br>
22<br>
23<input type="submit" value="検索">
24
25</form>

index.php
1<?php
2require_once ('database.php');
3
4$name=$_POST['name'];   
5$gender=$_POST['gender'];
6$birthday=$_POST['birthday'];
7
8
9if(!empty($name && $gender)){
10  
11  $sql="SELECT * FROM dogs WHERE name=:name AND gender=:gender";
12
13}else if(!empty($name || $gender)){
14 
15  $sql="SELECT * FROM dogs WHERE name=:name OR gender=:gender";
16
17}else{
18  $sql="SELECT * FROM dogs";
19  
20  }
21
22  //★
23  if($birthday == 'desc'){
24    $sql.=' order by birthday desc ';
25  }else if($birthday == 'asc'){
26    $sql.=' order by birthday asc ';
27  }
28
29$stmt = dbc()->prepare($sql);
30
31$stmt->bindValue('name', $name, PDO::PARAM_STR);
32$stmt->bindValue('gender', $gender, PDO::PARAM_STR);
33
34$stmt->execute();
35
36$posts = $stmt->fetchAll();
37
38?>
39
40
41<!DOCTYPE html>
42<html lang="ja">
43<head>
44  <meta charset="UTF-8">
45  <meta name="viewport" content="width=device-width, initial-scale=1.0">
46  <title>Document</title>
47</head>
48<body>
49
50<form method="post" action="./index.php">
51
52 <select name="birthday">
53 <option  value="desc">生年月日の遅い順</option>
54 <option value="asc">生年月日の早い順</option>
55 </select>
56<input type="submit" value="検索">
57
58</form>
59
60<br>
61<br>
62 <?php
63  foreach($posts as $post){
64    
65    echo $post['name'];
66    echo '<br>';
67    echo $post['gender'];
68    echo '<br>';
69    echo $post['birthday'];
70    echo '<br>';
71    echo '<br>';
72  
73    }
74?>
75<a href="./search.php">戻る</a>
76</body>
77</html>

ビーグルで検索した結果のデータを生年月日の早い、遅い順に並べたいです。そのために、index.php内の★の部分を加えました。
ですが、ビーグルだけで検索しているのに全体のデータが出てきてしまいます。　名前がビーグルのデータだけを昇順、降順にするやりかたを教えてください。　また、それぞれ検索したときに、エラーが出てきてしまうのですが、この直し方も教えていただきたいです。
因みに4,5,6行目は、
$name=$_POST['name'];
$gender=$_POST['gender'];
$birthday=$_POST['birthday'];
の部分です。よろしくお願いします。

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

回答2件

nameやgenderは任意の値のなのでprepareで処理
ソートカラムのしてはホワイトリスト方式でチェック
昇順・降順は別項目にしてもいいと思います

投稿2020/12/10 04:40

編集2020/12/10 10:04

yambejp

総合スコア114833

まず，大前提としてsubmitボタンを押下するとページはリロードされ定義していた変数は一度全て開放(削除)されてしまいます。そこでSESSION変数の使用をお勧めします。SESSION変数であれば，unsetしない限り基本的には保存されます。

検索時に出てくるNoticeエラーについては，if文でPOSTデータが存在する時のみ各変数へ代入するという処理で回避できます。
$name,$gender,$birthdayといった変数系は以下に示したSESSION変数で置き換えると良いと思います。

PHP
1
2$name=$_POST['name'];   
3$gender=$_POST['gender'];
4$birthday=$_POST['birthday'];

上記部分をこのようにしてみてはいかがでしょうか。

PHP
1if(!isset($_SESSEION['name'])):
2header("Location:前のページのパス名をここへ入力");
3endif;
4if(isset($_POST['name'])&&isset($_POST['gender'])):
5$_SESSION['name']=$_POST['name'];   
6$_SESSION['gender']=$_POST['gender'];
7endif;
8if(isset($_POST['birthday'])):
9$_SESSION['birthday']=$_POST['birthday'];
10endif;