foreachでＤＢ内データを呼び出したとき、配列が文字列で保存されてしまう

Question

前提・実現したいこと

ＰＨＰで簡単な会員登録、登録情報の編集フォームを作成しています。
編集画面では、ＤＢに格納したデータを呼び出し、UPDATEしています。

![

５つのチェックボックスのチェック有無は　0_1_2_3_4　という形で保存しています。
※１つ目と３つめをチェックなら→　0_2 というデータを格納

編集のときDBから、trafficというkey内の、配列（0_2）呼び出しますが、
foreachしたあと文字列（Array）になってしまいますが、配列のまま上書きしたいです。

どこがおかしいのか判っていないため、アドバイスいただければ幸いです。

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

trafficの値が、配列から文字列になってしまう

http://localhost/phpmyadmin/の画面

ＤＢに格納しているKeyと値
Key : traffic
value : 0_1_2_3_4

UPDATE時に格納されるデータ
Key : traffic
value : Array

####var_dump($dataArr)した結果

ray(18) { 
["family_name"]=> string(6) "捕鯨" ["first_name"]=> string(6) "太郎" 
["family_name_kana"]=> string(6) "ほげ" ["first_name_kana"]=> string(12) "たろう" 
["sex"]=> string(1) "1" 
["year"]=> string(4) "1948" ["month"]=> string(2) "08" ["day"]=> string(2) "01" 
["zip1"]=> string(3) "110" ["zip2"]=> string(4) "0003" 
["address"]=> string(24) "東京都台東区根岸" ["email"]=> string(14) "[e-mail]" 
["tel1"]=> string(3) "090" ["tel2"]=> string(4) "0000" ["tel3"]=> string(4) "1111" 
["traffic"]=> array(2) { [0]=> string(1) "0" [1]=> string(1) "2" } 
["contents"]=> string(0) "" ["mem_id"]=> string(2) "14" }

####var_dump($column)した結果

string(291) "family_name='捕鯨',first_name='太郎',
family_name_kana='ほげ',first_name_kana='たろう',
sex='1',year='1982',month='05',day='03',
zip1='110',zip2='0001',address='東京都台東区谷中',
email='[e-mail]',tel1='090',tel2='0000',tel3='1111',
traffic='Array',contents='',mem_id='16',"

該当のソースコード

case 'update_complete'://編集からの登録

  unset($_POST['update_complete']);
  $dataArr = $_POST; //この時点ではtraffic内のvalueが配列になっている

  $column = '';

  foreach ($dataArr as $key => $value) {
    $column .= $key . '=' . "'" . $value . "'" . ',';

  }
  
  //traffic内のvalueが配列→文字列になっている

  $query = " UPDATE member SET "
    . $column
    . " update_date=11"
    . " WHERE "
    . " mem_id = "
    . $dataArr['mem_id'];

  $res = $db->execute($query); //SQL実行
  $db->close(); //DB Close

  var_dump($dataArr);
  var_dump($column);

補足情報（FW/ツールのバージョンなど）

XAMMPローカル環境
MySQL

Answer 1

こういうコードを見ていると、POST送信するデータに細工されたらデータベースを破壊されそうだなって思っちゃって、なかなか本筋の問題に目が向かなくなっちゃうところですが。
フォーム送信されたデータを適切に検証してから保持するべきです。

たぶん、ラクするために配列に$_POSTデータを全ツッコミしていることが良くないのだと思います。
チェックボックスのデータをどう受けているかを点検するべきです。

一見難しそうなfilter_input_array()を駆使したほうが、
その後の処理が質問者さんが想定する配列のイメージで扱いやすくなりそうに見えますね。

下記、他の方の回答からヒントを得て：

  foreach ($dataArr as $key => $value) {
    if (is_array($value)) {
      $column .= $key . '=' . "'" . implode('_', $value) . "'" . ',';
    } else {
      $column .= $key . '=' . "'" . $value . "'" . ',';
    }
  }

とかすれば、一応狙ったとおりになるんだろうか、うごかしてないので、なんとも。

Comments

[Hirakin3]

初心者で不正対策まで念頭にありませんでした。勉強になります。

Answer 2

$_POST['traffic']が配列になっていてforeach内でシングルクォートでくくっているので文字列になってしまうのでは？
上手く説明できないのですが、以下のような感じでどうですか？？

  
  // 0_1_2_3形式にする
  $_POST['traffic'] = implode('_', $_POST['traffic']);
  
  $column = '';

  foreach ($dataArr as $key => $value) {
    $column .= $key . '=' . "'" . $value . "'" . ',';

  }

Comments

[退会済みユーザー]

Array(型)だよ、って返しているのをシングルクォートをつけて文字列'Array'にしちゃっている、それだ！

[Hirakin3]

指摘の点をまだ理解できていないため、こちらの方法で試してみます！

Answer 3

考え方がいくつかあるのですが手っ取り早く

• チェックボックスの項目を利用してDBにデータを登録するなら「set型」を使う
• 特に制限なくデータの羅列でいいなら文字列型でもよい
• チェックボックスの項目が高い頻度で不特定に増減するなら別途正規化を行う

上記1番の例をあげておきます

create table tbl(
id int not null primary key auto_increment,
v1 set('A','B','C') not null,
v2 varchar(20) not null
);
insert into tbl(v1,v2) values('A','A'),('',''),('A,B','A,B'),('A,B,B','A,B,B'),('C,A,B','C,A,B');

上記、v1とv2の挙動の違いは以下

• set型なら重複をゆるさない
• set型なら投入順にかかわらずsetに登録順にデータが格納される

さらに、set型はデータ自体がビットで登録されているため
データ投入にbit和を利用できます

insert into tbl(v1,v2) values(1,1),(5,5);

※上記v1=1→「A」が登録される
v1=5→「A,C」が登録される

このデータからv1にAを含むレコードを取り出すためにはfind_in_set関数を利用します。

select * from tbl where find_in_set('A', v1);

実はfind_in_set関数は文字列型でも検索が可能です

select * from tbl where find_in_set('A', v2);

Comments

[Hirakin3]

指摘の点をまだ理解できていないため、こちらのＳＥＴでの方法も試してみます！