チェックボックスのデータベース登録について

Question

チェックボックスにチェックされた数によって、動的にデータベースに反映させたいのですが、行き詰まっています。。現状、以下のコードです。$checkboxにはチェックボックス（配列が入っています）

$sql='INSERT INTO table(sample,sample,sample,sample,sample) VALUES(?,?,?,?,?)';
↑ここの「？」を動的に？いや、そしたらsample,sample,sample,sample,sampleも動的にしないといけないしな、、、という訳でNullの状態でも$sample[]=$valueに入れたら、うまくいくんじゃないかという考えに終着しました。（この考えがあっているかも不明です）

どうするのが最も正解なのでしょう。。

<?php
$checkbox=$_POST['checkbox'];//値を取得（$checkboxは配列）

				 try{
						 require_once('../../conf/db_con.php');
						 //データ反映
						$sql='INSERT INTO table(sample,sample,sample,sample,sample) VALUES(?,?,?,?,?)';
						$stmt=$dbh->prepare($sql);
						foreach($checkbox as $value){
									echo $sample[]=$value;
						} 
						$stmt->execute($sample);
						$dbh=null;
				 }catch(Exception $e){
					print 'ただいま障害により大変ご迷惑をお掛けしております。';
					exit();
				 }
?>

【追加・修正依頼】
チェックボックスが複数あり、チェックされたもの（当然、複数の可能性あり）の値をデータベースに登録したい、ということですね？
↑はい、その通りです。

チェックボックスの数が一定か、不定か
↑チェックボックスの数は一定です。（10個のチェックボックスがあります、今後増えるかもしれませんが今のところ10個です）

最終的にどのような機能（WEBページ?）を作成したいのか
↑最終的にはジャンルで絞り込めるようにしたいです。（【不動産の場合】バス・トイレ別、南向き等【飲食店の場合】喫煙可、貸し切り有など） よろしくお願いいたします。

【テーブル構造】※サンプルで動かしているので簡易的です。すいません。
・id
・user_code
・checkbox_data1
・checkbox_data2
・checkbox_data3
・checkbox_data4
・checkbox_data5
・checkbox_data6
・checkbox_data7
・checkbox_data8
・checkbox_data9
・checkbox_data10

といった構造になっています。

Comments

[KiyoshiMotoki]

チェックボックスが複数あり、チェックされたもの（当然、複数の可能性あり）の値をデータベースに登録したい、ということですね？

これはチェックボックスの数が一定か、不定かによって対処方法が変わってくると思います。

最終的にどのような機能（WEBページ?）を作成したいのかを教えていただけると、
具体的な回答を得やすくなると思います。

[ssk]

チェックボックスが複数あり、チェックされたもの（当然、複数の可能性あり）の値をデータベースに登録したい、ということですね？
↑はい、その通りです。

チェックボックスの数が一定か、不定か
↑チェックボックスの数は一定です。（10個のチェックボックスがあります、今後増えるかもしれませんが今のところ10個です）

最終的にどのような機能（WEBページ?）を作成したいのか
↑最終的にはジャンルで絞り込めるようにしたいです。（【不動産の場合】バス・トイレ別、南向き等【飲食店の場合】喫煙可、貸し切り有など）

よろしくお願いいたします。

[退会済みユーザー]

table の構造が分かるように明記してください。

[ssk]

【テーブル構造】※サンプルで動かしているので簡易的です。すいません。

・id
・user_code
・checkbox_data1
・checkbox_data2
・checkbox_data3
・checkbox_data4
・checkbox_data5
・checkbox_data6
・checkbox_data7
・checkbox_data8
・checkbox_data9
・checkbox_data10

といった構造になっています。

[退会済みユーザー]

捕捉してくれているのに、余計にわかりにくくなっています。
不動産の「設備」を登録するプログラムなのに、「user...」ってどういう意味なんでしょう？

[ssk]

いえ、不動産の設備を登録するプログラムではないです（不動産のバス・トイレ別などは、あくまで一例です）。ジャンルで絞り込めるようにしたいのです。

Answer 1

情報の追記、ありがとうございます。

（SUUMOのような）会員制の物件情報サイトで、
"お気に入りの検索条件"
を登録できるような機能を想定されているのでしょうか？

であれば、SQLは動的にする必要はなさそうです。

チェックボックスごとに対応するカラムがあるようなので、以下を前提に回答させていただきます。

• チェックボックスがチェックされていれば対応するカラムに1を、チェックされていなければ0を格納する。
• 各チェックボックスのvalue属性の値は、対応するカラムの名前と一致している。
• user_codeは会員IDで、チェックボックスの値と同様にPOSTデータから取得する。

'id'というカラムは用途が不明なので、とりあえず無視します。
（オートインクリメントのPKでしょうか？）

$columnList = array(
        'checkbox_data1',
        'checkbox_data2',
        'checkbox_data3',
        'checkbox_data4',
        'checkbox_data5', 
        'checkbox_data6',
        'checkbox_data7',
        'checkbox_data8',
        'checkbox_data9',
        'checkbox_data10'
);

$placeholders = array_fill(0, count($columnList), '?');

$userCode = $_POST['user_code'];
// チェックボックスが1つもチェックされていない場合、空配列を代入する。
$checkbox= (isset($_POST['checkbox'])) ? $_POST['checkbox'] : array(); //値を取得（$checkboxは配列）

try{
    require_once('../../conf/db_con.php');

    //データ反映
    $sql= sprintf(
        'INSERT INTO table (user_code, %s) VALUES (?, %s)',
        implode(', ', $columnList),
        implode(', ', $placeholders));
    $stmt = $dbh->prepare($sql);

    $stmt->bindValue(1, $userCode, PDO::PARAM_STR);

    $index = 2;
    foreach($columnList as $column){
        $value = in_array($column, $checkbox) ? 1 : 0;
        $stmt->bindValue($index, $value, PDO::PARAM_INT);

        $index++;
    }

    $stmt->execute();
    $dbh=null;
}catch(Exception $e){
    print 'ただいま障害により大変ご迷惑をお掛けしております。';
    exit();
}

Comments

[ssk]

ありがとうございます！初心者の僕でもなんとか理解できました＞＜
ただ、以下だけ理解ができなったです。ここでは何をやっているのでしょうか？

$index = 2;
foreach($columnList as $column){
$value = in_array($column, $checkbox) ? 1 : 0;
$stmt->bindValue($index, $value, PDO::PARAM_INT);

$index++;
}

[ssk]

Invalid parameter number: parameter was not defined in
試してみたのですが、このようなエラーが表示されてしまいました。。

[ssk]

$index = 1;
↑$index = 2;を$index = 1;に変更するとできました（$index = 1;に変えて合っているのか不安です。。）

$index = 1;
foreach($columnList as $column){
$value = in_array($column, $checkbox) ? 1 : 0;
$stmt->bindValue($index, $value, PDO::PARAM_INT);

$index++;
}

[KiyoshiMotoki]

むむ、動きませんでしたか。。

ともあれ、解決できたようで幸いです。

[退会済みユーザー]

$checkbox = $_POST['checkbox'];
だと、一つもチェックされていない時にエラーで、止まりますね。

$checkbox= (isset($_POST['checkbox'])) ? $_POST['checkbox'] : array();

[KiyoshiMotoki]

Kosuke_Shibuya様、ご指摘ありがとうございます。

確かにおっしゃる通りですね。

回答のコードを修正させていただきました。

[ssk]

こちらで再度、質問失礼します。

$index = 1;
↑$index = 2;を$index = 1;に変更するとできました（$index = 1;に変えて合っているのか不安です。。）
こちらで意図した動きをしたのですが、間違っていますでしょうか？

[退会済みユーザー]

横レスすみません。
KiyoshiMotoki さんの回答が正しく、$index = 2; でないと正しく、プレースホルダに値が割り当てされないと思いますが…。写経した時に、どこかおかしくなっているのではありませんか？もしくはDBの構造がKiyoshiMotokiさんの提示したものと異なっているか。

[KiyoshiMotoki]

少なくとも、私の意図した動きではありません。

私が書いたコードは、foreach文の直前に
$stmt->bindValue(1, $userCode, PDO::PARAM_STR);
という一行があり、1個目のプレースホルダ("?")にユーザーコードをバインドしています。

そのため、10個のチェックボックスの値は2個目から11個目のプレースホルダにバインドすることになるので、
$index = 2;
だったわけです。

$index = 1に変更して動いたということは、ssk様が書いたSQL文は
'INSERT INTO table (user_code, %s) VALUES (?, %s)',
ではなく
'INSERT INTO table (%s) VALUES (%s)',
になっていませんでしたか？

これだと確かにINSERT文はエラーなく実行されますが、
カラム'user_code'に値を登録していないため、
誰がチェックボックスにチェックを入れたのかが分からなくなるかと思います。

[ssk]

>'INSERT INTO table (%s) VALUES (%s)',
たしかに、このようになっています。

>誰がチェックボックスにチェックを入れたのかが分からなく
誰がチェックボックスにチェックを入れての、わからなくて良い場合はこのままでも大丈夫でしょうか？

[ssk]

>$index = 2; でないと正しく、プレースホルダに値が割り当てされないと思いますが…。
なぜか意図した動きになっていまして、あら？これで良いのかなと思っていました。。

[退会済みユーザー]

「なぜか」という言葉が出なくなることを期待します…w

私もKiyoshiMotokiさんもあなたが提示されたコードをそのままの状態で動かしているはずと思って回答しています。
あなたがどうそのコードをアレンジしたのかなんてエスパーではありませんから、あなたが書いた手元のコードを提示しない限り知る由もないのです。

回答する立場の人は同時に「初心者のいう『そのまま書いた』は信用してはいけない。」とも思っているので、「手元のコードを提示してください」と書かれている質問よく見かけますね。

[KiyoshiMotoki]

> 誰がチェックボックスにチェックを入れての、わからなくて良い場合はこのままでも大丈夫でしょうか？

そうすると、逆に、誰がチェックを入れたのか分かりたい場合に対応できないコードになってしまいますよ？

「そんな場合はない」
ということでしたら、構わないと思います。

[ssk]

>「なぜか」という言葉が出なくなることを期待します…w
すいません。。やっていることが理解できずでした。
※一応、そのまま書いたつもりです＞＜

$value = in_array($column, $checkbox) ? 1 : 0;
ここの「？」はどういう意味でしょうか？

[ssk]

>「そんな場合はない」
ということでしたら、構わないと思います。

そうなんですか！今のところは大丈夫なのですが、今後、今回のこととは別件で必要になるかもしれないので、理解だけはしておきます。

[退会済みユーザー]

> $value = in_array($column, $checkbox) ? 1 : 0;
> ここの「？」はどういう意味でしょうか？

「三項演算子」です。
http://php.net/manual/ja/language.operators.comparison.php#language.operators.comparison.ternary

> ※一応、そのまま書いたつもりです＞＜
いいってことよ…気にしなさんな。

[ssk]

参考URLまでありがとうございます！理解できました。

[KiyoshiMotoki]

> ここの「？」はどういう意味でしょうか？
まずは、ご自分で調べてみることをお勧めします。

厳しいことを申し上げますが、
ssk様が今後もエンジニアをお続けになるのであれば、
「誰の力も借りることができない」
というシチュエーションが、いつか必ず訪れます。

それに、私は
「知りたいことを自力で調べる」
というのも一種のスキルだと思っており、それを身に付けることは、
長い目で見れば、ssk様のエンジニアとしての成長に大きく影響すると思います。

調べてみて分からなければ、別途、質問を投稿するなどしましょう。

[ssk]

調べる範囲が甘かったかもしれません。

>知りたいことを自力で調べる」というのも一種のスキル
たしかに、そうですよね。色々なサイトを参考に、それでもわからなければ質問を投稿します！その際は恐縮ですが、お力添えいただけると助かります＞＜

[KiyoshiMotoki]

お力になれると良いですが・・・w

Answer 2

補足より

不動産の設備を登録するプログラムではないです（不動産のバス・トイレ別などは、あくまで一例です）。ジャンルで絞り込めるようにしたいのです。

具体的な例をあげないと、お互いに想定するケース、前提に相違が生じる恐れがあるため、「不動産」を例に説明します。

DBに「部屋」の情報を持つ room テーブルがあり、「設備」をどう持つかですが、通常「正規化」して、roomとは別に、equipment テーブル・設備マスターを持つべきです。こうしておくことで、設備の項目の増減に対応することが容易になります。

具体的には以下のような最小構成になります。

部屋テーブル

CREATE TABLE `room` (
  `id` int(11) unsigned NOT NULL AUTO_INCREMENT COMMENT 'ID',
  `name` varchar(16) DEFAULT NULL COMMENT '部屋番号',
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

部屋・設備リンクテーブル

CREATE TABLE `equipment` (
  `id` int(11) unsigned NOT NULL AUTO_INCREMENT COMMENT 'ID',
  `room_id` int(11) DEFAULT NULL COMMENT '部屋番号',
  `name` int(11) DEFAULT NULL COMMENT '設備名',
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

設備マスター

CREATE TABLE `master_equipment` (
  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
  `name` varchar(11) DEFAULT NULL COMMENT '設備名',
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

こうしておくことで、以下のようなSQLで抽出可能になります。

SELECT 
	room.* 
FROM 
	room
WHERE 
	room.id IN (
		SELECT 
			room_id 
		FROM 
			equipment 
		WHERE id IN (1, 2)
	)

html は以下のように出力するようにしておけばいいですね。

<form action="">
	<!-- master_euipment から動的に生成する -->
	<label><input type="checkbox" name="equipment[]" value="1" />オートロック</label>
	<label><input type="checkbox" name="equipment[]" value="2" />南向き</label>
	<label><input type="checkbox" name="equipment[]" value="3" />インターネット</label>
	<label><input type="checkbox" name="equipment[]" value="4" />対面キッチン</label>
</form>

---

追記

<?php
if (null != filter_input_array(INPUT_POST)) {
	$arrPost = filter_input_array(INPUT_POST);

	$room_id = $arrPost['room_id'];
	$arrEquipmentId = (isset($arrPost['equipment'])) ? $arrPost['equipment'] : [];

	//$dbh =

	foreach ($arrEquipmentId as $equipmentId) {
		$vals = [];
		$sql = 'INSERT INTO equipmet (id, room_id, euipment_id) VALUES (null, ?, ?)';
		$vals[] = $room_id;
		$vals[] = $equipmentId;

		$stmt = $dbh->prepare($sql);
		$stmt->execute($vals);
	}
}
?>
<!DOCTYPE HTML>
<html lang="en-US">
	<head>
		<meta charset="UTF-8">
		<title></title>
	</head>
	<body>
		<form action=""  method="post">
			<input type="hidden" name="room_id" value="1" />
			<!-- master_euipment から動的に生成する -->
			<label><input type="checkbox" name="equipment[]" value="1" />オートロック</label>
			<label><input type="checkbox" name="equipment[]" value="2" />南向き</label>
			<label><input type="checkbox" name="equipment[]" value="3" />インターネット</label>
			<label><input type="checkbox" name="equipment[]" value="4" />対面キッチン</label>
		</form>
	</body>
</html>

Comments

[ssk]

サンプルコードまで、ありがとうございます＞＜

ただ、知らいたいことは、チェックボックスが複数あり、チェックされたもの（当然、複数の可能性あり）の値をデータベースに登録したいのですが、その際の処理はどのようになりますでしょうか？

説明不足で申し訳ございません。

[退会済みユーザー]

ざっくり、追記しました。
実際には、バリデーションやトランザクション処理、CSRF対策用のコードも入れることになります。

[ssk]

ありがとうございます。

こちらはHTMLを動的に生成する方法でしょうか？
理解できず、すいません。。

[退会済みユーザー]

ちゃんと読みましょうよ…
INSERT INTO 〜　というSQLがあるのに「HTMLを動的に生成する方法」ですかとは…。

[ssk]

読んだのですが、わからなかったです＞＜

<label><input type="checkbox" name="equipment[]" value="1" />オートロック</label>
<label><input type="checkbox" name="equipment[]" value="2" />南向き</label>
<label><input type="checkbox" name="equipment[]" value="3" />インターネット</label>
<label><input type="checkbox" name="equipment[]" value="4" />対面キッチン</label>

こちらのチェックボックスを選択（複数も可）して

foreach ($arrEquipmentId as $equipmentId) {
$vals = [];
$sql = 'INSERT INTO equipmet (id, room_id, euipment_id) VALUES (null, ?, ?)';
$vals[] = $room_id;
$vals[] = $equipmentId;

$stmt = $dbh->prepare($sql);
$stmt->execute($vals);
}

こちらでデータベースに入れるのでしょうか？

[退会済みユーザー]

そうですよ。
実際に動かしてみればわかります。コードだけを眺めていたって何にもなりません。実際に動かして、試すことを繰り返すしか上達の道はありません。

動くコードを回答しているので動かしてください。

[ssk]

すいません、ありがとうございます。試しています。

[退会済みユーザー]

> 読んだのですが、わからなかったです＞＜
ただ漠然と「わからない」では、どう答えてあげればいいのかわからないので、どこまで理解できて、どこがわからないのかを明確に質問してください。

[ssk]

初学者で知識が乏しく。。以下のPHP、特に前半が何をやっているのか、どうやって「オートロック」や「南向き」の情報を渡しているのかがわかりませんでした。

<?php
if (null != filter_input_array(INPUT_POST)) {
$arrPost = filter_input_array(INPUT_POST);

$room_id = $arrPost['room_id'];
$arrEquipmentId = (isset($arrPost['equipment'])) ? $arrPost['equipment'] : [];

//$dbh =

foreach ($arrEquipmentId as $equipmentId) {
$vals = [];
$sql = 'INSERT INTO equipmet (id, room_id, euipment_id) VALUES (null, ?, ?)';
$vals[] = $room_id;
$vals[] = $equipmentId;

$stmt = $dbh->prepare($sql);
$stmt->execute($vals);
}
}
?>

[退会済みユーザー]

> どうやって「オートロック」や「南向き」の情報を渡している
HTMLにSUBMIT を設定していませんが、ボタンを設置して、クリックすれば送信できますよね。

[ssk]

$arrEquipmentId = (isset($arrPost['equipment'])) ? $arrPost['equipment'] : [];
ここでは、$arrEquipmentIdに「オートロック」などの値を入れているのでしょうか？

[退会済みユーザー]

デバッグの仕方はわかりませんか？

$arrEquipmentId = (isset($arrPost['equipment'])) ? $arrPost['equipment'] : [];
この行の下に、
var_dump($arrEquipmentId);
をいれて確認できます。

[ssk]

できました！！
しっかり読んで1つ1つ何をしているのか、理解します。
Kosuke_Shibuyaさん助かりました！＞＜

[退会済みユーザー]

ソースと合わせて、リファレンスをしっかり読むといいですよ。
読むといいよいうよりは、必須です。
http://php.net/manual/ja/funcref.php

初心者に限らず、エキスパートな人ほどリファレンスは読んでいるものです。

[ssk]

はい！
貴重なお時間をお付き合い頂きありがとうございました。