while文の中でif文を使いたい

たびたび、質問申し訳ございません。
3時間くらい悩んで解決しないので、いい案をいただけると幸いです。

DBのレコード$user_idの１件１件と、入力したID($_POST['user'])が
一致した際は、「そのユーザーIDはすでに使用されています」というエラーを出し、
一致しない場合は「登録ボタン」を表示させたいです。

下記のようなコードを作ってみたのですが、
elseで「登録ボタン」を表示させてもwhileの中なので、いくつもボタンが作られてしまいますし、
だからといってwhileを消すと、１件１件を見ることができません。

何かいい方法があれば教えていただきたいです。

あと、もし「このように検索すればいいよ」というのがあれば
それも教えていただけると幸いです。
３時間検索しまくりましたが、解決しません。。

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

			while($user_id = $stmt->fetch()) {
				if($user_id['user_id'] === $_POST['user']) {
			 		echo '<br>*そのユーザーIDはすでに使用されています';
			 	}
			}

javahack

2021/05/30 00:49

既に回答でも指摘されていますが、DBを使用しているなら必要なデータの絞込はDBでやりましょう。常に全件検索なんてDBを使用するメリットを放棄しています。

kens_Freelance

2021/05/30 06:11

ありがとうございます。当たり前の部分が見えなくなってしまっていました。素早いご回答ありがとうございます。

行動規範の内容に同意します

回答2件

他の方も言われているとおり DB の検索処理を見直すのがオススメですが、参考まで。

「ひとつでも一致するものがあったか」または「すべて一致しなかったか」を判断するには、すべての要素をループしてチェックし終わらないと分からないので、以下のようにフラグ変数($found)を用意してループを抜けたあとに判定すると良いです。

php
1$found = false;
2while (...) {
3    if (...) {
4        $found = true;
5        break;
6    }
7}
8if ($found) {
9    // ひとつでも一致するものがあった場合
10} else {
11    // すべて一致しなかった場合
12}

投稿2021/05/30 05:30

yh1224

総合スコア653

kens_Freelance

2021/05/30 06:11

ありがとうございます。検索処理を見直したらうまくいきました。問題点にとらわれすぎていると、当たり前の処理が思いつかないので、もう少し広い視野をもって考えます。また、ループの処理も教えていただきありがとうございます。今後、このような処理を使う場合は参考にします。

行動規範の内容に同意します

ベストアンサー

そもそも、SELECT時点でそのPOSTされたuser_idを条件に検索すればループ内で比較する必要もループする必要もないのでは。

MySQL 5.6 リファレンスマニュアル #行のカウント

ただ、過去質問見てると、全体像考えずにとりあえずその場その場で考えながらコードを書いているように見受けられます。
メモレベルでいいので考えてから（できればフローチャートくらいは欲しい）書いたほうが時間だけが過ぎていくことは少なくなると思います。
いきなりコードを書くのではなく、全体の流れと必要そうな材料の選定と、材料の組み合わせを考える。

投稿2021/05/30 00:29

編集2021/05/30 00:39

m.ts10806

総合スコア80875

kens_Freelance

2021/05/30 06:09

いつもありがとうございます。アドバイスいただいたとおりにすれば、うまく動きました。念のため、うまくいったコードを下記します。＞ただ、過去質問見てると、全体像考えずにとりあえず＞その場その場で考えながらコードを書いているように見受けられます。本当にそのとおりでした。といいますか、webアプリを作るときはフローチャートを作るのは当たり前なのですね。。知りませんでした、教えていただけて感謝します。今からでもいいので、作ってみます。 $user_id = $stmt->fetchAll(); if(empty($user_id)) { echo '重複しているIDはありません'; } else { echo 'IDが重複しています'; }