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

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

ただいまの
回答率

89.07%

mysqli_query()を削除、更新ともに1回に押さえたい

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 2,189

a-_.

score 133

データベースへのアクセス回数を減らすべく、mysqli_queryでの更新・削除処理を一回にまとめたいと思って、where句にandを追加して下記コードを書いてみたんですが、foreach文では処理できない様で他のループ文を書いてみようと試みてるんですが、他のループ文の条件式をどのように書けばいいのか調べても不明です
ご指摘いただけませんか

$link = mysqli_connect($host, $user, $pass, $dbname) or die("NG");

    mysqli_select_db($link, "XXX");

    if (isset($_POST['name'], $_POST['comment'], $_POST['bbs']) && $_POST['bbs'] == "post") {
        $name = $_POST['name'];
        $comment = $_POST['comment'];
        mysqli_query($link, "INSERT INTO XXX (name, comment) VALUES ('$name', '$comment')");
    } elseif (isset($_POST['bbs']) && $_POST['bbs'] == "update") {
        foreach($_POST['chkid'] as $id) {
            $name = $_POST['name'];
            $comment = $_POST['comment'];
            $update = $id;
            $delete = $id;
            mysqli_query($link, "UPDATE XXX SET name = '$name', comment = '$comment' WHERE id = $update and DELETE FROM XXX WHERE id = $delete");
        }
    }

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

<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="radio" name="bbs" value="update">更新
                        <input type="radio" name="bbs" value="delete">削除
                        <input type="submit" value="投稿">
                    </td>
                </tr>
            </table>

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

            <table>
                <input type="checkbox" name="chkid[]" 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ベストアンサー

+1

DB側に投げるSQLステートメント自体が誤ってます。

AND句を用いてのINSERT文、UPDATE文、DELETE文を連結できません。
そのためSQL構文エラー(MySQL側によるエラー通知)となっているものと思われます。

maisumakunさんのおっしゃられるように、UPDATE句とDELETE句を同時に実行するメリットより実装のややこしさなどによるデメリットの方が大きいので、
それぞれ1文ずつでmysqli_queryでするようにすると良いでしょう。

補足

ロジック誤りは既に指摘されてるので僕からはそれ以外で2点ほどアドバイス。

先ずは1点目。
INSERT、UPDATE、DELETEの実行時のmysqli_query戻り値は、
SQL実行が正常終了したらTRUE失敗(SQL実行時エラーなど)したらFALSEを返却するので、

SQL実行が成功したか判定するための処理を加えることをお勧めします。

以下は利用例

$success = mysqli_query($link, $sql) // INSERT、UPDATE、DELETEの場合

    if (!$success) {
        // SQL実行エラー時の処理
    }

最後に2点目。
INSERT、UPDATE、DELETEでは、
更新・削除件数が0件であってもSQL実行が正常であればTRUEを返却するので、
例えば指定したIDでの更新件数が0件の場合は想定外としたい場合は、
mysqli_affected_rows関数の利用をお勧めします。

以下は利用例

$success = mysqli_query($link, $sql) // INSERT、UPDATE、DELETEの場合

    // 1つ目のサンプルと判定逆ににしてるので注意
    if ($success) {
        // SQL実行成功時
        if (mysqli_affected_rows( mysqli $link ) === 0) {
            // 更新件数0件の場合はエラーとするならここに処理記載
        }

    } else {
        // SQL実行エラー時の処理
    }

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/08/16 13:30

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

    質問投稿後も調べて確かにAND句はふさわしくないことが分かりました
    改めてコードを書き直し、AND句の代わりにIN演算子を用いてIDの特定をやってみようかと思いました
    下記コードはIN演算子を用いたコードなんですが、Notice: Undefined index: idと表示され、IDをどう式に反映させればデータベースのIDと認識させられるのかが分かりません

    } elseif (isset($_POST['bbs']) && $_POST['bbs'] == "update") {
    foreach($_POST['chkid'] as $id) {
    $name = $_POST['name'];
    $comment = $_POST['comment'];
    $id = $_POST['id'];
    mysqli_query($link, "UPDATE XXX SET name = '$name', comment = '$comment' WHERE id in('$id')");
    }
    } elseif (isset($_POST['bbs']) && $_POST['bbs'] == "delete") {
    foreach($_POST['chkid'] as $id) {
    $id = $_POST['id'];
    mysqli_query($link, "DELETE FROM XXX WHERE id in('$id')");

    キャンセル

+1

WHERE id = $update and DELETE FROMなんて書き方はできません。セミコロンで区切れば複数の分を並べることもできますが、複数の文を一気に実行するには専用の書き方が必要となって、余計にややこしくなります。

似たようなSELECTを大量に発行する状況から「削減したい」のならともかく、UPDATEDELETEを一気にやることにそこまでメリットはないと考えます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/08/16 11:23

    複数列に対して同じ更新をかける場合、WHERE id IN (1, 2, 3, 4)のような方法があります。

    別件ですが、元のコードではエスケープなどが不足しているので、変なデータをPOSTされれば一撃でSQLインジェクションの被害に遭います。

    キャンセル

  • 2016/08/16 11:37

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

    メリットデメリットでコードを打つというよりかは、本来であれば行う必要はないのでしょうけどやり方が知りたいのでお聞きしました

    WHERE id INとループ文ではどう処理が異なるのでしょうか

    キャンセル

  • 2016/08/16 11:51

    > a-_.さん
    実際にそういうコードを書く機会がなかったので断言はできないですが、
    複数クエリをまとめて実行する機能という意味ではmaisumakunさんの掲載リンクが参考になるのではないでしょうか?

    またINを使ったものとforによるループの違いですが、
    これこそ正にデータベースとのアクセス回数に違いがでます。

    INリストのパラメタとしてIDリストを引き渡しした場合は、1回の通信でリストに指定したIDに対応するデータを更新・削除できます。

    キャンセル

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

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

関連した質問

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