質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.49%

PHPで掲示板の削除機能をつけたい(指定した番号以外を上書き)

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 923

score 32

数日前に同じ質問をし、一旦は解決しました。
指定した番号以外も削除されるようになってしまいました。
file_put_contents()だと排他ロックが掛けられないとの指摘をうけ、fopenに書き換えました。
その書き方が間違っているせいか、目的が果たせなくなってしまいました。

コードは以下です。(削除部分について)

<?php
/*もしポストで送信されたら以下の処理を行いGET送信でリダイレクトする(更新時の二重書き込み防止)*/
if ( $_SERVER[ 'REQUEST_METHOD' ] === 'POST' ) { /*もしPOSTで送信されたら*/

  /*フォームで送信された値を受け取り、テキストファイルに保存する。
  その際1行ごとに「{番号}<>{名前}<>{コメント}<>{投稿された時間}」と言う形で保存する*/
  /*変数の定義*/
  $filename = "keijiban.txt"; /*保存するファイル*/
  $name = $_POST[ 'name' ]; /*投稿者の名前*/
  $comment = $_POST[ 'comment' ]; /*投稿するコメント*/
  $date = date( "Y-m-d H:i" ); /*投稿日時*/

  /*投稿番号の定義*/
  if ( file_exists( $filename ) ) { /*ファイルの存在確認。*/
  //最後の行にプラス1
      $lines=file($filename);
      $lastline= $lines[count($lines) - 1];
    $num = explode('<>',$lastline)[0]; 
  } else { /*ファイルが無かった場合変数の定義を1とする*/
    $num = 1;
  }
  //書き込み内容
  $hozon = $num . "<>" . $name . "<>" . $comment . "<>" . $date . "\n";

  //投稿部分
  if ( !empty( $_POST[ 'name' ] ) && !empty( $_POST[ 'comment' ] ) ) { /*もし名前とコメントが送信されたら*/
    $fp = fopen( $filename, "a" ); /*追記モードでファイルを開く*/
    if ( flock( $fp, LOCK_EX ) ) { /*ロックする*/
      fwrite( $fp, $hozon ); /*指定したファイルに追記モードで書き込み*/
      flock( $fp, LOCK_UN );
    }
    fclose( $fp );
  }
  //削除部分
  if ( !empty( $_POST[ 'deleteNO' ] ) ) { /*もし、削除番号がポスト送信されたら*/
    $deleteNO = $_POST[ 'deleteNO' ]; /*削除番号の変数定義*/
    $ret_array = file( $filename ); /*ファイルを配列として読み込む*/

    $fp = fopen( $filename, "w" );
    if ( flock( $fp, LOCK_EX ) ) {
      foreach ( $ret_array as $value ) { /*ループ*/
        $bangou = explode( "<>", $value ); /*<>で分割し投稿番号取り出す*/

        if ( $deleteNO != $bangou[ 0 ] ) { /*もし読み込んだ番号と送信した番号が違ったら*/

          fwrite( $fp, $value ); /*ファイルに書き込む*/
          flock( $fp, LOCK_UN ); //ロック開放
        }
        fclose( $fp );
      }
    }

  }

  header( 'Location: keijiban.php' );
  exit;
}
?>
<!doctype html>
<html lang="ja">
<head>
<meta charset="utf-8">
<title>簡易掲示板</title>
</head>

<body>
<p>
<form action="keijiban.php" method="post" >
  <label for="name-field">お名前<span style="color: red;">【必須】</span><br>
  </label>
  <input type="text" name="name" id="name-field" required="required">
  </p>
  <p>
    <label for="comment">コメント<span style="color: red;">【必須】</span></label>
    <br>
    <textarea name="comment" cols="30" rows="3" id="comment" required="required"></textarea>
  </p>
  <input type="submit" value="投稿">
</form>
<?php


$filename = "keijiban.txt";
$ret_array = file( $filename );
foreach ( $ret_array as $value ) {
  $bunkatu = explode( "<>", $value );
  foreach ( $bunkatu as $value2 ) {
    echo $value2 . "<br>\n";
  }
}
?>
<form action="keijiban.php" method="post">
  <P>
    <label for="deleteNO" >削除対象番号</label>
    <input type="text" name="deleteNO" id="deleteNO" >
  </p>
  <input type="submit" value=”削除”>
</form>
</body>
</html>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

ぱっとみ

    $fp = fopen( $filename, "w" );
    if ( flock( $fp, LOCK_EX ) ) {
      foreach ( $ret_array as $value ) { /*ループ*/
        $bangou = explode( "<>", $value ); /*<>で分割し投稿番号取り出す*/

        if ( $deleteNO != $bangou[ 0 ] ) { /*もし読み込んだ番号と送信した番号が違ったら*/

          fwrite( $fp, $value ); /*ファイルに書き込む*/
          flock( $fp, LOCK_UN ); //ロック開放
        }
        fclose( $fp );
      }
    }


foreach処理でflockfcloseをしてはいけないと思う(ifの最後でやるべきでは)

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/09/09 16:23

    書き込む直前ということでしょうか

    キャンセル

  • 2019/09/09 16:27

    flock(解除)とfcloseは基本的にファイル操作終了したときに行うものです
    foreachではファイル書き込み操作をしてるのでflockはまだしも(とはいえしたらだめ)fcloseはしてはいけません

    キャンセル

  • 2019/09/09 16:37

    なるほど。ありがとうございます。やってみます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る