チェックボックス（複数選択可）をデータベースに登録

Question

バージョン

CakePHP3

やりたい事

・チェックボックス（複数選択可）をデータベースに登録
・◯◯；◯◯；◯◯と1つのカラムに登録

できている事

データベースのカラム（◯◯；◯◯；◯◯）と照合して既に値がある場合はcheckedにする。
ところまで、完成しています。その他のファイルはいじっていません。

✓edit.ctp

echo $this->Form->input('hoge29',[   
				  'type'   => 'select',
				  'options'  => $checkbox,
				  'value'  => explode(';', $hoges->hoge29),
				  'checked' => true,
				  'multiple' => 'checkbox',
				  'label'  => false,
				]);

わからない事

・edit.ctpで選択されたチェックボックス（配列）の受け取り方
・その値を◯◯；◯◯；◯◯という形式で1つのカラムに追加・更新する方法

以上です。
宜しくお願い致しますm(_ _)m

Answer 1

コメント等のやり取りを見て、思うところですが、

このような要件の場合、データベースは正規化すべきと思われます。
あえて非正規化している理由は何かあるのでしょうか？

---

#現状の設計で起こる問題

一つ前の質問を参照し、現状で起こりうる問題の具体例を示します。

例えば、以下のようなデータになっている場合、「コーヒー」を含むレコードをどうやって取り出しますか？

ID	CHECKBOXES
1	コーヒー,紅茶,ココア
2	コーヒー牛乳,紅茶
3	コーヒーゼリー

Comments

[ssk]

Kosuke_Shibuya様

コメントありがとうございますm(_ _)m
現状、データの構成上、1つのカラムで◯◯；◯◯；◯◯と表示しなければいけない状態です。

[退会済みユーザー]

正規化した上で、表示はGROUP_CONCAT 使えば済むことなので、自分だったら正規化しますね。
ソースコードが複雑になりだけだし、検索方法も限定されてしまうので。

[ssk]

正規化とは、◯◯；◯◯；◯◯を1つのカラムで表示
ではなく、1つ１つカラムを分けるということでしょうか？

[退会済みユーザー]

違いますよ。
具体的な例で説明した方がわかりやすいかと思うのですが、今回使用しているテーブル名と、チェックボックスの値に利用している名前はなんですか？
また、チェックボックスには具体的にどのような項目がありますか？

[ssk]

Kosuke_Shibuya様

以下、現在開発環境で行っている情報でございます。
情報は足りますでしょうか？

テーブル名：drinks
チェックボックスの値：■コーヒー　■紅茶　■ココア

✓具体的なHTML
<input type="checkbox" name="drink[]" value="コーヒー">
<input type="checkbox" name="drink[]" value="紅茶">
<input type="checkbox" name="drink[]" value="ココア">

[退会済みユーザー]

これはどういった意味合いになるのでしょうかね。

例えば、「店舗」と「取扱商品」といった感じですか？

[ssk]

Kosuke_Shibuya様

具体的なテーブルまでご提示いただき、ありがとうございますm(_ _)m
コーヒー◯◯というケースがある場合は難しいですね、、

どのように正規化するのがベストでしょうか？

[退会済みユーザー]

それがどういうシステムなのか、概略をお伝え願います。

[ssk]

CakePHPの勉強中でデータベースの設計まで回っていませんでしたm(_ _)m
クライアント管理に利用しようと考えています。

[退会済みユーザー]

あまりぼんやりとした回答だと、サンプル作れないし、sskさんの技量わからないので、
・クライアントとは何？（店？個人？）
・クライアントと「ドリンク」ってどういう意味合いで関連しているのか
をお伝え下さい。

[ssk]

クライアントは個人の方です。
ドリンクというのはあくまでサンプルでして
これから自分に合わせてプログラミングをしていきます。

今回では、クライアントに定休日を紐付ける場合
チェックボックスで行おうとしていましたm(_ _)m

その実装をするために、開発環境で
わかりやすくドリンクで進めていた次第です。

[退会済みユーザー]

解決済になっていますが、正規化のサンプルの提示は不要？
それとも新しく質問を立てる感じですか？

[ssk]

質問した内容は解決しましたので、解決済とさせていただきましたm(_ _)m

後程、必ず躓くと思うので
正規化も是非、教えていただければ幸いです。

解決済にしたため、誤解を招いてしまい申し訳ございません。

[退会済みユーザー]

この手の類似質問は定期的に見受けられるので、お手数ではありますが、ぜひ新しく質問を立ててください。sskさんのみならず、検索でやって来た同様の初学者のためにもなりますので。

例示としては、「取引先」と「定休日」で良いかと思います。

[ssk]

Kosuke_Shibuya様

https://teratail.com/questions/55926?modal=q-comp
↑こちらで新しい質問を立てさせていただきました。是非、よろしくお願い致しますm(_ _)m

Answer 2

saveする前に、配列をimplode関数で文字列に変換すればいいと思います。patchEntityより前に文字列にしないと不正な値と判断されて捨てられてしまいます。Controllerでやるなら、patchEntity前までに変換するといいかと。Tableクラスでやるなら、beforeMarshalでやる感じでしょうか。

Comments

[ssk]

HogesController.phpの以下でデータベースに登録していると思うのですが
どのように受け取って、データベースに追加できますか？

public function edit($id = null)
{
$hoges = $this->Hoges->get($id, [
'contain' => []
]);
if ($this->request->is(['patch', 'post', 'put'])) {
$hoges = $this->Hoges->patchEntity($hoge, $this->request->data);
if ($this->Hoges->save($hoge)) {
$this->Flash->success(__('The hoge has been saved.'));

return $this->redirect(['action' => 'index']);
} else {
$this->Flash->error(__('The hoge could not be saved. Please, try again.'));
}
}
$this->set(compact('hoge'));
$this->set('_serialize', ['hoge']);
}

icchii様、度々ありがとうございますm(_ _)m

[popobot]

postデータは$this->request->dataに格納されています。debug($this->request->data);等で出力して見て確認してください。データベースへの更新は、$this->Hoges->save($hoge)で実行されます。
なので、その間で文字列化したらいいと思いますけど

[popobot]

よくよく考えると...patchEntityしたときに、文字列ではなく配列のままだと、不正な値と判断されて、捨てられてしまうので、patchEntity前に文字列化しないとだめですね...。

[ssk]

ありがとうございます。
以下の形で$hoge29に◯◯；◯◯；の文字列を入れるところまでできました。
この$hoge29をデータベースに反映するには、どのように書いたらよいでしょうか？

if ($this->request->is(['patch', 'post', 'put'])) {
$hoge29 = $this->request->data('hoge29');
$hoge29 = implode(';',$hoge29);
$hoges = $this->Hoges->patchEntity($hoge, $this->request->data);
if ($this->Hoges->save($hoge)) {
$this->Flash->success(__('The hoge has been saved.'));

return $this->redirect(['action' => 'index']);

[ssk]

icchii様

以下のコードで意図した結果を得られました。
正常に動作はしているので、合っているとは思うのですが
「こう書いた方が良い」などございましたら、教えて頂けると後学のために助かりますm(_ _)m

if ($this->request->is(['patch', 'post', 'put'])) {
$hoge29 = $this->request->data('hoge29');
$hoge29 = implode(';',$hoge29);
$this->request->data['hoge29'] = $hoge29;
$hoges = $this->Hoges->patchEntity($hoge, $this->request->data);
if ($this->Hoges->save($hoge)) {
$this->Flash->success(__('The hoge has been saved.'));

return $this->redirect(['action' => 'index']);

[popobot]

それで大丈夫だと思いますよ。
Tableクラス側に書くこともできますが、やや複雑になるので、あまりおすすめしません。

[ssk]

かしこまりましたm(_ _)m
ありがとうございます！