forループが正しく動作してくれません。アドバイスお願いします。

Question

どうしてもわからずに困っております。
以下に説明と具体的にスクリプトを載せておりますので、
アドバイスを宜しくお願い致します。


単純にユーザーがformのチェックボタンによって送信してきたデータをphpが受け取って、$v_madori1から$v_madori6までの変数を用意します。

php側としては、nullが6個、つまりv_madori1からv_madori6まですべて中身が空っぽの場合だけ、
エラーのhtmlページへ飛ばすという仕組みです。
(ユーザーがhtmlのページでチェックボタンを一つもチェックしなかったらエラーにしたいという考えです。)


そこで、for文を使って簡単にエラーをチェックしてみようと思いました。

forループ内の最後のheader関数は、
結局６つともnullだったということなので、
errorページへ飛ばすという意味の関数です。


以下にスクリプトの一部を抜粋して載せました。
間違いをご指摘ください。
宜しくお願い致します。


//間取りの情報受信
$v_madori1 = $_REQUEST['v_madori1'];
$v_madori2 = $_REQUEST['v_madori2'];
$v_madori3 = $_REQUEST['v_madori3'];
$v_madori4 = $_REQUEST['v_madori4'];
$v_madori5 = $_REQUEST['v_madori5'];
$v_madori6 = $_REQUEST['v_madori6'];


for($i=1 ; $i=<6 ; $i++){
if($v_madori.$i == null){
continue;//nullだったので次のループへ
}else{
break;//nullじゃないということなので、forループを抜ける
}
//全部nullだったのでそれではまずいのでエラーページへ。
header("location:http://www.xxxxx.biz/error.html");
}

imasyu · Accepted Answer

可変変数を使ってNo3さんと同じ考え方でひとつ

$check = false;
for($i=1 ; $i<=6 ; $i++){
　　if(${"v_madori".$i} <> null){
　　　$check = ture;
　　　break;
　　}
}
if(!$check)header("location:http://www.xxxxx.biz/error.html");

質問のソースではforループの中でコンテニューとブレイクを両方使用していましたが、これだと条件がどうであれヘッダー関数にたどり着けないような気がします。なのでNo3さんの考えが一番簡単かつよい方法だと思います。
あとforループの条件　$i=>6　じゃなくて　$i<=6　だと思います。　もう既にわかってらっしゃる場合はご容赦ください。
ただこういう間違いは結構気づきにくいんですよね。

DPE · Answer

こう考えてはいかがでしょう。

変数をもう１つ用意します。ここではとりあえず、$check という名前だとします。for ループに入る前に、$check に初期値として０を入れておきます。
for ループを使って、$v_madori1 ～ 6 をチェックします。ループ中は、$v_madori が null でない場合は $check に $i の値を入れるという処理をします。
ループ終了後、$v_madori1 ～ 6 の全てが null でなければ $check に１～６のいずれかが入っているはずですが、全て null の場合、$check は０のままです。
あとは if 文で $check の値を見て、０の時にはエラーのページに飛ぶ、という処理をすれば完成です。
なお、$check が０でなくなれば $v_madori1 ～ 6 のいずれかはチェックされているということですから、$check が０でなくなった時点で for ループを打ち切っても構いません。

当方は php には詳しくないので文法的には自信がありませんが、こんな感じでしょうか。


　//チェック用変数を初期化
　$check = 0;
　//$v_madoriを全て調べる
　//$checkが変更されたら、その時点でチェック終了
　for( $i=1 ; $i<=6 && $check == 0 ; $i++ )
　{
　　//nullでなければ、チェック用変数を更新
　　if( $v_madori.$i != null )
　　{
　　　$check = $i;
　　}
　}
　//全てnullだった場合、エラーページへ
　if( $check == 0 )
　{
　　header("location:http://www.xxxxx.biz/error.html"); 
　}


チェック用の変数を用意し、その値がループの前と後とで変わっているかどうかで判断する方法はよくある手法なので、よろしければご参考までに。
（上記のプログラムで文法の誤りがありましたら、お手数ですが正しく修正してご利用下さい）

okotaegoo · Answer

全く試してみてないので、自信ないのですが
if(($v_madori.$i) == null)
にしては？

GuruGuru22 · Answer

配列使うか、可変変数使うかかな。
ただの数字なら、普通は配列かと。
可変変数だったら、${"v_madori".$i}

forループが正しく動作してくれません。アドバイスお願いします。

可変変数を使ってNo3さんと同じ考え方でひとつ

こう考えてはいかがでしょう。

全く試してみてないので、自信ないのですが

配列使うか、可変変数使うかかな。

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング