【PHP】Invalid argument supplied for foreach() エラーが表示されたりされなかったり・・・

Question

###前提・実現したいこと
SQLのテーブルに登録したIDとパスワードを、HTMLのtable内に全て正しく表示させるのが最終目標です。

問題は、コピペして同じように書いているはずなのに、何故かエラーが吐かれる箇所とエラーのない箇所が同時に発生してしまっていることです。
###発生している問題・エラーメッセージ

Warning: Invalid argument supplied for foreach() in /home/users/****/****/****/****/****/manage_list.php on line 78
Warning: Invalid argument supplied for foreach() in /home/users/****/****/****/****/****/manage_list.php on line 83

###該当のソースコード

<?php $stmt4 =$pdo](1147ac7e24e1de6ee8e92aee88938b41.png)->query('SELECT
login_no
FROM
login_member'); ?>

<?php $stmt5 =$pdo->query('SELECT
login_id
FROM
login_member'); ?>

<?php $stmt6 =$pdo->query('SELECT
login_pass
FROM
login_member'); ?>
<?php while($re2 = $stmt4->fetch(PDO::FETCH_ASSOC) && $re3 = $stmt5->fetch(PDO::FETCH_ASSOC) && $re4 = $stmt6->fetch(PDO::FETCH_ASSOC))  :?>
<tr>
	<td>
		<?php foreach($re2 as $k => $v):?>
		<?php print_r($v)?> **←エラー箇所**
		<?php endforeach;?>
	</td>
	<td>	
		<?php foreach($re3 as $k => $v):?>
		<?php print_r($v)?> **←エラー箇所**
		<?php endforeach;?>
	</td>
	<td>							
		<?php foreach($re4 as $k => $v):?>
		<?php print_r($v)?> **←ここは何故かエラーがなく、正しくlogin_passが表示される**
		<?php endforeach;?>
	</td>
	<form action="" method="post">
	<td><input type="button" name="edit" onclick="location.href='./manage_edit.php'" value="編集" />
	<input type="button" name="delete" onclick="location.href='./manage_conf.php'" value="削除" /></td>
</tr>
<?php endwhile;?>

###試したこと
1.Invalid argument supplied for foreach()エラーをググってみたところ、foreach内の変数が配列ではない場合に吐かれるエラーとの事なので、

<?php foreach($re2 as $k => $v):?>を <?php foreach((array)$re2 as $k => $v):?>に

変更してみた所、
エラーは吐かれなくなったのですが、login_noもlogin_idも「１」しか表示されなくなってしまいました。

2.<?php while($re2 = $stmt4->fetch(PDO::FETCH_ASSOC) && $re3 = $stmt5->fetch(PDO::FETCH_ASSOC) && $re4 = $stmt6->fetch(PDO::FETCH_ASSOC)) :?>
の条件式の前に、

<?php while($re2 = $stmt4->fetch(PDO::FETCH_ASSOC)):?>

						<?php foreach($re2 as $k => $v):?>
						<?php print_r($v)?>
						<?php endforeach;?>
					<?php endwhile;?>
					

<?php while($re3 = $stmt5->fetch(PDO::FETCH_ASSOC)):?>

						<?php foreach($re3 as $k => $v):?>
						<?php print_r($v)?>
						<?php endforeach;?>
					<?php endwhile;?>

と試しに打ってみたら、
login_noもlogin_idも正しくprint_rされたので、
なぜ該当ソースコードの中ではエラーが吐かれてしまうのかわけがわかりません。。。

わかりづらい所、至らぬ所があったらご指摘ください。

ご教授宜しくお願い致します。

###補足情報(言語/FW/ツール等のバージョンなど)
FWは使っていません

以下エラーのスクショです

Answer 1

while文の条件式が危なっかしいですね。

とりあえずwhileループ中の$re2とか$re3とかの値を確認してみては？

Comments

[zoizoi]

ご回答ありがとうございます。

while文の条件式がスマートではないのは自分でも感じているのですが、
試行錯誤した末にこの条件式で惜しい所まで来たので他にどう書いたら良いのかわからないのです・・・。

Answer 2

がるです。
すみません、こちらのシステムだと、コメントにはソースコードがうまく書けないので、改めて「回答」のほうで書かせていただきます。

分割して書かないと、HTMLの各<td>内に、login_no, login_id, login_passを、添付した画像のように分けて表示させる事が出来なくて

とあったので、おそらく、(連想)配列等に慣れてらっしゃらないように見受けました。
ざっくりとノーデバックで記述したので修正が必要かと思うのですが。
大まかな「こんな感じで書けば、SQLの発行が１回ですむよ」のサンプルとして、以下のコードを見ていただければ、と思います。

<?php
// SQLを発行
$stmt =$pdo->query('SELECT * FROM login_member');

// 取得したデータ分、ループを回す
while($re = $stmt->fetch(PDO::FETCH_ASSOC)) :?>
<tr>
    <td>
        <?php print_r($re['login_no'])?>
    </td>
    <td>    
        <?php print_r($re['login_id'])?>
    </td>
    <td>                            
        <?php print_r($re['login_pass'])?>
    </td>
    <form action="" method="post">
    <td><input type="button" name="edit" onclick="location.href='./manage_edit.php'" value="編集" />
    <input type="button" name="delete" onclick="location.href='./manage_conf.php'" value="削除" /></td>
</tr>
<?php endwhile;?>

質問等あったら、また記入していただければ、と思います。

Comments

[zoizoi]

がるさん

ご丁寧にありがとうございます！
ほとんどこのまま書いたら、login_noとlogin_passが正しく表示されました！

ご指摘の通り自分は連想配列が苦手で、更にfetchの挙動もよく理解せずに書いていました。

教えていただいたら、なんでこんな簡単なことに・・・と言った感じなので、これからはもう少し考えてから質問したいと思います。

がるさんどうもありがとうございました！

[gallu]

連想配列は、確かにイメージがしにくいもの、かもしれません(苦笑
print_rもよいのですが、個人的には var_dump のほうを出力ではよく使いますね。
「うまくいった」コードの、連想配列が「どんなデータが入っているのか」表示をしてなんとなし眺めてみる、なんていうのもよいかもしれません。

頑張ってください＾＾ノ

Answer 3

がると申します。

ものすごく単純な疑問なのですが。

SELECT login_no FROM login_member
SELECT login_id FROM login_member
SELECT login_pass FROM login_member

のSQLを発行するのであれば、代わりに

SELECT * FROM login_member

またはより適切(元のコードに忠実)には

SELECT login_no, login_id, login_pass FROM login_member

の１回だけ、SQLを発行したほうがコードが明らかに楽、かつ見通し良く書けると思うのですが、なにか「分割して書かなければいけない」理由など、ある感じでしょうか？

Comments

[gallu]

あと、追記ですが。
基本的には「パスワードを平文で所持する」のは、大変に「お勧めをしない」実装になります。

[zoizoi]

がるさん

ご回答ありがとうございます。

>「分割して書かなければいけない」理由など、ある感じでしょうか？

分割して書かないと、HTMLの各<td>内に、login_no, login_id, login_passを、添付した画像のように分けて表示させる事が出来なくてこう書いています。
冗長だとは思いますが・・・

パスワード平文に関しては、今回の課題の条件で、パスワードはあえて表示させることを要求されているので大丈夫です。
しかし実務システムを作る際には気をつけます。ありがとうございます。