Q&A
できればコードはなるべく全て、コピペで再現できるものをご提示いただけますか？
PHP初心者でする
掲示板の削除機能を付けるところです。
番号を指定して送信し、
送信された番号と、書き込まれた投稿番号とを比較し、
異なれば上書きする。
ということコードを書きたいです。
色々修正してみたもののうまくいかず、
現在の状況は、指定した番号の以外も消え、一つだけ投稿が残ります。
どのような規則で残っているのかよくわかりません。
どこが間違っているのでしょうか。
以下が書いてみたコードです。
投稿部分
php
1<?php 2/*もしポストで送信されたら以下の処理を行いGET送信でリダイレクトする（更新時の二重書き込み防止）*/ 3if ( $_SERVER[ 'REQUEST_METHOD' ] === 'POST' ) { /*もしPOSTで送信されたら*/ 4 5 /*フォームで送信された値を受け取り、テキストファイルに保存する。 6 その際1行ごとに「{番号}<>{名前}<>{コメント}<>{投稿された時間}」と言う形で保存する*/ 7 /*変数の定義*/ 8 $filename = "keijiban.txt"; /*保存するファイル*/ 9 $name = $_POST[ 'name' ] ; /*投稿者の名前*/ 10 $comment = $_POST[ 'comment' ] ; /*投稿するコメント*/ 11 $date = date( "Y-m-d H:i" ); /*投稿日時*/ 12 13 /*投稿番号の定義*/ 14 if ( file_exists( $filename ) ) /*ファイルの存在を確認*/ { /*真ならファイルの配列の行をカウントし、プラス１を変数の定義とする*/ 15 $count = count( file( $filename ) /*ファイルを配列として読みこむ*/ ) /*配列を数える*/ + 1; 16 } else { /*ファイルが無かった場合変数の定義を１とする*/ 17 $count = 1; 18 } 19 $hozon = $count . "<>" . $name . "<>" . $comment . "<>" . $date . "\n"; 20 if ( !empty( $_POST[ 'name' ] ) && !empty( $_POST[ 'comment' ] ) ) { /*もし名前とコメントが送信されたら*/ 21 file_put_contents( $filename, $hozon, FILE_APPEND ); /*指定したファイルに追記モードで書き込み*/ 22 } 23 if ( !empty( $_POST[ 'deleteNO' ] ) ) { /*もし、削除番号がポスト送信されたら*/ 24 25 $deleteNO = $_POST[ 'deleteNO' ] ; /*削除番号の変数定義*/ 26 $ret_array = file( $filename ); /*ファイルを配列として読み込む*/ 27 28 foreach ( $ret_array as $value ) { /*ループ*/ 29 $value4 = explode( "<>", $value ); /*<>で分割し投稿番号取り出す*/ 30 31 if ( $deleteNO != $value4[ 0 ] ) { /*もし読み込んだ番号と送信した番号が違ったら*/ 32 33 file_put_contents( $filename, $value ); /*ファイルに書き込む*/ 34 35 } 36 } 37 38 } 39 40 header( 'Location: keijiban.php' ); 41 exit; 42} 43?> 44<!doctype html> 45<html lang="ja"> 46<head> 47<meta charset="utf-8"> 48<title>簡易掲示板</title> 49</head> 50 51<body> 52<p> 53<form action="keijiban.php" method="post" > 54 <label for="name-field">お名前<span style="color: red;">【必須】</span><br> 55 </label> 56 <input type="text" name="name" id="name-field" required="required"> 57 </p> 58 <p> 59 <label for="comment">コメント<span style="color: red;">【必須】</span></label> 60 <br> 61 <textarea name="comment" cols="30" rows="3" id="comment" required="required"></textarea> 62 </p> 63 <input type="submit" value="投稿"> 64</form> 65<?php 66 67 68$filename = "keijiban.txt"; 69$ret_array = file( $filename ); 70foreach ( $ret_array as $value ) { 71 $value2 = explode( "<>", $value ); 72 foreach ( $value2 as $value3 ) { 73 echo $value3 . "<br>\n"; 74 } 75} 76?> 77<form action="keijiban.php" method="post"> 78 <P> 79 <label for="deleteNO" >削除対象番号</label> 80 <input type="text" name="deleteNO" id="deleteNO" > 81 </p> 82 <input type="submit" value=”削除”> 83</form> 84</body> 85</html>
回答2件
0
ベストアンサー
file_put_contents()で毎回上書きしているからです。
ファイルを読み込んだ時点でファイルを削除し、file_put_contents()の第3引数でFILE_APPENDを指定し、追記で書き込んでください。
投稿2019/09/04 07:27
総合スコア7914
追記で書き込んでみると、既存のものに加えてさらに同じものが追記されてしまいました。
回答に記載している通りファイルを読み込んだ時点でファイルを削除してください
申し訳有りません。
しっかり読まないとだめですね。
ありがとうございます。
m6uさんの回答にある通り、
投稿番号については出力するときに振り直しが必要になるので別途対応ください。
ありがとうございました。
タイトルでのやりたいことは無事解決しました。
これからm6uさんのアドバイスに基づき、色々手直しをしていきます。
0
まずはじめに、htmlspecialchars()の使い方が間違っています。
「何故htmlspecialcharsを通すのか？」を一言でどうぞ - Qiita
データとして保持する文字列（ファイルに書き出す文字列）は
htmlspecialchars()加工前の文字列で扱い、
htmlとして出力する場合に直前にhtmlspecialchars()で臨時に加工するのがセオリーです。
但し、今回のデータファイル上で、データの区切りに「<>」を使っているため、
もしも投稿POSTデータに「<>」が含まれていたらどうするか、
拒否するのか、
含まれていても差し支えないように置き換えるか、
の工夫は絶対必要です。
投稿番号を管理する上で、$countを使用していますが、
例えば5番まで使っている時に3番を削除したら、
本来1,2,4,5の行がある状態になりますよね。
でもこのとき$countは行数4に対して1を加えて5にしてしまうと、
投稿番号5番がダブることになります。
なので、投稿データの順序が入れ替わることがないのであれば、
必ず最後の行の投稿番号が最大値になるはずなので、
「最後の行の投稿番号＋１」を使うのも手ですし、
あるいは、
別途投稿番号発番用データファイルを用意して、
投稿番号を払い出したら＋１して保存するなどすればよいです。
やっと削除部分の処理の評価ですが、
変数名がわかりにくすぎます。
変数名を見て、それがなんの目的のものなのかがピンとこないものは
見直すべきです。
$valueと$value4って何がどう違うのか、パット見で把握できないでしょ。
削除処理foreachループにて、
投稿番号が指定の番号と違ったらファイルに書き出す、のもいいですが、
元のデータファイルを残したまま「追記モード」で書き出すと、
それだけでデータのダブリを生みますね。
読みだした直後にファイルを削除するのが妥当でしょう。
あるいは、
例えば書き出すのを後回しにして、書き出し用配列を用意して、
書き出す配列をもとに改行コードで連結してから上書きで書き出す、
という技でもできそうです。
web上のシステムなので、実際に複数の人が使い始めると、
ほぼ同じタイミングで投稿されることも起こりえます。
file()やfile_get_contents()やfile_put_contents()などの簡易ファイルアクセス関数は
ファイルの排他ロックがかからないため、
タイミングによってはデータファイル破損を招きます。
ファイルでやりきるならfopen()～fgets()～fputs()～fclose()といっしょに
flock()にて排他ロックを行い、
ロック中は待ち合わせするような工夫も必要です。
簡易掲示板だからそこまで求めないと考えるのか、
簡易掲示板だけどしっかり作り込むかは考え方次第です。
Aさんが削除のアクションを、Bさんが投稿のアクションを同じタイミングで起こしたとき、
削除処理でファイルを削除したところで投稿の処理が走ると、
データが消えるか、投稿が無視されるか、
毎度実行しても結果がバラバラな動作になりかねません。
投稿2019/09/04 07:24
編集2019/09/04 07:59
退会済みユーザー
総合スコア0
ありがとうございます。
投稿番号の管理について、ご指摘の通りの現象が起こっていましたので、後で考えようと思っていたところでした。
htmlspecialchars()についても勉強しておきます。
色々とアドバイスありがとうございます。
気をつけるべきところがたくさんありますね。
根気よく直していきます。
ファイルの書き込みも、アドバイス通り書き直したいと思います。
非常に有用なアドバイスありがとうございます。
勉強になります。
file_put_contents()でも排他ロックかけられるものだと思っていました。
ご指摘を受けて検索してみたら以下の記事を見つけました。
http://kannokanno.hatenablog.com/entry/20120624/1340532542
本当に色々ありがとうございました。
あなたの回答
tips
プレビュー
