投稿番号を削除する機能を実装したい

Question

前提・実現したいこと

投稿番号の削除機能を実装したい。

ここに質問の内容を詳しく書いてください。
phpで指定した投稿番号を削除しファイルを更新する機能を実装したいです。
投稿番号の削除の機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

Warning: fwrite(): supplied argument is not a valid stream resource in /home/tt-190.99sv-coco.com/public_html/mission_2-3.php on line 68

Warning: fclose(): supplied argument is not a valid stream resource in /home/tt-190.99sv-coco.com/public_html/mission_2-3.php on line 71

該当のソースコード

<?php
//フォームからの受け取り
	$comment =  $_POST['comment'];
	$name = $_POST['name'];
	$delete = $_POST['delete'];
		
	
//ファイルの作成
	$filename = 'mission_2-3_MasayaKokubo.txt';
	$fp_make = fopen($filename,'a');
	$fwrite = ($fp_make);
	$fclose = ($fp_make);

//ファイルへの追記変数
	$fp_add = fopen($filename,'r+');
	
//日付けの変数
	$date = date ('Y年m月d日H時m分s秒');
	
//投稿番号
	$a = 0;
	flock($fp_add,LOCK_EX);
	while(fgets($fp_add) !== false){
			$a++;
	}
	$next = $a +1;
	
	
	$conntent = $next.'<>'.$name.'<>'.$comment.'<>'.$date;
	
	
//テキストファイルへの書き込み	
	if(!empty($comment) and !empty($name)){
		fwrite($fp_add,$conntent.PHP_EOL);
	}
		flock($fp_add,LOCK_UN);
		fclose($fp_add);
		
//ファイルを配列に格納
	$files = file($filename);

//配列を分割しループ
	for($i = 0 ; $i <count($files); $i++){
		$line_ex = explode('<>',$files[$i]);
		echo $line_ex[0]." ";
		echo $line_ex[1]." ";
		echo $line_ex[2]." ";
		echo $line_ex[3]."<br>";
		}

//投稿の削除
	if(!empty($delete)){
		$fp_delete = fopen($filenamea,'w');
		for($j = 0; $j <count($files); $j++){
			$deldate = explode("<>",$files[$j]);
			if($deldate[0] != $delete){
				fwrite($fp_delete,$files[$j]);
			}
		}
	fclose($fp_delete);
	}
?>

Comments

[m.ts10806]

折角エラーメッセージで原因を教えてくれているのだから、まずそのエラーメッセージで調べて、調べた結果試してみてください。おそらくこの内容だと質問するより調べた方が早いですし、もし、調べてやってみた結果解決に至ってないのでしたらその経緯や調べてやってみた内容も質問に記載してください。

[m.ts10806]

あとファイル名はそのままWeb上で公開しても大丈夫なものですか？場合によってはこの質問は削除依頼を出してもらって、修正した内容で新たな質問を立ててもらった方がいいかもしれません。ご検討ください。

[mk77]

ご指摘ありがとうございます。ただ、削除ボタンがどこを見ても発見できません。削除ボタンはどの変にありますか？？基本的な質問ですみません

[m.ts10806]

回答がついた質問は質問から削除依頼が出せませんので運営問い合わせで送ってください。

Answer 1

ちゃんと構造設計してない雰囲気がしますね。

他の方の回答にもあるように、
データをファイルに蓄えるのは複数から同時タイミングでアクセスが有った場合に
いかに排他ロックをかけていたとしてもデータファイルの破損を招きやすいので、
学習のステップとして(リレーショナル)データベースの扱い方を学んでほしいところです。

それは置いといて、データファイルを読み書きして管理する方針を尊重するとします。
どうやらデータファイルには、改行コード PHP_EOL を境目として投稿を区別し、
番号、名前、コメント、日付時刻を「<>」を区切り文字列として連結していると。
当然番号はユニークでないといけないのだけど、
投稿を削除した場合に番号の管理はどうするのだろうかと。
普通は欠番にするよね。
例えば、1,2,3,4,5の番号を使っているときに、
AさんとBさんが観ています、
Aさんが5番を削除した場合に
削除する前から見ていたBさんが投稿を追加したら
Bさんの投稿は6番になってほしいけど、
どうやったら番号の一貫性を保てるのか。
Bさんが投稿したときに5番になっちゃって、
Aさんが削除した5番と違う5番になったりすると混乱のもとになるよね。
こういう細かいところまで配慮して、
例えば番号の発番を投稿データファイルとは別に番号ファイルを持ちましょうって考えてもいいかもしれない。
これが構造の設計。

ついでにいうと、
いわゆる簡易掲示板の話によく出てくる file()って、
ファイル名を与えるだけで配列に読み込んでくれる便利関数なのですが、
fileで
flagsとしてFILE_IGNORE_NEW_LINESを与えることで、
配列に読み込むときに自動的に末尾の改行コードを無視してくれ（除去してくれ）ます。
将来の実装で、
投稿の削除をするときの編集で、うっかり PHP_EOL の二度漬けにならないようにしましょう。

実用性を踏まえた上で検討する話として、
file()とかfile_get_contents()とかの読み込みって、
数百や数千のオーダーの投稿データを溜め込んでいる想定のときに
読み込みの一貫性を確保してくれるのかなぁ。
配列データとして読み込んで、加工して、再度ファイルに書き出すまで、
他の人のアクセスによって既にファイルが改変されている可能性もありうるよね。

それと、フォームの入力欄で、
もしも「<>」を含んだ文字列を入力されたらどうなるだろう。
プログラム内でつける区切り文字「<>」と
入力された文字列「<>」との区別はつかなくなるよね。
ちゃんと対策方法を考えないとね。
入力を受け付けないというのも手だし、
そういう入力があっても扱えるようにデータの持ち方を工夫したりもできる。
これも構造の設計、仕様の決め方次第。

そういう意味においても実用性のない「簡易掲示板」なんだよね、
データの読み込みや編集にかかる処理時間がどのくらいかかるか、
同時アクセスに耐えうるのかって話をすっ飛ばした、
この手の「簡易掲示板」でわかった気になってしまうと、
実用性の低いコードしかかけない人になってしまうかもしれません。

あと、これも構造の設計という点で一言。
投稿されたデータをファイルに保存する、という処理、
投稿データから指定番号の投稿を削除する、という処理、
投稿データを読み出して表示する、という処理、
など、「簡易掲示板」に関わる処理はいくつもありますが、
これ、全部一つのphpで実現しないといけないものなのかどうか、
というところも構造の設計。
自分の理解がまだちゃんと及んでいないうちは、
単機能で一つphpファイルを書く、ってのをやっても良いと思う。
それが複数合わさってもいいので。
一つのphpファイルで複数の処理をこなすこともできるけど、
自分の理解の早さに合わせてやれることをやったらいいと思うよ。

長くてすまん。

Answer 2

ざっとしか追ってませんが、ファイル操作系のコマンドの利用方法がおかしいです。
open lock close の利用方法をおさらいしてください。

余談
個人的には、lock を使用するようなデータを扱うのであれば、データベースを使用したほうが楽だと思います。