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

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

ただいまの
回答率

88.34%

PHPの記述方法について

解決済

回答 3

投稿

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

ssk

score 288

<?php
    $count=0;
    while(true){
        $rec = $stmt->fetch(PDO::FETCH_ASSOC);
        if($count==0){//最初のループで取り出すデータがない場合
                if($rec==false){
                echo 'テキストテキストテキストテキスト';
            }
        }
            if($rec==false){
                break;
            }
            echo '<li class="list-group-item clearfix">';
            echo 'データを出力'
            echo '<li>';
            $count++;
    }
    ?>

1行ずつデータベースから取り出して、表示したいです。
ただ、取り出せなかった場合の処理を追加したく、悩んでいます。

今、一応取り出せているのですが
方法は最適でしょうか?

他のファイルにも適用していくので
もっと、良い方法があれば教えていただきたいです。

よろしくお願い致します。

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • KiyoshiMotoki

    2016/06/03 18:20

    WHERE句をわざとあり得ない条件に書き換えるなどして、1行も取得できなかった場合にどのように表示されるか確認してみてください。そうすれば、この方法が最適か否か分かると思います。

    キャンセル

  • KiyoshiMotoki

    2016/06/03 18:23

    失礼しました、プログラムを読み違えました。
    上の指摘は無視願います。

    キャンセル

回答 3

checkベストアンサー

+1

一見する限り、おそらく意図通りに動きますが、ちょっと冗長な書き方に感じますね。

例えば、while文の条件式の中で$stmt->fetchを呼び出してやるようにすると、以下のように書けます。

// $count=0; 行をカウントする必要はない
$exists = false;
while (($rec = $stmt->fetch(PDO::FETCH_ASSOC)) !== false) {
    echo '<li class="list-group-item clearfix">';
    echo 'データを出力';
    echo '<li>';
    $exists = true;
}

if (!$exists) { //取り出すデータがなかった場合
    echo 'テキストテキストテキストテキスト';
}


さらには、$stmt->fetchの代わりに$stmt->fetchAllを使うと、以下のようにも書けます。
http://php.net/manual/ja/pdostatement.fetchall.php

$resultSet = $stmt->fetchAll(PDO::FETCH_ASSOC);
if (!$resultSet) { //取り出すデータがなかった場合
    echo 'テキストテキストテキストテキスト';
} else {
    foreach ($resultSet as $rec) {
        echo '<li class="list-group-item clearfix">';
        echo 'データを出力';
        echo '<li>';
    }
}


個人的には、使用する変数の数が少ない後者の方法が好みです。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/06/03 20:57

    取得失敗時に $resultSet は空配列になるので,厳密な比較を行う
    「if ($resultSet === false)」
    だと意図したように動かないと思います.
    「if (!$resultSet) 」または「if (empty($resultSet))」
    で十分でしょう.
    (個人的には無駄なNotice抑制のない前者のほうが好みなんですが,emptyという語のわかりやすさからPHPコア開発者の中でも後者を推奨する派が多いようです)

    キャンセル

  • 2016/06/03 22:38


    KiyoshiMotoki様、CertaiN 様

    こんなにスッキリするとは、、、
    ありがとうございます。

    キャンセル

  • 2016/06/04 13:43

    CertaiN様

    ご指摘ありがとうございます。

    改めて確認したところ、fetchAll の戻り値は以下のように説明されていました。

    > 取得結果がゼロ件だった場合は空の配列を返し、 失敗した場合は FALSE を返します。
    http://php.net/manual/ja/pdostatement.fetchall.php

    おっしゃる通り if ($resultSet === false) では意図通りに動作しないので、
    回答を修正させていただきました。

    ssk様

    お役に立てたなら幸いです。

    キャンセル

0

KiyoshiMotoki様

ありがとうございます。

WHERE句をわざとあり得ない条件に書き換えるなどして、1行も取得できなかった場合にどのように表示されるか確認してみてください。
↑試してみたのですが、意図した結果を得られています。

記述方法には問題なさそうでしょうか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

ざっと書くと

// データをとるところ
    $ar = array();
    while($rec = $stmt->fetch(PDO::FETCH_ASSOC)){
        $ar[] = $rec;
    }

    if (empty($ar)) { // とれなかったとき
        echo 'テキストテキストテキストテキスト';
    } else { // とれたとき
        foreach ($ar as $r) {
            echo '<li class="list-group-item clearfix">';
            echo 'データを出力'
            echo '<li>';
        }
    }


さらに、(できれば)機能ごとに関数化して切り分けるのが好みかなあ。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/06/03 22:36

    takasima20様

    ありがとうございます。
    関数化は考えていませんでした。試してみます。

    キャンセル

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

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

関連した質問

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