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

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

ただいまの
回答率

90.01%

全ての入力フィールドにて内容が変わってもON DUPLICATE KEY UPDATEで新規追加させたくない

解決済

回答 4

投稿 編集

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

kuzurotto

score 403

1つのページで登録と更新を作っているのですが、
下記コードの場合、projectTitleをUNIQUEにしているので
projectTitleを変えると変更ではなく追加になってしまいます。

この場合どうやって回避できるでしょうか?

$sql = "INSERT INTO user_vitae(
user_id,
projectTitle,
projectsContents
) VALUES (?, ?, ?)
ON DUPLICATE KEY UPDATE
user_id = ?,
projectTitle = ?,
projectsContents = ?
";
        for($i=0;$i<count($_POST['projectTitle']);$i++) {
            $stmt = $this->pdo->prepare($sql);
            $stmt->execute([
$_SESSION['id'],
$_POST['projectTitle'][$i],
$_POST['projectsContents'][$i],

$_SESSION['id'],
$_POST['projectTitle'][$i],
$_POST['projectsContents'][$i]
            ]);
        }

イメージ説明

案件名をUNIQUEにしているので、例えば
「test1」登録しておいて、
案件名を「test2」に変えたいなぁ~ってなったら、
更新ではなく新規登録になってしまいます。

これはどう防げばよいのでしょうか?

入力(変更)を必要としないカラムは
・id
・created
・user_id

この3つです。

イメージ説明


ソースはこうなっております。
全部載せると量が多いので必要なところだけを載せています。

<?php
$dsn = 'mysql:dbname=test;host=localhost;charset=utf8';
$user = 'root';
$password = '';
$option = array(PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION);
$pdo = new PDO($dsn, $user, $password, $option);

$sql = "SELECT * FROM user_vitae WHERE user_id=? ORDER BY id ASC";
$stmt = $pdo->prepare($sql);
$stmt->execute([$_SESSION['id']]);
?>

<form action="" method="post">


    <?php foreach ($stmt as $test) { ?>
    <table border="">
        <tbody>
            <tr>
                <td>案件名</td>
                <td>
                    <input type="text" name="projectTitle[]" maxlength="30" size="" placeholder="" value="<?php echo $test['projectTitle'] ?>">
                </td>
            </tr>
            <tr>
                <td>案件内容</td>
                <td>
                    <textarea name="projectsContents[]" rows="2" cols="21">
                        <?php echo $test['projectsContents'] ?>
                    </textarea>
                </td>
            </tr>
        </tbody>
    </table>
    <br />
    <br />
    <?php } ?>


    <table id="vitae" class="vitae" border="">
        <tbody>
            <tr>
                <td>案件名</td>
                <td>
                    <input type="text" name="projectTitle[]" maxlength="30" size="" placeholder="" >
                </td>
            </tr>
            <tr>
                <td>案件内容</td>
                <td>
                    <textarea name="projectsContents[]" rows="2" cols="21">
                    </textarea>
                </td>
            </tr>
        </tbody>
    </table>
    <div id="vitaeAdd">
        <input type="button" value="業務経歴を追加する" onClick="vitaeAdd()">
        <input type="button" value="業務経歴を削除する" onClick="vitaeDel()" id="vitae_btnDel" disabled="true">
    </div>
    <br />
    <input type="submit" name="submit" value="登録する">
</form>


↓登録&更新用の処理です。

$sql = "INSERT INTO user_vitae(
user_id,
projectTitle,
projectsContents
) VALUES (?, ?, ?)
ON DUPLICATE KEY UPDATE
projectTitle = ?,
projectsContents = ?
";

for($i=0;$i<count($_POST['projectTitle']);$i++) {
    $stmt = $this->pdo->prepare($sql);
    $stmt->execute([
$_SESSION['id'],
$_POST['projectTitle'][$i],
$_POST['projectsContents'][$i],
$_POST['projectTitle'][$i],
$_POST['projectsContents'][$i]
    ]);
}

↓現在の表示
イメージ説明

困っているところ

・ON DUPLICATE KEY UPDATEで案件名や案件内容をUNIQUEキーにしたら2つとも変更した場合、新規追加扱いになる

・ON DUPLICATE KEY UPDATEでuser_idをUNIQUEキーにしたら1件のレコードしか追加できなくなる

感じているところ

・案件名や案件内容をUNIQUEキーにしたら2つとも変更した場合、新規追加扱いになるのを防ぐために、idをUNIQUEにして、ON DUPLICATE KEY UPDATEに組み込んだらこの質問は解決されると思うが、そのSQLの組み方が分からない。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

checkベストアンサー

0

ON DUPLICATE KEY UPDATE
user_id = ?,
projectTitle = ?,

とありますが、user_idをキーとするので
INSERTにだけ反映すれば良いのでは?

$sql = "INSERT INTO user_vitae(
user_id,
projectTitle,
projectsContents
) VALUES (?, ?, ?)
ON DUPLICATE KEY UPDATE
projectTitle = ?,
projectsContents = ?
";
for($i=0;$i<count($_POST['projectTitle']);$i++) {
   $stmt = $this->pdo->prepare($sql);
  $stmt->execute([
    $_SESSION['id'],
    $_POST['projectTitle'][$i],
    $_POST['projectsContents'][$i],
    $_POST['projectTitle'][$i],
    $_POST['projectsContents'][$i]
    ]);
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/07/11 13:26

    ちなみに、文書の修正をするなら、あたりまえですが
    文書IDか元文書のタイトルが必要ですよ
    修正元が特定できなければ修正のしようがないので

    キャンセル

  • 2016/07/11 13:30

    それは「誰の業務経歴か」を特定するために、下記で誰のものかを取得してvalueに当てはめていってますがどういうことですよね?


    <?php
    $dsn = 'mysql:dbname=test;host=localhost;charset=utf8';
    $user = 'root';
    $password = '';
    $option = array(PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION);
    $pdo = new PDO($dsn, $user, $password, $option);

    $sql = "SELECT * FROM user_vitae WHERE user_id=? ORDER BY id ASC";
    $stmt = $pdo->prepare($sql);
    $stmt->execute([$_SESSION['id']]);
    ?>






    キャンセル

  • 2016/07/11 13:39

    修正用にはすでにある文書のidもしくはユーザーidとタイトルの組み合わせを
    hiddenなどで引き継ぐわけです。
    仮に自分がユーザーid=100番だとして、タイトル=テスト1をテストXに変えようとするなら
    INSERT 構文でテストXを投入すれば投入できてしまいます。
    (たぶん今がこの状態)

    なので、修正の時は単純にUPDATE構文を発行してください

    キャンセル

0

更新時には、IDも含めるようにしてください。

IDなしでON DUPLICATE KEY UPDATEとすると、2列にまたがってuniqueをかけているところで変更した場合に、そもそも重複しませんので、INSERTとして動いてしまいます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/07/11 13:38

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

    更新時には、IDも含めるとはどういうことでしょうか???


    $sql = "INSERT INTO user_vitae(
    id,
    user_id,
    projectTitle,
    projectsContents
    ) VALUES (?, ?, ?, ?)
    ON DUPLICATE KEY UPDATE
    projectTitle = ?,
    projectsContents = ?
    ";
    for($i=0;$i<count($_POST['projectTitle']);$i++) {
    $stmt = $this->pdo->prepare($sql);
    $stmt->execute([
    これから挿入するレコードのID???
    $_SESSION['id'],
    $_POST['projectTitle'][$i],
    $_POST['projectsContents'][$i],
    $_POST['projectTitle'][$i],
    $_POST['projectsContents'][$i]
    ]);
    }

    キャンセル

  • 2016/07/11 13:43

    MySQLのauto_incrementの場合、id列にNULLを入れておけば普通にINSERTできます。

    もっとも、新規登録と更新を1本のSQLで済ませようということ自体、余計に面倒なことになっている気もしますが。

    キャンセル

0

更新前、更新後のレコードを送信して処理をすればできると思います。
delete_flagなど論理削除用のフラグを用います。
・delete_flag=0 論理削除していない (boolean値でもOK)
・delete_flag=1 論理削除している

※提示されたレコードを例に記載します。
①更新前のレコードを一度論理削除する(delete_flag=1)
a)test1をdelete_flag=1で更新(UPDATE)

②追加したいレコードをdelete_flag=0でINSERTする
b)test2をdelete_flag=0で追加(INSERT)
c)(変更なければ)test1をdelete_flag=0で更新(UPDATE)

③delete_flag=1のレコードを削除する(DELETE)
d)b)の場合は、a)のレコードが削除されます。
e)c)の場合は、削除が空振りします。(DELETEは対象がなくてもステータスが正常になります)

上記①~③の処理を行えば、不要なレコードは消え、追加したレコードで置き換えできます。
③はループの外(COMMIT直前)で処理すればいいです。
(いちいち削除のクエリを①~②と一緒にする必要はないです、delete_flag=1でまとめて削除できます)

質問とは別ですが、データの一意性を保つため、トランザクション~コミットでくくったほうがいいですかね。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

$sql = "INSERT INTO user_vitae(
  id,
  user_id,
  projectTitle,
  projectsContents
) VALUES (?, ?, ?, ?)
  ON DUPLICATE KEY UPDATE
  id = ?
";

for($i=0;$i<count($_POST['projectTitle']);$i++) {
    $stmt = $this->pdo->prepare($sql);
    $stmt->execute([
      empty($_POST['id'][$i])?null:$_POST['id'][$i], // hiddenで持つ。※新規追加場合からなのでnullを渡す。
      $_SESSION['id'],
      $_POST['projectTitle'][$i],
      $_POST['projectsContents'][$i],
      $_POST['id'][$i]
    ]);
}

ですかね。rentoさんの設計の場合、タイトルと内容で一意とみなすのではなくIDで一意とみなすべきで、タイトルと内容はIndexとして利用すべきですね。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

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

  • トップ
  • PHPに関する質問
  • 全ての入力フィールドにて内容が変わってもON DUPLICATE KEY UPDATEで新規追加させたくない