php web掲示板で削除機能を実装したい。

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
phpでweb掲示板を作っています。
削除機能を実装中に以下のエラーメッセージが発生しました。

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

php掲示板で削除機能を実行したいです。
Notice: Undefined variable: dataFile in /public_html/mission_3-3.php on line 39
Warning: Invalid argument supplied for foreach() in /public_html/mission_3-3.php on line 39

該当のソースコード

php
1<html>
2<body>
3         <form action = "mission_3-3.php" method="post">
4              名前：
5              <input type="text" name="name"><br/>
6
7              コメント：
8              <input type="text" name="comment"><input type="submit" value="送信" ><br/>
9              
10              削除対象番号：
11              <input type="text" name="deleteNo" placeholder="削除対象番号">
12              <input type="submit" name="delete" value="削除">         	
13        </form>
14
15 <?php
16  $filename = "mission_3-3.txt";
17  if(!empty($_POST["name"])) {
18
19  $fp = fopen($filename,'a');
20  $dataFile = explode(" ",$filename) ;
21  fwrite($fp,count(file($dataFile[0]))."<>".$_POST["name"]."<>".$_POST["comment"]."<>".date( "Y年m月d日  H:i:s" ));
22  fwrite($fp,"\n");
23  fclose($fp);
24  }
25 
26
27$fp = fopen("mission_3-3.txt", "r");
28while ($line = fgets($fp)) {
29  $line2 = explode("<>",$line);
30  print_r($line2[0]." ".$line2[1]." ".$line2[2]." ".$line2[3]);
31  print_r("<br>");
32}
33fclose($fp);
34
35
36$delete = $_POST['deleteNo']; 
37if(!empty($delete)){
38    $fp=fopen($filename,'w'); 
39    foreach( $dataFile as $key => $value){
40        $line = explode("<>",$value);
41        if($line[0] != $delete){
42            fwrite($fp,$line[0]." ".$line[1]." ".$line[2]." ".$line[3]);
43}
44
45}
46fclose($fp);
47}
48?>
49</table>
50</body>
51</html>

試したこと

配列などがよく分からず躓いています。

補足情報（FW/ツールのバージョンなど）

退会済みユーザー

2019/10/31 02:20

出典はどちらのものでしょうか？

行動規範の内容に同意します

回答3件

基本を確認したければPHPマニュアル

あとエラーは「何が起きたか」「どこがおかしいか」教えてくれるものなので読めば分かることがほとんど。
つまり、読もう。

エラーメッセージの読み方と対処, 検索や質問の原則

投稿2019/10/31 02:28

m.ts10806

総合スコア80875

ベストアンサー

if(!empty($_POST["name"])) {のブロック内で
$dataFile = explode(" ",$filename) ;と定義していますが、
このifの条件に当てはまらない場合には$dataFileは未定義となります。
そのために表示されているエラーと考えられます。

対策としては、if()ブロックの外で$dataFileを定義すればよいです。

それはそれとして。
あれこれおかしなところがあるので順に。

php
1    $filename = 'mission_3-3.txt';
2    if (!empty($_POST['name'])) {
3        $fp = fopen($filename, 'a');
4        $dataFile = explode(' ', $filename);
5        fwrite($fp, count(file($dataFile[0])).'<>'.$_POST['name'].'<>'.$_POST['comment'].'<>'.date('Y年m月d日  H:i:s'));
6        fwrite($fp, "\n");
7        fclose($fp);
8    }

投稿を受け付けてデータファイルに記録する箇所だと推察するのだけど、
投稿番号の発番アルゴリズムが雑すぎて困る。

投稿を削除できる機能を追加する前提で、投稿データをどう管理するかを整理すると、
投稿データに削除フラグの項目をもたせて、通常は「未削除」削除されたら「削除済」を示すデータを記録する【論理削除】の方法もあるが、
投稿データから抹消して残さない【物理削除】の方法があるわけ。こっちだろう。

物理削除を繰り返すと、投稿番号が途中削除によって抜けることも出てくるわけで、
投稿データの読み込んだ行数を元にうっかり発番すると番号が重複することもある。
例を挙げると、１，２，３，４，５とあるところで、３を削除すると１，２，４，５になるが、
次の投稿番号は行数を元にすると５になってしまって重複してしまう、など。

物理削除で管理する場合は、必ず投稿データの各行を検索して、投稿番号の最大値＋１を求めて使うのが原則になる。
やり方はいろいろあるけど、例えば、
投稿データを行単位で一旦配列に持たせて、
foreach()ループでその配列を１行ずつ、
投稿番号箇所を切り出して最大値を調べるのが無難だろうか。
投稿データが何千行などになったらもたないかもしれないが、
学習用途としては十分なのではないかと。
机上でバッグのみで動かしてないけど、こんな雰囲気。

php
1$max_no = 0;
2$dat = file($filename);
3foreach ($dat as $line) {
4    list($no, $name, $comment, $date) = explode("<>", $line);
5    if ($max_no > intval($no)) {
6        $max_no = intval($no);
7    }
8}
9$max_no += 1;  // 次の投稿番号

投稿の追加はファイルへの「追加書き出し」でもいいし、
せっかく配列に格納したのなら、配列末尾に追加したものをファイルに書き出すのもいい。

それと、投稿の追加や削除に関して、$_POSTを検証をもう少し丁寧にやりたい。
名前、コメント、削除対象番号、
それぞれどういう入力を許可・許容するのかをはっきりさせておきたい。
名前だけ入力があってコメントがカラだった場合、
$_POST['comment']は未定義になるため、
未定義の変数を参照したりするとNotice: Undefined index: comment in $_POSTなんてエラー表示が起こりうる。
投稿の追加をするには、名前とコメントの入力が必須、
であるならば、
if (isset($_POST['name']) && isset($_POST['comment']) {
みたいになるだろうか。
なお、変数の存在確認にはempty()を使ってはならない。isset()推奨。
PHP isset, empty, is_null の違い早見表 - Qiita
文字列"0"をempty("0")はtrue判定してしまうため。