ファイルに1行しか書いていないのに2回読まれる - PHPプロ！Q&A掲示板
１）ファイルへ書き込むプログラム
別のHTML入力画面からデータを取得して、
ファイルに書き込んでいます。
<?php
#データを取得する
$name = $_POST["name"];
$mail = $_POST["mail"];
$title = $_POST["title"];
$message = $_POST["message"];
# 日本語表記の日時を得る
$time = date("Y/m/d l H:i:s");
# データを１行にまとめる（項目の区切りは%）
$data = $time."%".$name."%".$mail."%".$title."%".$message."\n";
# 追加書き込みモードでファイルを開く
$handle = fopen("bbs","a") ;
flock($handle, LOCK_EX);
fwrite($handle,$data);
#ファイルをアンロックする
flock($handle, LOCK_UN);
fclose($handle);
#?>
２）ファイルから読み込むプログラム
１）で書き込んだファイルから1行ずつ読み込み、表示し、
ファイルが終了するまで継続しています。
このとき、1行余分に、読み込まれます。
すなわち、while(!feof($handle))で、1回余分にループしていることになります。
なぜでしょうか。
# 読み込みモードでファイルを開く
$handle = fopen("bbs","r") ;
#ファイルの最後まで読み込む
while(!feof($handle)){
#1行読み込む
$line=fgets($handle);
#1行の文字列を「%」で分解して項目と取り出す
$word=explode("%",$line);
$time=$word[0];
$name=$word[1];
$mail=$word[2];
$title=$word[3];
$message=$word[4];
# １件分のデータを表示する
print <<< HTML
<HTML>
<HEAD>
<TITLE>(^_^)</TITLE>
</HEAD>
<BODY>
<TABLE width="500" cellpadding="10" bgcolor="wheat">
<TR>
<TD>
<B>$time</B> <B>$title</B><BR>
<FONT color="green">$name</FONT>
<IMG src="email.gif" align="middle" hspace="5">
<A href="mailto:$mail">$mail</A><BR>
<BR>
$message
</TD>
</TR>
</TABLE>
<P>
</BODY>
</HTML>
HTML;
}
fclose($handle);
よろしくお願いします。
別のHTML入力画面からデータを取得して、
ファイルに書き込んでいます。
<?php
#データを取得する
$name = $_POST["name"];
$mail = $_POST["mail"];
$title = $_POST["title"];
$message = $_POST["message"];
# 日本語表記の日時を得る
$time = date("Y/m/d l H:i:s");
# データを１行にまとめる（項目の区切りは%）
$data = $time."%".$name."%".$mail."%".$title."%".$message."\n";
# 追加書き込みモードでファイルを開く
$handle = fopen("bbs","a") ;
flock($handle, LOCK_EX);
fwrite($handle,$data);
#ファイルをアンロックする
flock($handle, LOCK_UN);
fclose($handle);
#?>
２）ファイルから読み込むプログラム
１）で書き込んだファイルから1行ずつ読み込み、表示し、
ファイルが終了するまで継続しています。
このとき、1行余分に、読み込まれます。
すなわち、while(!feof($handle))で、1回余分にループしていることになります。
なぜでしょうか。
# 読み込みモードでファイルを開く
$handle = fopen("bbs","r") ;
#ファイルの最後まで読み込む
while(!feof($handle)){
#1行読み込む
$line=fgets($handle);
#1行の文字列を「%」で分解して項目と取り出す
$word=explode("%",$line);
$time=$word[0];
$name=$word[1];
$mail=$word[2];
$title=$word[3];
$message=$word[4];
# １件分のデータを表示する
print <<< HTML
<HTML>
<HEAD>
<TITLE>(^_^)</TITLE>
</HEAD>
<BODY>
<TABLE width="500" cellpadding="10" bgcolor="wheat">
<TR>
<TD>
<B>$time</B> <B>$title</B><BR>
<FONT color="green">$name</FONT>
<IMG src="email.gif" align="middle" hspace="5">
<A href="mailto:$mail">$mail</A><BR>
<BR>
$message
</TD>
</TR>
</TABLE>
<P>
</BODY>
</HTML>
HTML;
}
fclose($handle);
よろしくお願いします。
この質問への意見の募集は締め切られ、ポイントは既に配分されました。
意見を投稿することはできますが、ポイントを受け取ることはできません。
ツリー一覧
A01-1masahiroyamamotoありがとうございました。回答一覧
A01
answerershimix [8月3日 17:55] (最終編集：8月3日 18:00)
1行目を読み込んだ時点では行末の改行で処理を終えているのでまだEOFに達していないと判断されているハズです。つまり「EOFが書かれている2行目がある」のですがそれはまだ不定です（次にEOFがあるかどうかはこの時点ではわかっていません）。feofはEOFに達しているかのチェックですからまだ（1行目を読み終えた時点では）EOFに達していないことになります。
なので
としてやればいいです。
なので
- $line = fgets($handle);
- if ($line === false) { break; }
- if (($line = fgets($handle)) === false) { break; }
としてやればいいです。
A02
answererpannna [8月4日 01:16] (最終編集：8月4日 01:46)
# shimixさんのレスの指摘で事足りるのですが…
ファイルから文字列を読み込んだ際、読み込んだ文字列の長さや
explode関数で分割したそれぞれの文字列の長さをチェックする癖を付けたほうがいいです。
また、デリミタとして使用する記号も記述で使われることがないような文字列のほうがいいですね。
見たところ掲示板のログのようですが、"%"なんかは普通に使われる文字ですし。
あとはHTMLタグのエスケープ処理がないと表示したときにまずいことになるかと。
個人的には掲示板のログはXMLで保存したほうがいいのかな、なんて思います。
ファイルから文字列を読み込んだ際、読み込んだ文字列の長さや
explode関数で分割したそれぞれの文字列の長さをチェックする癖を付けたほうがいいです。
また、デリミタとして使用する記号も記述で使われることがないような文字列のほうがいいですね。
見たところ掲示板のログのようですが、"%"なんかは普通に使われる文字ですし。
あとはHTMLタグのエスケープ処理がないと表示したときにまずいことになるかと。
個人的には掲示板のログはXMLで保存したほうがいいのかな、なんて思います。
ページのトップへ
GETのままでは検索エンジンのロボットが拾ってくれなかったためにSEO対策として有効だと言われていますね。