1回のINSERTで複数行のレコードを一括追加したい

Question

前提・実現したいこと

PHPでSQLを実行したいです。

複数の値を$_POSTで受け取り、複数行のレコードをデータベースに一括追加したいです。
受け取る値は1〜5つで変動するのですが、2つ以上を受け取っても1つしか登録できません。

if文で考えたのですが期待した結果を得られません。
$_POSTの値は受け取れています。

If文も長くなってしまい、、もっとでスッキリ書き替える方法もアドバイス頂けると助かります。

よろしくお願いします。

コード全体

<?php
//値を受け取る
$day1 = htmlspecialchars( $_POST[ "day1" ], ENT_QUOTES, "UTF-8" );
$time1 = htmlspecialchars( $_POST[ "time1" ], ENT_QUOTES, "UTF-8" );
$day2 = htmlspecialchars( $_POST[ "day2" ], ENT_QUOTES, "UTF-8" );
$time2 = htmlspecialchars( $_POST[ "time2" ], ENT_QUOTES, "UTF-8" );
$day3 = htmlspecialchars( $_POST[ "day3" ], ENT_QUOTES, "UTF-8" );
$time3 = htmlspecialchars( $_POST[ "time3" ], ENT_QUOTES, "UTF-8" );
$day4 = htmlspecialchars( $_POST[ "day4" ], ENT_QUOTES, "UTF-8" );
$time4 = htmlspecialchars( $_POST[ "time4" ], ENT_QUOTES, "UTF-8" );
$day5 = htmlspecialchars( $_POST[ "day5" ], ENT_QUOTES, "UTF-8" );
$time5 = htmlspecialchars( $_POST[ "time5" ], ENT_QUOTES, "UTF-8" );

require_once( '../../core/config.php' );
try {
  // PDOインスタンスを生成
  $dbh = new PDO( DSN, DB_USER, DB_PASS );
} catch ( PDOException $e ) {
  $msg = $e->getMessage();
}

if ( empty( $day1 ) && empty( $time1 ) && empty( $day2 ) && empty( $time2 ) && empty( $day3 ) && empty( $time3 ) && empty( $day4 ) && empty( $time4 ) && empty( $day5 ) && empty( $time5 ) ) {
$msg = '日付、時間が選択されていません。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}elseif($day1 && empty( $time1 )){
$msg = '日付と時間をセットで選択してください。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}elseif( empty( $day1 ) && $time1  ){
$msg = '日付と時間をセットで選択してください。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}elseif($day2 && empty( $time2 )){
$msg = '日付と時間をセットで選択してください。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}elseif( empty( $day2 ) && $time2  ){
$msg = '日付と時間をセットで選択してください。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}elseif($day3 && empty( $time3 )){
$msg = '日付と時間をセットで選択してください。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}elseif( empty( $day3 ) && $time3  ){
$msg = '日付と時間をセットで選択してください。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}elseif($day4 && empty( $time4 )){
$msg = '日付と時間をセットで選択してください。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}elseif( empty( $day4 ) && $time4  ){
$msg = '日付と時間をセットで選択してください。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}elseif($day5 && empty( $time5 )){
$msg = '日付と時間をセットで選択してください。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}elseif( empty( $day5 ) && $time5  ){
$msg = '日付と時間をセットで選択してください。';
  $link = '<a href="javascript:history.back()">前に戻る</a>';
}else{
$today = date( "Y-m-d H:i" );

if($day1){
$sql = "INSERT INTO holiday(holiday, break_time, shop_id, created) VALUES (:day, :time, :shop_id, :created)";
  $stmt = $dbh->prepare( $sql );
  $stmt->bindValue( ':day', $day1 );
  $variable = $day1.$time1;
  $variableSplit = str_split($variable, 10);
  $result = implode(' ', $variableSplit);
  
  $stmt->bindValue( ':time', $result );
  $stmt->bindValue( ':shop_id', $shop_id );
  $stmt->bindValue( ':created', $today );
}
if($day2){
$sql = "INSERT INTO holiday(holiday, break_time, shop_id, created) VALUES (:day, :time, :shop_id, :created)";
  $stmt = $dbh->prepare( $sql );
  $stmt->bindValue( ':day', $day2 );
  $variable = $day2.$time2;
  $variableSplit = str_split($variable, 10);
  $result = implode(' ', $variableSplit);
  
  $stmt->bindValue( ':time', $result );
  $stmt->bindValue( ':shop_id', $shop_id );
  $stmt->bindValue( ':created', $today );
}
if($day3){
$sql = "INSERT INTO holiday(holiday, break_time, shop_id, created) VALUES (:day, :time, :shop_id, :created)";
  $stmt = $dbh->prepare( $sql );
  $stmt->bindValue( ':day', $day3 );
  $variable = $day3.$time3;
  $variableSplit = str_split($variable, 10);
  $result = implode(' ', $variableSplit);
  
  $stmt->bindValue( ':time', $result );
  $stmt->bindValue( ':shop_id', $shop_id );
  $stmt->bindValue( ':created', $today );
}
if($day4){
$sql = "INSERT INTO holiday(holiday, break_time, shop_id, created) VALUES (:day, :time, :shop_id, :created)";
  $stmt = $dbh->prepare( $sql );
  $stmt->bindValue( ':day', $day4 );
  $variable = $day4.$time4;
  $variableSplit = str_split($variable, 10);
  $result = implode(' ', $variableSplit);
  
  $stmt->bindValue( ':time', $result );
  $stmt->bindValue( ':shop_id', $shop_id );
  $stmt->bindValue( ':created', $today );
}
if($day5){
$sql = "INSERT INTO holiday(holiday, break_time, shop_id, created) VALUES (:day, :time, :shop_id, :created)";
  $stmt = $dbh->prepare( $sql );
  $stmt->bindValue( ':day', $day5 );
  $variable = $day5.$time5;
  $variableSplit = str_split($variable, 10);
  $result = implode(' ', $variableSplit);
  
  $stmt->bindValue( ':time', $result );
  $stmt->bindValue( ':shop_id', $shop_id );
  $stmt->bindValue( ':created', $today );
}
  $stmt->execute();
  $msg = '登録が完了しました';
  $link = '<p><a href="ownerList.php">予約管理</a></p>';

}
?>
<div class="form-wrapper">
<h2><?php echo $msg; ?></h2>
<!--メッセージの出力-->
<div class="form-footer"> <?php echo $link; ?> </div>
<!--コンテンツ終了--> 
</div>

Answer 1

$stmt = $dbh->prepare( $sql );

で毎回新たなプリペアドステートメントが作られているので
最後にセットされたSTMTだけexecute()されます。

毎回executeされるか、バルクインサートの形でSQL文作られては。
※名前付きパラメータだとキーがその数必要となるので注意。

INSERT INTO HOGE (A,B,C) VALUES 
(:A1,:B1,:C1)
,(:A2,:B2,:C2)
,(:A3,:B3,:C3)

Comments

[emi_ono]

ありがとうございます！
毎回executeすることで解決できました！

[m.ts10806]

「一括で」と書いてあったので「一度のSQL実行で」とも思ったのですが、そういうことではなかったのですね。
同じテーブルに対するInsertならバルクインサートにしたほうがパフォーマンスも良いです。

Answer 2

質問文の構造を尊重するなら、こうなりますでしょうか。

$sql = "INSERT INTO holiday(holiday, break_time, shop_id, created) VALUES (:day, :time, :shop_id, :created)";
$stmt = $dbh->prepare( $sql );
if($day1){
  $stmt->bindValue( ':day', $day1 );
  $variable = $day1.$time1;
  $variableSplit = str_split($variable, 10);
  $result = implode(' ', $variableSplit);
  
  $stmt->bindValue( ':time', $result );
  $stmt->bindValue( ':shop_id', $shop_id );
  $stmt->bindValue( ':created', $today );
  $stmt->execute();
}
if($day2){
  $stmt->bindValue( ':day', $day2 );
  $variable = $day2.$time2;
  $variableSplit = str_split($variable, 10);
  $result = implode(' ', $variableSplit);
  
  $stmt->bindValue( ':time', $result );
  $stmt->bindValue( ':shop_id', $shop_id );
  $stmt->bindValue( ':created', $today );
  $stmt->execute();
}
// 以下略

あと、変数名が1,2,3,4の違いで一通り揃っているのであれば、
それこそ配列で持ってforやforeachで回したほうがよりコンパクトになりますね。

$sql = "INSERT INTO holiday(holiday, break_time, shop_id, created) VALUES (:day, :time, :shop_id, :created)";
$stmt = $dbh->prepare( $sql );
for ($i = 0; $i < 4; $i++) {  // インデックスは0,1,2,3として
  if($day[$i]){
    $stmt->bindValue( ':day', $day[$i] );
    $variable = $day[$i].$time[$i];
    $variableSplit = str_split($variable, 10);
    $result = implode(' ', $variableSplit);
  
    $stmt->bindValue( ':time', $result );
    $stmt->bindValue( ':shop_id', $shop_id );
    $stmt->bindValue( ':created', $today );
    $stmt->execute();
  }
}
// 以下略

あるいは関数化してもいいし。

データベースへのトランザクション処理もセットでお願いしますね。
$dbh->beginTransaction();を$dbh->prepare()の前で行い、
最大4回分の$stmt->execute();実行後に$dbh-commit();をすれば、データベースに反映されます。
途中でなにか実行上のエラーが生じても、$dbh-commit();をする前ならもとに戻ります。
$dbh-rollback();を実行しても戻ります。
PHP: PDO::commit - Manualのサンプルコードを参考にしてください。

なにも、1回のINSERT文で全部やらなくても、1回のトランザクション中に分割して最大4回のINSERT文でやってもいいと思います。
たかだか数件ですし。

Comments

[退会済みユーザー]

beginTransaction、commit、rollbackも加えてくださいね。

[emi_ono]

ありがとうございます！

[emi_ono]

ありがとうございます。
毎回executeすることで解決できました！