$POSTで渡した配列データをmysqlのテーブルに一括でinsertできない

Question

前提・実現したいこと

フィールド数が多いので、ｈｔｍｌでvalueに配列変数を使って値を保持し、POSTで渡してmysqlのデータベースのテーブルに一括で登録したい
一つ気になることがありまして、テーブルにAUTO_INCREMENTのフィールドがあります。
間違いないのは配列の数もテーブルのフィールドの数も同じです。
数を合わせるため、下記のタグも入れています
<input type="hidden" name="dat[]" value="" />

■■な機能を実装中に以下のエラーメッセージが発生しました。

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

Column count dosen't match value count at row 1

該当のソースコード

ｈｔｍｌの内容

<td><input type="hidden" name="dat[]" value="" /></td> <td><input maxlength='10' size="6" type="date" name="dat[]" /></td> <td><input maxlength='2' size="1" type="text" name="dat[]" /></td> <td><input maxlength='2' size="1" type="text" name="dat[]" /></td> こんな感じで３０個あります。

ｐｈｐの内容

$dat=$_POST['dat'];

$res = implode( ',', $dat );

$query = "INSERT INTO テーブルA VALUES('$res')";

$result = $mysqli->query($query);

if (!$result) {
print('クエリーが失敗しました。' . $mysqli->error);
$mysqli->close();
exit();
}

$mysqli->close();
$result->close();

Comments

[m.ts10806]

コードはマークダウンのcode機能を利用して提示してください。
回答者が正しくコードを再現確認・調整に利用するために平で提示するのは推奨されていません。

[m.ts10806]

またエラー自体は珍しいものではありません。エラーメッセージで調べたことはありますか？
テーブルAのテーブル定義（CREATE文が良）も提示されたほうが的確なアドバイスが得られます。

[bunbun007]

ご意見まことにありがとうございました。
検証不足でした。見直します。

[退会済みユーザー]

AI　項目って NULL(もしくは０）の場合に自動で数値をふるのがMySQLの仕様で、任意の値に設定できないというわけではない

Answer 1

Column count dosen't match value count at row 1

列数が1行目の値の数と一致しません。

これはそのままで、INSERTしようとしている列数とデータ（VALUES）が合致していないことを指します。

$queryという変数にSQLを入れていますが、これが想定通りになっているか確かめましたか？

$dat=$_POST['dat']; 
$res = implode( ',', $dat ); 
$query = "INSERT INTO テーブルA VALUES('$res')"; 
echo $query ;

おそらくこうなっています。
INSERT INTO テーブルA VALUES('1,2,3,4,5・・・（略）')

1つのカラム（テーブル定義上、一番最初のカラム）にしか投入されていません。

テーブルにAUTO_INCREMENTのフィールドがあります。

間違いないのは配列の数もテーブルのフィールドの数も同じです。

であれば、AUTO_INCREMENT（および初期値指定がある）フィールドを除いた全フィールドを指定したINSERT文を作成しないといけません。

フィールド名分からないですが、例えばこんな感じ。

※未検証。構文エラーは避けていますが、そのまま動くものではありません。実装イメージだけもってください。

<?php
$dat=[1,3,5,6,7,8]; //検証のための仮の値。
$columns = [];
$values = [];
foreach($dat as $dnum=>$dval){
    $columns[] = "data".$dnum;
    $values[] = "'".$mysql->real_escape_string($dval)."'";
}

$query = "INSERT INTO テーブルA (".implode(",",$columns).") VALUES(".implode(",",$values).");"; 

echo $query;
// INSERT INTO テーブルA (data0,data1,data2,data3,data4,data5) VALUES('1','3','5','6','7','8');

SQLにPOSTされた情報をそのまま入れるのはNGです。必ずエスケープしてください。
できればPDO使った方がすっきり書けます。

もう1点。(むしろこっちのほうが大事)
既に指摘があるとおり全て配列に突っ込んでいては処理が明確にわけられません。
nameにはきちんと役割をそれぞれ明確に与えた名前をつけるべきです。（もしフィールド名も同じようにマジックナンバーなどつけていれば改めるべき）
何1ついいことはないので、今からでも改善してください。
おそらくこのコードから離れて数ヶ月たったときこのコードを見たときにどこが何をやっているか全て見直さなければ理解できない事態が確実に起きます。
メンテナンス性・可読性の低いコードは低ければ低いほど、不具合が起きる可能性が高くなります。

• まずコードの可読性を最適化しよう
• 初心者が保守性の高いプログラムを書くために注意するべきこと
• 保守性の高いコードを書くためにはどうしたらよいか

Comments

[bunbun007]

アドバイス誠にありがとうございました。

[m.ts10806]

私の「もう一点」のところまで読まれたのでしたら別のコメントをしているのはいささか納得いかないですね。
プログラムは囚人ではないのできちんと名前をつけてあげてください。

[bunbun007]

アドバイス誠にありがとうございました。
とりあえず例文をもとにやり直しましたらうまくいきました。
ありがとうございました。

１つ質問をさせてください。
postされるデータがあまりにも多いのでnameに配列をつかって、一括でinsertできないかと考えてこのようなやり方をしました。
皆様のご意見を聞きますと、配列にみんなぶち込むのはナンセンスというご意見でした。
そこで、nameにきちんとした変数名を指定した場合のpostでの受け取り方を間単にするやり方をアドバイスいただけないと思いコメントさせていただきました。

例えば
htmlの内容

<td><input maxlength='8' size="6" type="date" name="dat01" /></td>
<td><input maxlength='2' size="1" type="text" name="dat02" /></td>
<td><input maxlength='2' size="1" type="text" name="dat03" /></td>
　　　　　　↓
<td><input maxlength='2' size="1" type="text" name="dat30" /></td>
こんな感じで３０個あります。

phpの内容

■配列の場合は
$dat=$_POST['dat'];
この一行で受け取れます

■単なる変数（dat01～dat03）を受けとるためには
$dat01 = $_POST[‘dat01’];
$dat02 = $_POST[‘dat02’];
$dat03= $_POST[‘dat03’];
　　　　　↓
$dat30= $_POST[‘dat30’];
こんな感じで30個必要なのでしょか

配列のように一行にできますか？（一行とはいわず数行とか）

よろしくお願いします。

[m.ts10806]

30と決まってるならforで1から回せば良いのでは。可変変数と合わせればその取得処理だけなら三行程度です。

[m.ts10806]

回答にも書きましたがdat01というnameですら何を指してるのか分からないのでナンセンスですけどね。マジックナンバーは避けるべきです。

[bunbun007]

アドバイスありがとうございます
そうしましたらname="namae"、name="address"、name="age"…と30個名前が違う変数の受け取り方を簡単にするにはどうしたらいいですか。

よろしくお願いします

[m.ts10806]

そこは調べるなり考えるなりしません？なんでも教えてでは良くないです。
理論的には連番をforで受けとるのと同じです。
配列をうまく使えばhtml生成もバリデートも取得も登録もできます。
だいぶ前ですが回答に書いたことがあるので頑張って探してみてください

[bunbun007]

分かりました
ありがとうございました

Answer 2

何でもかんでも一緒くたにdat[]にまとめるというのは
いわゆるクソミソな処理ですね。正直感心しません。
また一つのカラムにカンマで区切って流し込むという仕様は
おおよそSQL的な処理ではありません、かならずデータ毎にカラムは分けてください

とりあえずテーブルの状況がわかりませんが主キーのidとデータカラムのみで
構成されているテーブルなら

$query = "INSERT INTO テーブルA(col1) VALUES('$res')";
//もしくは
$query = "INSERT INTO テーブルA VALUES(null,'$res')";

とするところからでしょうか？

それとデータ投入時にはprepareで処理しないとインジェクション問題や
セパレータの競合などが発生しますので、きちんと学習してください

sample

ロジックだけ理解してもらえばいいので今回はエスケープ処理やprepare処理は省略します

<form method="get">
<input type="text" name="dat[]" value="123"><br>
<input type="text" name="dat[]" value="abc"><br>
<input type="text" name="dat[]" value="hoge"><br>
<input type="submit" value="go">
</form>
<?PHP
$dat=filter_input(INPUT_GET,"dat",FILTER_DEFAULT,FILTER_REQUIRE_ARRAY);
$res=implode("','",$dat);
$sql= "INSERT INTO テーブルA VALUES(null,'{$res}')";
print $sql;
?>

※auto_incrementのカラムが先頭にある前提でnullを与えればカラム数は合うでしょう
逆にあわなければなにか重要な情報を提示していないと推測されるので改めて
テーブル構成と入力データの状況を例示してもらうしかありません

Comments

[bunbun007]

アドバイス誠にありがとうございます。

一つお尋ねします。
テーブル数が３０個ありまして、各テーブルのフィールドは５０個くらいある仕様なのですが、ｈｔｍｌ内のnameはやはり、例えばname="dat01"・・・name="dat50"としたほうがいいということでしょうか。配列の方がすっきりするようなきがしますが。
なにしろ、フィールド数が多いので一括で登録、更新ができないかと試行錯誤しております。
なにかよいやり方がございましら是非、ご指導いただきたくよろしくお願い致します

[yambejp]

> 各テーブルのフィールドは５０個

implodeの仕方を見る限り、すべてのデータを１カラムに突っ込むような
書き方にしかなっていませんよ
今回のご質問の趣旨が、汎用的に公開するものではなく、個人用で
データを一切バリデートする必要もないというのであれば
いくらでもやりようはあります。
たとえば全部textareaに一度流し込んでしまえば、連結さえいりません

> ｈｔｍｌ内のnameはやはり、例えばname="dat01"・・・name="dat50"としたほうがいい

そのとおりです。
理由は簡単で、今回トラブルになっている内容をご自身で解決できないように
ぐちゃっとしたデータは検証が非常に難しく、トラブル解消が困難だからです
逆になんでもかんでも自己解決できるのであれば、別にどんなやり方でも
好きにすれば良いと思います

[bunbun007]

アドバイス誠にありがとうござました