foreach文とfor文について

Question

下記のソースは、書籍(気づけばプロ並みPHP)を参考に作成したショッピングカートの買い物かごにあたる部分の処理です。
foreach文を使って、データベース上にある商品をとりだし、foreach文の中で商品を表示しています。
しかし参考書には、foreach文で商品を表示することはよくないのでforeachから分離し別途for文で表示するように書いてありました。理由は、「表示したい配列が一つならforeachで構わないが、配列変数が複数あるためfor文を使うべき」とのことです。、foreach文だけでも問題なく商品は表示されているし、理由で挙げられている、単数ならforeach、複数ならforという区分けもいくつかの参考書を手に取ってみましたが解かりませんでした。

なぜ、foreach文ではなくfor文で商品を表示させるのでしょうか。

require_once "../lib/dbaccess.php";

$cart = $_SESSION['cart'];
//$_SESSION['cart']は、$cart配列がどのページでも使えるようにしたものだ。中身は、商品に関する情報を管理する一意の商品番号が入っている。
foreach($cart as $key => $val) {
	
$dbh = dbaccess();	
//mst_productテーブルに商品の番号、名前、価格、画像が入っている。
$sql = 'SELECT * FROM mst_product WHERE code=?';
$stmt = $dbh->prepare($sql);
//$valは、$_SESSION['cart']に代入された商品番号が代入されている。
$data[0] = $val;
$stmt->execute($data);
$dbh = null;

$rec = $stmt->fetch(PDO::FETCH_ASSOC);
echo <<<EOL

<p>{$rec['code']}</p>
<p>{$rec['name']}</p>
<p>{$rec['price']}</p>
EOL;
if(!$rec['gazou'] == '') {
 print'<p><img src="../product/gazou/'.$rec['gazou'].'"></p>';
}
print'<hr>';


}//foreach

http://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q11151770801

申し訳ありませんでした。書き損じておりました。
参考にした書籍の内容は第二版で、ページは184Pです。
提示してあるソースは、内容を見る前に自身で作成したものを184pに掲載されているソースで加味したものであります。

Comments

[KiyoshiMotoki]

書籍を確認し、それらしい記述を見つけたのですが、ご提示いただいたソースコードと書籍に載っているものが異なるため、確証が持てません。

第二版の184ページで、間違いないでしょうか？

Answer 1

情報の追記、ありがとうございます。

今、手元に書籍がないので思い出しながら回答させていただきます。

該当ページのソースコードでは、foreach文の中で１行ずつDBから取り出したデータを
$pro_name、$pro_price、$pro_gazou
という3つの配列変数に格納していました。

以下のような感じだったと思います。

foreach($cart as $key => $val) {

    $dbh = dbaccess();    
    //mst_productテーブルに商品の番号、名前、価格、画像が入っている。
    $sql = 'SELECT * FROM mst_product WHERE code=?';
    $stmt = $dbh->prepare($sql);
    //$valは、$_SESSION['cart']に代入された商品番号が代入されている。
    $data[0] = $val;
    $stmt->execute($data);
    $dbh = null;

    $rec = $stmt->fetch(PDO::FETCH_ASSOC);


    $pro_name[] = $rec['name'];
    $pro_price[] = $rec['price'];
    $pro_gazou[] = is_null($rec['gazou']) ? '' : $rec['gazou'];
}

で、これら3つの変数に格納された値をWEBページに表示しようとする際に

foreach文で商品を表示することはよくない（または、できない）

という趣旨の記述が（理由を説明することもなくw）あったと思います。

私の想像では、ここで著者が言いたいことは
「foreach文では同時に2つ以上の配列に逐次アクセスすることができないため」
ではないかと思います。

// このようなことはできない。
foreach ($pro_name as $name, $pro_price as $price, $pro_gazou as $gazou) {
    echo "名前は" . $name . "<br>";
    echo "値段は" . $price . "<br>";
    echo "<img src='" . $gazou . "'";
}

なので、代わりにfor文を使用することを推奨しているわけです。
for文なら、複数の配列変数に同時に逐次アクセスできますので。

// これは可能。
for ($i = 0; $i < count($pro_name); $i++) {
    echo "名前は" . $pro_name[$i] . "<br>";
    echo "値段は" . $pro_price[$i] . "<br>";
    echo "<img src='" . $pro_gazou[$i] . "'";
}

※質問を投稿する際は、なるべく精確かつ詳細に情報を載せて下さい。
今回のように記述されている箇所を特定できなかったり
そもそも書籍に書かれているのとは異なるソースコードを載せられたりすると、
回答できないばかりか、見当違いな回答をしてしまう可能性がありますので。
それは、お互いにとって不幸なことです。

Comments

[aaaaaaaa]

ご回答有難うございます。とても分かりやすい解説で理解できました。
>>「foreach文では同時に2つ以上の配列に逐次アクセスすることができないため」
無理やりforeachにしようとすると、pro_name用、pro_price用、pro_gazou用のforeachが必要になってしまうので、一つで上手くいくforで済ませようとしたということなのですね。

質問文に関しては重ね重ね申し訳ありませんでした。

[KiyoshiMotoki]

> 無理やりforeachにしようとすると、pro_name用、pro_price用、pro_gazou用のforeachが必要になってしまうので、一つで上手くいくforで済ませようとしたということなのですね。

はい、おっしゃる通りです。

Answer 2

引用テキストしかし参考書には、foreach文で商品を表示することはよくないのでforeachから分離し別途for文で表示するように書いてありました。理由は、「表示したい配列が一つならforeachで構わないが、配列変数が複数あるためfor文を使うべき」とのことです。、foreach文だけでも問題なく商品は表示されているし、理由で挙げられている、単数ならforeach、複数ならforという区分けもいくつかの参考書を手に取ってみましたが解かりませんでした。

上記の文章からは私も理由がわかりませんでした。
ループ処理を行う場合に通常はfor/foreachどちらを使うべきという事はありません。

プログラムを拝見すると1点問題がありますので、書籍ではそちらを懸念しているのかもしれません。
例示されたものではforeachループの中でDBから1つずつデータを取ってきていますので、
例えば100アイテムある場合はSELECTを100回発行することになります。
これはデータベースに連続で負荷をかける為、全データを1回でSELECTする形に改善するべきです。
簡単に例示すると以下のようになります。（エラー処理等は省いています）

require_once "../lib/dbaccess.php";

$cart = $_SESSION['cart'];

//配列が空か？
if (!is_array($cart) || count($cart) < 1) die("NODATA");

//mst_productテーブルに商品の番号、名前、価格、画像が入っている。
$dbh = dbaccess();
$sql = 'SELECT * FROM mst_product WHERE code in ('.str_repeat('?,', count($cart)-1).'?)';
$stmt = $dbh->prepare($sql);
//$cart[n]='商品コード'の形で記録されていると仮定
$stmt->execute($cart);
$items = $stmt->fetchAll(PDO::FETCH_ASSOC);
$dbh = null;

//$_SESSION['cart']は、$cart配列がどのページでも使えるようにしたものだ。中身は、商品に関する情報を管理する一意の商品番号が入っている。
foreach($items => $rec) {
echo <<<EOL

<p>{$rec['code']}</p>
<p>{$rec['name']}</p>
<p>{$rec['price']}</p>
EOL;
if(!$rec['gazou'] == '') {
 print'<p><img src="../product/gazou/'.$rec['gazou'].'"></p>';
}
print'<hr>';


}//foreach

Comments

[aaaaaaaa]

ご回答有難うございます。

>>プログラムを拝見すると1点問題...
質問だけでなく別の問題点まで指摘してくださって有難うございます。
ループ内でDBに接続したりプリペアドステートメントを作ったり、ととても迂闊でした。勉強になります。