PHP 掲示板 削除方法について

Question

php初心者です。
phpにて、簡易掲示板で削除機能を実装する方法を教えてほしいです。

行いたいことは、
フォームより値が送信された場合のみ削除番号を送信して処理する
explodeを使って投稿番号を取得する
各投稿番号と削除番号を比較して、イコールじゃない時だけテキストに上書き保存する
です。

削除用フォーム

<form action="12.php" method="post">
削除対象番号<br>
<input type="text" name="delete"><br>
<input type="submit" value="削除"><br>
</form>

---

12.php

<?php
if (isset($_POST["delete"])) {
$delete = $_POST["delete"];
$delCon = file("11.txt");

$fp = fopen("11.txt", "w");
fwrite($fp, "");
fclose($fp);

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

if ($delDate[0] != $delete) {
$b = fopen("11.txt", "a");
fwrite($b, $delCon[$j]);
fclose($b);
} elseif ($delDate[0] == $delete) {
$c = fopen("11.txt", "a");
fwrite($c, "消去しました。");
fclose($c);
}
}
}
?>  

上記のプログラムで行ったものの削除されません、、
どうしてでしょうか、、

Answer 1

ファイルのフォーマットが1行＝1データで、デリミタが<>で、1列目が番号を表しているのであれば、問題ないように思えます（こういった条件は質問文に記載しておきましょうね）
という前提で他に気になる点を書いておきます。

書き込む都度、オープン・クローズは不要です。
ループ前にオープンし、最後に閉じれば良いです。

array_spliceを使う意味がわかりません。
見た感じ不要と思えます。
比較している箇所でelsifにする必要性はありません。
elseで良いです。

"消去しました。"には改行コードが必要かと。

上記を考慮し修正したコードを載せておきます。

<?php
if (isset($_POST["delete"])) {
	$delete = $_POST["delete"];
	$delCon = file("11.txt");

	$fp = fopen("11.txt", "w");
	for ($j = 0; $j < count($delCon); $j++) {
		$delDate = explode("<>", $delCon[$j]);
		if ($delDate[0] != $delete) {
			fwrite($fp, $delCon[$j]);
		} else {
			fwrite($fp, "消去しました。\n");
		}
	}
	fclose($fp);
}
?>  

Comments

[mochio_o7]

回答ありがとうございます！
分かりやすくて助かります(:_;)

Answer 2

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

で取ってるって言うことは一行一データと思いますが、

array_splice($delDate,$j,1);

$delDateの$j番目のデータを取る理由がちょっとわかりません。

あと、ファイル毎回開くのはあまりよろしくないので

<?php
if (isset($_POST["delete"])) {
	$delete = $_POST["delete"];
	$delCon = file("11.txt");

	$fp = fopen("11.txt", "w");
	for ($j = 0; $j < count($delCon); $j++) {
		$delDate = explode("<>", $delCon[$j]);
		array_splice($delDate,$j,1);

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

こんな感じでまとめたほうがいいかも

Comments

[mochio_o7]

回答ありがとうございます！

なるほど、、まとめられるのですね。

実行してみたところデータがすべて消えてしまったのですが、指定した番号の行だけ削除する方法はどうすればよいのでしょう、、

[rururu3]

array_splice($delDate,$j,1);
をする意味を考えてみてください