if文がifにならない？

質問

よろしくお願いします。
連続投稿防止の為に、getenvで取得したIPアドレスと
time()で取得した時間をcsvファイルに
xxx.xxx.xxx.xxx<>1249494949
xxx.xxx.xxx.xxx<>1249494949
という風に記述をしてます。

具体的には、
<?php
$ipdesu = "./ip.csv";
$ipgetdesu = getenv( "REMOTE_ADDR");
$times = time();
$hoge = 0;

$roop_open = fopen( $ipdesu, "r");
while(! feof($roop_open)){
$roop_get = fgets( $roop_open);
$roop_ex = explode( "<>", $roop_get);
$hogehoge = $hoge++;
//IPチェック
if( in_array( $ipgetdesu, $roop_ex)){
echo "配列検索TRUE";
//時間チェック
if ( "60" < $times - $roop_ex[1]){
$fp = file( $ipdesu);
unset( $fp[$hogehoge]);
$fp_open = fopen( $ipdesu, "w");
foreach ( $fp as $value){
fputs( $fp_open, $value);
}
fclose( $fp_open);
}else{
echo "連投規制";
}
}else{
$fal_open = fopen( $ipdesu, "a");
fputs( $fal_open, $ipgetdesu."<>".time()."\n");
fclose( $fal_open);
echo "配列検索FALSE";
}
}
fclose( $roop_open);
?>

と記述しています。
＜まだ、始めたばかりなのでロック制限とかしてませんがそこはご勘弁を

上記を実行すると、if　ここ else　そこ
の本来、真であれば「ここ」が
偽であれば「そこ」だけが実行されると思っていたのですが、
上記の実行結果は、「ここ」も「そこ」も実行されてしまい

かつ、IPを記述したファイルの検索が２回め実行した時に
同じIP
127.0.0.1<>時間
127.0.0.1<>時間
とローカルで試してるのでIPが変更になるはずないのですが(DHCPは念の為しようしていません
in_arrayでヒットしないのは何故でしょうか？
また、ifが正しく動作しないのは多重ifとでも言えばいいのでしょうか？
が悪いからでしょうか。

つたないソースで申し訳ないですが、どうかよろしくお願いします。

回答No.4

superside0 · ベストアンサー

全角空白でインデントを入れてみました。

$roop_open = fopen( $ipdesu, "r");
while(! feof($roop_open)){
　$roop_get = fgets( $roop_open);
　$roop_ex = explode( "<>", $roop_get);
　$hogehoge = $hoge++;
　if( in_array( $ipgetdesu, $roop_ex)){
　　echo "配列検索TRUE";
　　//時間チェック
　　if ( "60" < $times - $roop_ex[1]){
　　　$fp = file( $ipdesu);
　　　unset( $fp[$hogehoge]);
　　　$fp_open = fopen( $ipdesu, "w");
　　　foreach ( $fp as $value){
　　　　fputs( $fp_open, $value);
　　　}
　　　fclose( $fp_open);
　　}else{
　　　echo "連投規制";
　　}
　}else{
　　$fal_open = fopen( $ipdesu, "a");
　　fputs( $fal_open, $ipgetdesu."<>".time()."\n");
　　fclose( $fal_open);
　　echo "配列検索FALSE";
　}
}

if , else の構文自体は問題ないと思いますが
過去投稿を記録している ip.csvを fgetsで１行毎に読み込んで
合致しているかの処理にしているので、読み込んだ　ip.csvの
１行目では、if文が真のときの処理
２行目では、if文が偽のときの処理
といった感じで、ip.csvの行数だけ、何度も処理を実行している
という感じではないでしょうか。

しかも、読み込み中のファイルに対して、
同じファイル名を書き込みオープンして、上書きしているのも、
おかしなことになりそうです。

読み込みするwhile内では、連続投稿があったかどうかのフラグだけとり、
whileが抜けてから、そのフラグによって、
ファイルの上書き処理をする形がよいかと思います。

（ファイルのロック処理は別途）

それと、クライアントのIPアドレスは
getenv( "REMOTE_ADDR");
でなく、
$_SERVER['REMOTE_ADDR']
ですね。

お礼

お返事有難うございます。

>if , else の構文自体は問題ないと思いますが
中略
>という感じではないでしょうか。

凄く納得出来ました。
なるほど、そういう処理の仕方だったんですね。

>しかも、読み込み中のファイルに対して、
>同じファイル名を書き込みオープンして、上書きしているのも、
>おかしなことになりそうです。

いったん別名で保存してから内容の反映をしようと思ったのですが、
余計にややこしくなると思った処理が
余計にややこしくしてたんですね。
勉強になります。

まだ、ループを抜けての処理というのを
簡単な処理しかした事がないのでチャレンジしてみたいと思います。

$_SERVER['REMOTE_ADDR']
に変更して処理をしてみようと思います。

補足

同じ文章になってしまいますが、
ここに投稿したおかげでなんとなく脱け出せそうです。
回答して頂いて有難う御座いました。

一度、じっくり考えたいと思いますので
締め切らせて頂きたいと思います。

皆さん有難うございました。

回答No.5

A88No8

こんにちは
#1です。
>while(! feof($roop_open)){
　本当にごめんなさい。
　見落としてました。でも、有識者の方の適切なアドバイスがあってよかったです。

回答No.3

UmJammer

ip.csvのファイルの末尾に改行コードが挿入されているため、whileでまわしたとき最後に空行が処理され、そのとき$roop_exは空文字列となり、そのため後続のin_arrayで想定外の動きをしてしまっている状態と思われます。
一番最初（ip.csvが空のとき）は正常に動作すると思いますが、それはたまたまということになります。

個人的には連投の判定部分とip.csvへの書き込み部分の処理は分けた方が良いような気がします。

お礼

お返事有難うございます。

空文字削除の為に、trim系列？の関数を試していますが
中々、最後の行だけ削除が出来ても
どうして、次のループでまた旨く逝かないとかてこずってます。

書き込み部分の処理分けは、まだ未熟な為
そこまで頭が回りませんが（ユーザー関数とかなんでしょうか。
また、次回作成時頑張ってみます。

今は、目の前の壁に全力投球します！

出口が見えそうで見えないのは、
なんともいやはや歯がゆいですね。

補足

同じ文章になってしまいますが、
ここに投稿したおかげでなんとなく脱け出せそうです。
回答して頂いて有難う御座いました。

回答No.2

yambejp

in_array()の挙動に疑問があるのであれば
その直前行に、
print_r($ipgetdesu);
print_r($roop_ex);
などして、整合性があっているかデバッグしてみてはどうでしょうか？

お礼

お返事有難うございます。

直前に
print_r($ipgetdesu)."<br>\n";
print_r($roop_ex);
if( in_array( $ipgetdesu, $roop_ex)){
と記述してみたところ

127.0.0.1
Array ( [0] => 127.0.0.1 [1] => 1249450592 ) 配列検索TRUE連投規制
と表示されているので整合性は問題ないと思うのですが。
何か他のデバッグ方法はありますか？
一応、if直後とかに何がどう表示されるか等チェックはしているのですが
いかんせん内部でどのような処理が行われているかが謎ですので、
困り果ててます。

補足

同じ文章になってしまいますが、
ここに投稿したおかげでなんとなく脱け出せそうです。
回答して頂いて有難う御座いました。

回答No.1

A88No8

こんにちは
　PHPという言語システムについては、全くの門外漢ですので参考まで。
　下記のように//IPチェックから以降をインデントを付けてみました。
　門外漢が見てですがif文の書き方(ブロックの書き方)が"連投規制"後に、余ってしまっているように見えます。
　もう一度ブロックを見直してみて下さい。

//IPチェック
if( in_array( $ipgetdesu, $roop_ex)){
　echo "配列検索TRUE";
　//時間チェック
　if ( "60" < $times - $roop_ex[1]){
　　$fp = file( $ipdesu);
　　unset( $fp[$hogehoge]);
　　$fp_open = fopen( $ipdesu, "w");
　　foreach ( $fp as $value){
　　fputs( $fp_open, $value);
　}
　fclose( $fp_open);
}else{
　echo "連投規制";
}
}else{
　$fal_open = fopen( $ipdesu, "a");
　fputs( $fal_open, $ipgetdesu."<>".time()."\n");
　fclose( $fal_open);
　echo "配列検索FALSE";
　}
}
fclose( $roop_open);
?>

お礼

お返事有難うございます。
余っているというのは、処理の空白部分があるとい解釈でよろしいでしょうか？

いまいちwhileの挙動について理解が足りていませんが、
もしかして、
if　真　if　真　もしくは　偽　の後はwhileを抜けない？
という事で処理が手持ちぶたさなのでしょうか。
念の為、連投規制の後と、その一つ手間の fclose($fp_open)　に　break;　を設けてみた所

綺麗にIPを除去してから再び追記してるかと思いきや、
一気に、３個同じ行を作成したりと謎な動作をしてます。
更新ボタンを連打しなくてもそうなるようです。

補足

ここに投稿したおかげでなんとなく脱け出せそうです。
回答して頂いて有難う御座いました。