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

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

ただいまの
回答率

88.92%

php 簡易掲示板 削除機能について

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,694

dongu0223

score 11

今phpで簡易掲示板の作成をしています。
削除機能を付けたいのですが、
現状として

①「削除」の送信ボタンを二回押さないと更新されない
②入力フォームの下に表示されるのが、最新のフォームのみ
(本当はテキストファイルに書き込まれている内容全部を表示したい)

となっています。
改善のアドバイスお願いします!

<html>
 <html lang = "ja">
<head>
  <meta charset = "utf-8">
</head>



<body>
 <form method= "post" action="mission_2-3.php">

    <input type="text" name="name"><br>
     <input type="text" name="comment">
     <input type="submit"><br>
<!--削除用の入力フォームを設置-->
<input type="text" name="delete" value="削除対象番号">
    <input type="submit" value="削除"><br>
</form>

<?php
    $filename="mission_2-1_shima.txt";
//入力フォームのデータを受け取る
    $name= ($_POST['name']);
    $comment = ($_POST['comment']);
    $delete= ($_POST['delete']);
//日付データの取得
    $date=date('Y年m月d日 H:i:s');
//投稿番号の取得
//テキストファイルにデータがない場合は1を指定
//テキストファイルにある場合は+1
if (file_exists($filename)){
    $num=count(file($filename))+1;
}
else{
    $num=1;
}

//まとめた変数
    $newdate= $num."<>".$name."<>".$comment."<>".$date;

//もしフォームが空じゃなかったら書き込みをする
if (!empty($name) && !empty($comment)){
    $fp = fopen($filename,'a'); 
    fwrite($fp, $newdate."\n"); 
fclose($fp);
}
//削除番号に数字が入力された場合
$files=file('mission_2-1_shima.txt');

if(!empty($delete)){//条件分岐①
//ファイルの中身を空にする
    $fp=fopen($filename,'w');
// ファイルを0に丸める
    ftruncate($fp,0);
// ファイルポインタを先頭に戻す
    fseek($fp,0);

foreach ($files as $file){//ループ処理①
    $words=explode("<>",$file);

if($words[0] != $delete){//条件分岐②
    fwrite($fp,$file);

}}//条件分岐②とループ処理②おわり

    fclose($fp);    

}//条件分岐①おわり

echo "{$words[0]}{$words[1]}{$words[2]}{$words[3]}{$words[4]}";
echo "<br>";




?>
</body>
</html>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2018/12/28 09:49

    ひとまずインデントをきちんとつけてみてください。
    おかしいところがある程度見えてくるはずです。

    キャンセル

  • papinianus

    2018/12/28 10:11

    //削除番号に数字が入力された場合、のところで削除前のデータを全部取得して、//ループ処理①で削除を判定する前に全件必ず$wordに保存していることを、削除する前にセレクトと仰っているものと思います。端的に$wordへの代入をfwriteと同じ位置でやればよいのでは?

    キャンセル

  • m.ts10806

    2018/12/28 10:22

    作りがそもそも微妙なので「私ならこう作る」という感じで簡易雛形コードを回答に追記しました。
    処理部分は書いていませんのでご自身で埋めるようにしてください。

    キャンセル

回答 2

checkベストアンサー

+1

削除と登録 formを別けては。

 <form method= "post" action="mission_2-3.php">

    <input type="text" name="name"><br>
     <input type="text" name="comment">
     <input type="submit"><br>
</form>
<!--削除用の入力フォームを設置-->
 <form method= "post" action="mission_2-3.php">
<input type="text" name="delete" value="削除対象番号">
    <input type="submit" value="削除"><br>
</form>

それか、ボタンをinputではなくbuttonタグにして、nameとvalueをつけて分岐させる

 <form method= "post" action="mission_2-3.php">

    <input type="text" name="name"><br>
     <input type="text" name="comment">
     <button type="submit" name="mode" value="add">登録</button><br>
      <input type="text" name="delete" value="削除対象番号">
     <button type="submit" name="mode" value="delete">削除</button><br>
</form>
if($_SERVER['REQUEST_METHOD'] === 'POST'){
  $sendmode = $_POST["mode"];
  switch($sendmode){
    case "add":
     // 登録処理
     break;
    case "delete":
     // 削除処理
     break;

  }
}

//取得処理


//表示処理

雛形コード

私ならこういう風に作る という形の雛形を作りました。
それぞれ何をやっているか分からなければ調べてください。
出来上がっていない段階ではエラー表示は常時Onにしてどのようなエラー(変数未定義など警告レベルのものも含む)がおきても確認し、対応するようにしてください。

<?php
ini_set('display_errors', "On");//エラー表示強制ON 開発終了したら削除またはコメントアウトする

$list = ""; //登録した情報表示用の変数初期化

//POST送信されたときのみ処理を行う
if($_SERVER['REQUEST_METHOD'] === 'POST'){
    if(array_key_exists("mode", $_POST)){
        $sendmode = $_POST["mode"];
        switch($sendmode){
            case "add"://登録
                //必須項目など入力チェック処理

                // 登録処理

                break;
            case "delete"://削除
                //必須項目など入力チェック処理

                // 削除処理
                break;
            default:
        }
    }
}

//取得処理 $listに追記していく

?>
<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>Insert title here</title>
</head>
<body>

</body>
</html>
<form method="post">
    <input type="text" name="name" placeholder="名前"><br />
    <input type="text" name="comment" placeholder="コメント"><br />
    <button type="submit" name="mode" value="add">登録</button>
</form>
<form method="post">
    <input type="text" name="delete" value="" placeholder="削除対象番号"><br />
    <button type="submit" name="mode" value="delete">削除</button>
</form>
<?php echo $list?>
</body>
</html>

注意点:

  1. 登録処理や削除処理をしてからデータを取得すること。現在のコードだとdeleteのときしかデータ取得して内容に見受けられる。一緒にしない。
  2. 入力内容のチェックを行って必須項目や文字数オーバー(ここは仕様)のときはエラーメッセージを出すようにして登録処理・削除処理を行わないこと
    nameとかcommentに<>って入ってたらどうする?対策を考えましょう
  3. 入力された情報を画面に出力する際、HTMLエスケープを行うこと

Qiitaに初心者向けの記事もあがっているので、そちらも参考にしてください。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/12/28 09:50

    formを分けるほうをやってみたのですが、
    表示がなくなったのはどうすればいいですか?

    キャンセル

  • 2018/12/28 09:58 編集

    「表示がなくなった」の意味が分からないですね。
    エラーは出ていませんか?
    PHPファイル冒頭に下記を追加してみてください。
    ini_set('display_errors', "On");

    まあ、望ましいのは「それか、」で書いた案2の方ですが。
    処理がゴチャゴチャになって分かりづらいので。

    「やってみた」のはいいですが、本当にこちらの想定通りに対応したか分からないので、
    質問本文にコードを追記してください。

    キャンセル

  • 2018/12/28 10:06

    というか、この内容だとundefined index の警告は沢山でますね。

    キャンセル

0

①formを分けた
②表示機能のところでループ処理して$wordsを定義した

で直りました。

if (file_exists($filename)){
    $num=count(file($filename))+1;
}
else{
    $num=1;
}

//まとめた変数
    $newdate= $num."<>".$name."<>".$comment."<>".$date;

//もしフォームが空じゃなかったら書き込みをする
if (!empty($name) && !empty($comment)){
    $fp = fopen($filename,'a'); 
    fwrite($fp, $newdate."\n"); 
fclose($fp);
}
//削除番号に数字が入力された場合
$files=file('mission_2-1_shima.txt');

if(!empty($delete)){//条件分岐①
//ファイルの中身を空にする
    $fp=fopen($filename,'w');
// ファイルを0に丸める
    ftruncate($fp,0);
// ファイルポインタを先頭に戻す
    fseek($fp,0);

foreach ($files as $file){//ループ処理①
    $words=explode("<>",$file);


if($words[0] != $delete){//条件分岐②
    fwrite($fp,$file);

}}//条件分岐②とループ処理②おわり

    fclose($fp);    

}//条件分岐①おわり


//表示機能
$files=file('mission_2-1_shima.txt');
foreach ($files as $file){
    $words=explode("<>",$file);
    echo "{$words[0]}{$words[1]}{$words[2]}{$words[3]}{$words[4]}";
    echo "<br>";
}


?>
</body>
</html>

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/12/28 11:02

    コメントでも別途指摘しましたけど、インデント きちんとしないとバグったときに原因究明に影響しますよ。コードも読みづらいですし

    キャンセル

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

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

関連した質問

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