質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.61%

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

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 964

score 46

前提・実現したいこと

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>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • Orlofsky

    2019/01/23 14:04

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

    キャンセル

回答 2

checkベストアンサー

+1

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

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

    $sql .= " order by post_id desc";

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/01/23 14:10

    Notice: Undefined variable: where in D:\xampp\htdocs\test\2.php on line 15

    15行名
    if($where){
    エラーが出てしまいました。

    キャンセル

  • 2019/01/23 14:27

    別ファイルで試していませんか??
    元のファイルの以下部分を上書きするかコメントアウトして追記していただければ大丈夫だと思いますよ。

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

    キャンセル

  • 2019/01/23 16:01

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

    キャンセル

+1

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

PDO::query#返り値 

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

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

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

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

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

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

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

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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/01/23 14:08

    Notice: Undefined variable: where in D:\xampp\htdocs\test\2.php on line 15
    select * from hosts

    15行名
    if($where){

    エラーが出てしまいました。

    キャンセル

  • 2019/01/23 14:12

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る