php 掲示板 パスワード設定について

Question

こんにちは。

今、PHPについて勉強中です。

今回、掲示板にパスワードをつけたいです。

行いたいこととしては、
「入力されたパスワードと、書き込み時に保存したパスワードを比較し、一致の場合のみ削除する」
です。

<title>掲示板</title>
<p>
<h1><掲示板></h1>
</p>

<form action ="" method="POST">
名前<br>
<input type ="text" name="name"><br>
コメント<br>
<textarea name = "comment" cols = "30" rows = "5">
</textarea><br>
パスワード<br>
<input type ="text" name="pass"><br>
<p>
<input type = "submit" value = "投稿">
</form>



<form action="" method="post">
削除対象番号<br>
<input type="text" name="delete"><br>
パスワード<br>
<input type ="text" name="delpass"><br>
<p>
<input type="submit" value="削除"><br>
</p>
</form>



<?php

if (isset($_POST["delete"],$_POST["delpass"])) {

$delete = $_POST["delete"];
$delpass = $_POST["delpass"];

$delCon = file("kadai12.txt");

$fp = fopen("kadai12.txt", "w");

for ($j = 0; $j < count($delCon); $j++) {
$delDate = explode("<>", $delCon[$j]);

if ($delDate[4] == $delpass){

if ($delDate[0] != $delete) {
fwrite($fp, $delCon[$j]);
} else {
fwrite($fp, "消去しました。\n");
}
}
}
fclose($fp);
}

?>

あとは以下のコードも試してみました。

<?php

if (isset($_POST["delete"],$_POST["delpass"])) {

$delete = $_POST["delete"];
$delpass = ($_POST["delpass"]);

$delCon = file("kadai12.txt");

$fp = fopen("kadai12.txt", "w");

for ($j = 0; $j < count($delCon); $j++) {
$delDate = explode("<>", $delCon[$j]);

if ($delDate[0] != $delete && $delDate[4] != $delpass) {
fwrite($fp, $delCon[$j]);
} else {
fwrite($fp, "消去しました。\n");
}
}
fclose($fp);
}

?>

削除されてもパスワード関係なく削除されてしまったり、そもそもすべて削除されてしまったりしてうまくいきません。

「kadai12.txt」のサンプルは以下になります。

1<>なお<>test1<>2017年09月28日 11:01:57<>pass1
2<>まき<>test2<>2017年09月28日 11:02:05<>pass2

Comments

[m.ts10806]

kadai12.txtの内容をサンプルで結構ですのでご提示ください。また、teratail内に同様の質問が散見されますので、もしかしたらその中にヒントがあるかもしれません。teratail内でキーワード検索してみてください。

[mochio_o7]

ありがとうございます！提示しました。

[yambejp]

ファイルでのデータ管理なのにlockしてませんが大丈夫でしょうか？DBでのデータ管理は選択肢にはいっていませんか？

Answer 1

こんにちは。
ソースを拝見させていただきました。

まず、fopenのwモードで開くとファイルが空になります。
ですので、１つ目のソースですと、パスワードが一致しない場合、fwriteが実行されないルートを通りますのでファイルが空のままになります。

２つ目のソースは惜しいのですが、↓の書き方ですと
「対象番号が一致しない」と「パスワードが一致しない」の両方を満たすケースのみ
元々あった行がfwriteされます。
逆にいうと、どちらか一方が一致してしまうと消去のルートへ行ってしまいます。

if ($delDate[0] != $delete && $delDate[4] != $delpass) {

つまり、以下と同じ意味になります。

if ($delDate[0] == $delete || $delDate[4] == $delpass) {

そうすると、対象番号ではないにもかかわらず、パスワードが一致さえすれば、その行は消されてしまうわけです。

では、どうすればいいかというと、対象番号とパスワードの両方が一致すれば削除していいので、
以下のif文でよいと思います。

if ($delDate[0] == $delete && $delDate[4] == $delpass){
    // 両方一致すれば削除
    fwrite($fp, "消去しました。\n");
} else {
    // どちらかでも一致しなければ元の行を残す
    fwrite($fp, $delCon[$j]);
}

そして最後に、explodeで配列に変換している処理ですが、
おそらくパスワード(最後の要素)に改行が入ってしまって一致しないのでないかと思いますので、
trimを使ってみてください。

// $delDate = explode("<>", $delCon[$j]);
$delDate = explode("<>", trim($delCon[$j]));

Answer 2

とりあえず一つ目のソースについて言及すると、
掲示板データをせっかくfile()にて配列に取り込んでいるのだから、
配列を操作して削除するべき投稿の要素を配列から削除し、
再度掲示板データを書き出せば良いのではないかと。

簡単な考え方としては、
for()ループで
$delConの要素を点検して、
条件が合致したら配列の塊から要素を削除するarray_splice()でも使って$delConから削除する、
最後にfile_puts_contents("kadai12.txt", implode(PHP_EOL,$delCon));で
配列から改行コードを連結した文字列を作って書き出すことが出来る。
PHP: file_put_contents - Manual

あるいは、array_filter()を駆使してコールバック関数にて削除条件をチェックさせると
for()ループを回さなくても解決可能。
PHP: array_filter - Manual