PHPの記述方法について

Question

<?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行ずつデータベースから取り出して、表示したいです。
ただ、取り出せなかった場合の処理を追加したく、悩んでいます。

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

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

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

Comments

[KiyoshiMotoki]

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

[KiyoshiMotoki]

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

Answer 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>';
    }
}

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

Comments

[mpyw]

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

[ssk]

KiyoshiMotoki様、CertaiN 様

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

[KiyoshiMotoki]

CertaiN様

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

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

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

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

ssk様

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

Answer 2

ざっと書くと

    // データをとるところ
    $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>';
        }
    }

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

Comments

[ssk]

takasima20様

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

Answer 3

KiyoshiMotoki様

ありがとうございます。

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

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