　ファイルの行を指定して削除したい

ファイルの行を指定して削除したいです。
削除番号を指定して、削除するのですが、指定した番号+1の行が削除されてしまいます。
どの部分が間違えており、どのように修正すれば指定した行が削除するのか教えていただきたいです。

<!DOCTYPE html> <html lang='ja'> <meta charset='UTF-8'> <title>mission3-3</title> <body> <form action='' method='post'> <input type='txt' name='enter' placeholder='内容'> <input type='number' name='delete' min=1 placeholder='削除番号'> <input type='submit' name='submit'> </form> </body>

<?php

$filename = 'mission3-3.txt';
$date = date("Y年m月d日 H時i分s秒");
$num = 0;

if(file_exists($filename)){
    
    if(!empty($_POST['enter']) && !empty($_POST['delete'])){
        
        #投稿番号と削除番号を比較して、等しくなければ追記
        $enter = $_POST['enter'];
        $delete = $_POST['delete'];
        $fp = fopen($filename,"a");
        $lines = file($filename);
        
        $count = count(file($filename));
         
        for($i=0; $i<=$count; $i++){
            
            $line = explode('<>',$filename);
            
            $post_num = $line[0];
            
            if($delete != $post_num){
                
                $num = count(file($filename))+1;
                $total = $num."<>".$enter."<>".$date;
                fwrite($fp,$total."\r\n");
                break;
            }
        }
        fclose($fp);
    }
    
    
    if(empty($_POST['enter']) && !empty($_POST['delete'])){
        
        #削除
        $delete = $_POST['delete'];
      
        $lines = file($filename);
        
        foreach($lines as $line){
            
            $post = explode("<>",$line);
            $post_num = $post[0];
            
            if($delete == $post_num){
                
                unset($lines[$post_num]);
                file_put_contents('mission3-3.txt',$lines);
                
            }else{
                
            }
        }
    }
    
    if(!empty($_POST['enter']) && empty($_POST['delete'])){
        
        #強制追記
    }
    
    }elseif(!file_exists($filename) && !empty($_POST['enter'])){
    
    #新しく生成
    $enter = $_POST['enter'];
    $num = 1;
    $fp = fopen($filename,"a");
    $total = $num."<>".$enter."<>".$date;
    fwrite($fp,$total."\r\n");
    fclose($fp);
}

?>

</html>

退会済みユーザー

2022/07/15 10:09

１行目が0だから、だね。

m.ts10806

2022/07/15 22:50 編集

同じ書籍を使っていると思われる質問は多いですよ。 https://teratail.com/search?q=mission3-3.txt その中の結構な割合で指摘されていることですが、テキストファイルにデータを保存して取り扱う形式は覚えても現場で使えるスキルにはならないですし、コードも冗長になって面倒なだけだと思います。 MySQLやPostgreSQLなどのRDBを使いましょう。書籍もそれらを取り扱っているものを選びましょう。

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

回答2件

$lines = file($filename);でファイルから読み込んでいるなら、
`$lines[0]'が1行目、0から始まるので。

$lines = file($filename);でファイルから読み込んだあとに、
$count = count(file($filename));では、ファイルから2回も読み込んでしまい処理が無駄なので、
せめて
$count = count($lines);で同じ意味になる。

$line = explode('<>',$filename);
これ意味分かんないね。
たぶん
$line = explode('<>',$files[$i]);
じゃないかなぁ。
ファイルから読み込んだデータが１行ずつ$filesという配列に収まっているのだから、
各行のデータにアクセスするのに $files[$i] になり、
１行分のデータを項目ごとに分割するために
$line = explode('<>',$files[$i]);
となるはず。

for($i=0; $i<=$count; $i++){
これも、1行目が$i=0なので、
for($i=0; $i<$count; $i++){
が正しい。
10行あるとしたら、０～９で10行分。

$num = count(file($filename))+1;
これ良くない。
例をあげると、
１、２、３、４、５があって、４を削除すると１，２，３，５
次の番号は $num = count(file($filename))+1; を使うと５になってしまう。
必ず、ファイルの各行の行番号箇所を調べて、最大値を見つけるようにしないと重複するよ。

ファイルに記録している、各行ごとの、行番号との一致で削除したり更新したりするわけなので、
$iで単純に処理しないこと。

forループを回すなら、
$line = explode('<>',$files[$i]);
で１行分のデータをバラして
$num = $line[0];
で行番号が$numに入る。

forachループを回すなら、
foreach($lines as $line){
で１行ずつ$lineに入ってくるので、
$post = explode("<>",$line);
で１行分のデータをバラして
$num = $post[0];
で行番号が$numに入る。

投稿2022/07/15 10:24

退会済みユーザー

総合スコア0