🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

解決済

2回答

881閲覧

SQLで2回INSERTしてしまう

Mia.K

総合スコア5

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

0グッド

0クリップ

投稿2021/01/23 08:00

前提・実現したいこと

PHPを学習中です。会員同士が使える掲示板機能を作っています。

発生している問題・エラーメッセージ

メッセージを送ろうとpost送信すると2回INSERTが実行されてしまいます。
エラーメッセージはありません。

該当のソースコード

//問題のコードがこちらです try { $dbh = dbConnect(); $sql = 'INSERT INTO message (board_id, send_date, to_user, from_user, msg, create_date) VALUES (:board_id, :send_date, :to_user, :from_user, :msg, :create_date)'; $data = array( ':board_id' => $m_id, ':send_date' => date('Y-m-d H:i:s'), ':to_user' => $partnerUserId, ':from_user' => $_SESSION['user_id'], ':msg' => $msg, ':create_date' => date('Y-m-d H:i:s') ); $stmt = queryPost($dbh, $sql, $data); if ($stmt) { $_POST = array(); header("Location:" . $_SERVER['PHP_SELF'] . '?m_id=' . $m_id); exit(); } } catch (Exception $e) { error_log('エラー発生:' . $e->getMessage()); $err_msg['common'] = MSG07; } //DBへの接続を関数に切り出しています。 function dbConnect() { $dsn = 'mysql:dbname=market;host=localhost;port=8889;chars=utf8mb4'; $user = 'root'; $password = 'root'; $options = array( PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION, PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC, PDO::MYSQL_ATTR_USE_BUFFERED_QUERY => true, ); $dbh = new PDO($dsn, $user, $password, $options); return $dbh; } //クエリ実行を関数に切り出しています。 function queryPost($dbh, $sql, $data) { $stmt = $dbh->prepare($sql); $stmt->execute($data); if (!$stmt->execute($data)) { return false; } return $stmt; } //HTMLのフォームタグです <form action="" method="post"> <textarea name="msg" cols="30" rows="3"></textarea> <input type="submit" value="送信" class="btn btn-send"> </form>

試したこと

クエリが2回実行されてるからでは、と思い確認し
コードを変えてみたりもしましたが自分では原因がわかりませんでした。
考えられる原因を教えていただけないでしょうか。

補足情報(FW/ツールのバージョンなど)

ブラウザはchromeを使っています

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

2回executeしているからじゃないでしょうか。

php

1 2function queryPost($dbh, $sql, $data) 3{ 4 $stmt = $dbh->prepare($sql); 5 $stmt->execute($data); // <-- ココ 6 7 if (!$stmt->execute($data)) { // <-- ココ 8 return false; 9 } 10 return $stmt; 11}

投稿2021/01/23 08:06

umau

総合スコア831

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Mia.K

2021/01/23 08:43

早速ここをコメントアウトしてみたところ直りました!ありがとうございます。先頭に!をつけていればexecuteしていない時~となると勘違いしておりました。
guest

0

アクセスログを確認してみてください。
一度の更新作業で 2 回アクセスしている可能性があります。

ブラウザの拡張機能で 2 回アクセスしている現象を過去に何件か質問として見ました。
その場合、ブラウザの拡張機能を切ることで正常に戻ります。
Token 型の多重投稿対策でも対応できるはずです。

投稿2021/01/23 08:11

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Mia.K

2021/01/23 08:41

ありがとうございます。原因はやはり2回executeしていました。。おかげさまで問題解決の引き出しが増えました。次同じようなことがあればアクセスログも確認するようにします!
退会済みユーザー

退会済みユーザー

2021/01/23 09:48

失礼。 コードをあまり確認せず回答しました^^; 適切な回答が付いてよかったです。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問