検索結果の表示を上から最新順にしたい

前提・実現したいこと

phpから検索をしデータベースの情報を取得する機能を作っていますが、取得データをpost_idで新しいものを上に表示させる方法がわかりません。

＊データベース接続、その他の機能は正常に動作していますが、下記の部分に”order by post_id desc”を加えるとエラーが発生します。

よろしくお願いします。

$sql = 'select * from hosts where order by post_id desc ' . $whereSql ;

発生している問題・エラーメッセージ

Fatal error: Uncaught Error: Call to a member function fetch() on boolean in D:\xampp\htdocs\test\1.php:39 Stack trace: #0 D:\xampp\htdocs\test\1.php(66): getHostData(Array) #1 {main} thrown in D:\xampp\htdocs\test\1.php on line 39

39行目
while($row = $HostDataSet->fetch( PDO::FETCH_ASSOC )){

該当のソースコード

<?php
function getHostData($params){
	if(!isset($params['search'])){
		return[];
	}
	//DBの接続情報
	try {
$pdo = new PDO('mysql:host=localhost;dbname=mydb;charset=utf8','root','',
array(PDO::ATTR_EMULATE_PREPARES => false));
} catch (PDOException $e) {
 exit();
}

	//入力された検索条件からSQl文を生成
	$where = [];
	if(!empty($params['shop'])){
		$where[] = "shop like '%{$params['shop']}%'";
	}

	if($where){
		$whereSql = implode(' AND ', $where);
		$sql = 'select * from hosts where order by post_id desc ' . $whereSql ;
	}else{
		$sql = 'select * from hosts';
	}

	//SQL文を実行する
	$HostDataSet = $pdo->query($sql);
	//扱いやすい形に変える
	$result = [];
	while($row = $HostDataSet->fetch( PDO::FETCH_ASSOC )){
		$result[] = $row;
	}
	return $result;
}
?>
<!DOCTYPE HTML>
<html lang="ja">
<head>
<meta charset="utf-8" >
<meta name="viewport" content="width=device-width, initial-scale=1">
<title>トップページ</title>
</head>
<body>
<h1 class="col-xs-6 col-xs-offset-3">検索フォーム</h1>
<div class="col-xs-6 col-xs-offset-3 well">
	<form method="get">
		<div class="form-group">
			<label for="InputShop">店舗名</label>
			<input name="shop" class="form-control" id="InputShop" value="<?php echo isset($_GET['shop'])? htmlspecialchars($_GET['shop']) : '' ?>">
		</div>
		<button type="submit" class="btn btn-default" name="search">検索</button>
	</form>
</div>

<?php
if(!empty($_GET)){
$hostData = getHostData($_GET);?>
<div class="col-xs-6 col-xs-offset-3">
	<?php if(isset($hostData) && count($hostData)): ?>
		<p class="alert alert-success"><?php echo count($hostData) ?>件見つかりました。</p>
		<table class="table">
			<thead>
				<tr>
					<th>店舗名</th>
				</tr>
			</thead>
			<tbody>
				<?php foreach($hostData as $row): ?>
					<tr>
						<td><?php echo htmlspecialchars($row['shop']) ?></td>
					</tr>
				<?php endforeach; ?>
			</tbody>
		</table>
	<?php else: ?>
		<p class="alert alert-danger">検索対象は見つかりませんでした。</p>
	<?php endif; ?>
</div>
<?php } ?>
</body>
</html>

Orlofsky

2019/01/23 05:04

[MySQL]タグを追加してください。

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

回答2件

エラー自体とorder byは直接的には関係ありません。
失敗するSQLを作っているにも関わらずデータが取得可能な前提でfetchしているからです。

PDO::query#返り値

PDO::query() は、PDOStatement オブジェクトを返します。失敗した場合は FALSE を返します。

falseはfalseであり、fetchというメンバーは存在しません。

下記のようにして実際に実行される予定のSQLが正しいものであるか確認してください。

php
1if($where){
2    $whereSql = implode(' AND ', $where);
3    $sql = 'select * from hosts where order by post_id desc ' . $whereSql ;
4}else{
5    $sql = 'select * from hosts';
6}
7echo $sql;exit;

そしてそのSQLをプログラム通してではなく、直にDBにログインして実行してみてください。
プログラムで実行する前にそれが正しいQueryであるか確認は必須です。

逆に、「こういうSQLであれば欲しいデータが取得可能である」Queryを事前に作り、DBに対して直接SQLを投げて想定通りのデータがとれるかを確かめるのも必須です。

両者が合わさって初めてプログラムでの実行ができます。

何事もですが、DBのような外部の情報にアクセスする際に「必ず○○が返ってくる前提」の作りは非常によろしくありませんし、必ず破綻します。
車の運転と同じですね。「だろう運転」ではなく「かもしれない運転」。
「かもしれない」は最悪の事態（想定通りにいかない事態）を想定しておくことです。

つまり、今回のケースで言えば、fetch使いたいのであればそれより前にpdo::queryの返り値がfalseでないことを確認すべきです。
例え正しいQueryを組めていたとしても。

DB接続のところでtry-catchを導入していますが、これは何のためですか？
単なるおまじないとして（または何かのコピペで）とりあえず入れてませんか？

これは異常事態が起きることを「かもしれない運転」で意味があって導入されるものです。
これと同じことをQuery実行の際にする必要がありますし、徹底しなければいけません。

投稿2019/01/23 04:56

m.ts10806

総合スコア80842

ariiiiiga

2019/01/23 05:08

Notice: Undefined variable: where in D:\xampp\htdocs\test\2.php on line 15 select * from hosts １５行名 if($where){ エラーが出てしまいました。

m.ts10806

2019/01/23 05:12

私の回答に対するコメントになっていませんが・・。そのエラー自体は何も珍しいことはないので調べてください。「出てしまった」という割りにそのエラーが出たコードが提示されていません。

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

ベストアンサー

入れる場所が違いますね。。。
あと今の記述ですと条件が未指定の場合はソートが掛からないのでsqlの最後に追加すればよいと思います。

PHP
1    if($where){
2        $whereSql = implode(' AND ', $where);
3        $sql = 'select * from hosts where ' . $whereSql ;
4    }else{
5        $sql = 'select * from hosts';
6    }
7
8    $sql .= " order by post_id desc";

投稿2019/01/23 03:38

編集2019/01/23 03:41

mayoi_maimai

総合スコア1583

ariiiiiga

2019/01/23 05:10

Notice: Undefined variable: where in D:\xampp\htdocs\test\2.php on line 15 １５行名 if($where){ エラーが出てしまいました。

mayoi_maimai

2019/01/23 05:27

別ファイルで試していませんか？？元のファイルの以下部分を上書きするかコメントアウトして追記していただければ大丈夫だと思いますよ。 if($where){ $whereSql = implode(' AND ', $where); $sql = 'select * from hosts where order by post_id desc ' . $whereSql ; }else{ $sql = 'select * from hosts'; }

ariiiiiga

2019/01/23 07:01

違うファイルでやってました！解決しました！ありがとうございます！

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

あなたの回答