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

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

新規登録して質問してみよう
ただいま回答率
85.50%
PHP

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

Q&A

解決済

1回答

1309閲覧

更新だけしたいだけの場合でも、 空文字の新規追加が必ず1件はされてしまうのをなんとかしたい

earnest_gay

総合スコア615

PHP

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

0グッド

0クリップ

投稿2016/07/12 09:10

編集2016/07/12 09:13

とあるサイトの更新ページを作っています。

下記の画像は、
上から5行目まではDBへ登録している情報を表示させています。
そして、一番下のテキストボックスは新しく資格を取得した時に追加できるためのフィールドです。

イメージ説明

問題は
更新だけしたいだけの場合でも、
空文字の新規追加が必ず1件はされてしまうことです。

それが5行目になります。

画像の状態で更新すると
もう1件空のフィールドが表示されます。

4 この資格を削除する
3 この資格を削除する
2 この資格を削除する
1 この資格を削除する
空 この資格を削除する
空 この資格を削除する
空 新規追加

これはどう防げばよいでしょうか?
知恵をお貸しください...

参考になるか分かりませんが、
ソースは下記です。

<?php /所有資格->更新 $sql = "UPDATE user_qualifications SET qualification = ? WHERE id = ? "; for($i=0;$i<count($_POST['qualifications']);$i++) { $stmt = $pdo->prepare($sql); $stmt->execute([ $_POST['qualifications'][$i], $_POST['id_sq'][$i] ]); } //所有資格->新規追加 $sql = "INSERT INTO user_qualifications(user_id,qualification) VALUES (?, ?)"; for($i=0;$i<count($_POST['qualifications_add']);$i++) { $stmt = $pdo->prepare($sql); $stmt->execute([ $_SESSION['id'], $_POST['qualifications_add'][$i] ]); } //ログインしている人のユーザーデータを取得(lalue割り当て用) $sql = "SELECT * FROM user_data WHERE id=?"; $stmt = $pdo->prepare($sql); $stmt->execute([ $_SESSION['id'] ]); $RowData = $stmt->fetch(); ?> <tr> <td colspan="2">所有資格</td> </tr> <?php foreach ($stmtQualifications as $sq): ?> <tr> <td> <input type="hidden" name="id_sq[]" value="<?php echo $sq['id'] ?>"> <input type="text" name="qualifications[]" maxlength="50" value="<?php echo $sq['qualification'] ?>"> <br /> </td> <td> <a href='delete_skills.php?id=<?php echo $sq['id'] ?> ' onclick="return confirm('本当に削除してよろしいですか?')">この資格を削除する</a> </td> </tr> <?php endforeach ?> <tr id="qualification" class="qualification"> <td colspan="2"> <input type="text" name="qualifications_add[]" maxlength="50"> </td> </tr> <tr id="quaAdd"> <td colspan="2"> <input type="button" value="追加" onClick="quaAdd()"> <input type="button" value="削除" onClick="quaDel()" id="qua_btnDel" disabled="true"> </td> </tr>

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

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

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

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

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

guest

回答1

0

ベストアンサー

ループ内にif文でチェックするではNGでしょうか?

PHP

1// $_POST['qualifications_add'][$i]を判定する 2if($_POST['qualifications_add'][$i]!==''){ 3 $stmt->execute([ 4 $_SESSION['id'], 5 $_POST['qualifications_add'][$i] 6 ]); 7}

以上の手段ですと、必ず全件UPDATEになってしまうので、<input type="hidden" value-"<?php echo $hoge; //変更前?>">
とかで変更前変更後をぶつけて変更があったデータのみアップデートしたほうがよいと思います。

投稿2016/07/12 09:17

編集2016/07/12 09:20
roast_chicken

総合スコア254

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

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

earnest_gay

2016/07/12 09:21

あぁ、気づきませんでした???? ありがとうございます!
earnest_gay

2016/07/12 09:24

すいません。 知識不足で教えて頂きたいのですが 全件UPDATEになることでなにか不都合なことはあるのでしょうか?
roast_chicken

2016/07/12 09:29

単純にムダなSQLを発行しているので、データベースさんに負荷がかかるっていう話です。 更新不要なデータは極力更新しないほうがいいですよね? 10件程度であれば項目数に寄りますけどそんなに負荷はないと思いますが。。。 今後大量データを更新する際には覚えておいてくださいー。
earnest_gay

2016/07/12 09:35

教えて頂きありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問