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

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

ただいまの
回答率

87.78%

動作が確認できない

解決済

回答 2

投稿 編集

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

score 133

・「投稿」は投稿にチェックかつボタン押下で入力内容を新規に投稿 
・チェックボックスには各投稿のIDをvalueに設定し、ボタン押下で受け取れるようにして 
・「更新」は更新にチェックかつボタン押下で一覧のなかからチェックをつけた投稿に対し、名前・内容の入力値で内容を更新 
・「削除」は削除にチェックかつボタン押下で一覧のなかからチェックをつけた投稿を削除 
という動作にしたいんですが、それぞれブラウザ表示はされるけどボタンを押しても動作確認ができない状態です

調べてPHPコードをif文毎に書けば出力可能とあったので打ってみたんですが、このコードだと「投稿」は出来て、「更新」「削除」は各ボタンが出力されただけで動作は何もありませんでした

調べてみて恐らくフォームではなくPHPコードでおかしい点があるんじゃないか、という疑いはあるんですが、私が調べる限り有力な情報が得られなかったです
どなたかご指摘願います

<?php
    $link = mysqli_connect($host, $user, $password, $dbname) or die("NG");

    mysqli_select_db($link, "XXX");

    if (isset($_POST['name'], $_POST['comment'])) {
        $name = $_POST['name'];
        $comment = $_POST['comment'];
        mysqli_query($link, "INSERT INTO XXX (name, comment) VALUES ('$name', '$comment')");
    }

    if (isset($_POST['update'])) {
        $update = $_POST['update'];
        mysqli_query($link, "UPDATE XXX SET name = '$name', comment = '$comment' WHERE id = $update");
    }

    if (isset($_POST['delete'])) {
        $delete = $_POST['delete'];
        mysqli_query($link, "DELETE FROM XXX WHERE id = $delete");
    }

    $result = mysqli_query($link, "SELECT * FROM XXX");
?>

<html>
    <head>
        <meta http-equiv="content-type" content="text/html; charset=utf-8">
    </head>
    <body>
        <form method="post" action="">
            <table>
                <tr>
                    <td>
                        名前:<input type="text" name="name">
                        内容:<textarea name="comment" cols="30" rows="3"></textarea>
                        <input type="radio" name="bbs" value="post" checked>
                        <input type="submit" name="post" value="投稿">
                        <input type="radio" name="bbs" value="update">
                        <input type="submit" name="update "value="更新">
                        <input type="radio" name="bbs" value="delete">
                        <input type="submit" name="delete" value="削除">
                    </td>
                </tr>
            </table>

<?php
            while($row = mysqli_fetch_array($result, MYSQLI_ASSOC)) {
?>

            <table>
                <input type="checkbox" name="bbs[]" value="<? = $row['$id'] ?>">
                <tr>
                    <td>名前:
                        <?php echo $row['name'] ?>
                    </td>
                </tr>
                <tr>
                    <td>内容:
                        <?php echo $row['comment'] ?>
                    </td>
                </tr>
            </table>

<?php
            };
            mysqli_free_result($result);

            mysqli_close($link);
?>

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+2

なるほど、だいぶ修正してきましたね
この場合ラジオボタンを参照してないですよね?
参照しておいて新規で投稿するというのがちょっと解せませんが
以下の様な感じで組み立てればよいのでは?

<?php
$sql="";
$id      = filter_input(INPUT_POST,'id')?:"";
$name    = filter_input(INPUT_POST,'name')?:"";
$comment = filter_input(INPUT_POST,'comment')?:"";
$post    = filter_input(INPUT_POST,'post')?:"";
$update  = filter_input(INPUT_POST,'update')?:"";
$delete  = filter_input(INPUT_POST,'delete')?:"";
if($post) {
  $sql="INSERT INTO XXX (name,comment) VALUES ('$name', '$comment')";
}
if($id!==""){
  if($update) {
    $sql="UPDATE XXX SET name = '$name', comment = '$comment' WHERE id = $id";
  }
  if($delete) {
    $sql="DELETE FROM XXX WHERE id = $id";
  }
}
print $sql;//ここでSQLを発行する
?>

<form method="post" action="">
<input type="hidden" name="id" value="123456">
名前:<input type="text" name="name" value="hoge">
内容:<textarea name="comment" cols="30" rows="3">fuga</textarea>
<input type="submit" name="post" value="投稿">
<input type="submit" name="update" value="更新">
<input type="submit" name="delete" value="削除">
</form>

ちなみにmysqliの手続き型でSQLに値を埋め込む場合はmysqli_real_escape_string()
など適切にエスケープをしないと文法エラーやセキュリティ問題になります

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/08/09 15:53

    ご回答ありがとうございます

    >ラジオボタンを参照してない
    すいませんが、この意味を教えてもらえますか

    キャンセル

  • 2016/08/09 15:59

    <input type="radio" name="bbs" ・・・>
    で書いてあるラジオボタンを、postしていますが
    $_POST["bbs"]を参照していないので、なんの役にも立っていない・・・
    という意味です。
    今回のケースであればラジオボタンを書かないでください

    キャンセル

  • 2016/08/09 16:31

    せっかくのアドバイスありがたいんですが今回はラジオ~は必須条件としておきたいので引き続き書きたいと思います

    お話の通りだと質問文のコードからだと、ラジオ~を押すことで$_POST[’bbs’]を実行するためのコードが必要という事ですよね

    キャンセル

  • 2016/08/09 16:37

    いや、書いておくのは自由ですが、
    PHP側からなにも使われていないということです。
    今後の仕様拡張を考えているのであれば好きにしてください

    ただ質問をするときに無駄なコードを入れると
    質問の本質にたどり着くのに時間がかかるし
    プログラムの上達にマイナスですよ
    ということだけ念頭においてください

    キャンセル

+1

以下の部分が違うのではないでしょうか?

if (isset($_POST['name'], $_POST['comment'])) {
        $name = $_POST['name'];
        $comment = $_POST['comment'];
        mysqli_query($link, "INSERT INTO XXX (name, comment) VALUES ('$name', '$comment')");
    }


上記のissetの条件を以下のように、$_POST['post']についてもチェックする

if (isset($_POST['post'], $_POST['name'], $_POST['comment'])) {
        $name = $_POST['name'];
        $comment = $_POST['comment'];
        mysqli_query($link, "INSERT INTO XXX (name, comment) VALUES ('$name', '$comment')");
    }

$_POST['name']と$_POST['comment']は、更新、削除時にも存在するため、必ず1個目のif文でtrueとなるため、INSERTしてから update 、 deleteが動くような感じとなるため誤動作しているものと思われます。

こういった場合は、$_POSTの値を比較することが肝要です。
var_dump($_POST); などを入れ、$_POSTの内容が予定しているかどうかを調べながらチェックするデバック方法もあります。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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