修正

2016/12/06 06:15

なぜ前回の質問が終わってないのにこちらに立てたのでしょう？内容も同じように見受けられますが？

2016/12/06 07:04

コメントありがとうございます。前回の質問は、一つのテーブルのidの値を、別のテーブルにフォームで送信する方法でした。今回の質問は、ループしているチェックボックスで、複数選択したものをどのようにしてデータベースに値を送るかでした。言葉足らずで申し訳ありません。ですが、考えてみると同じような質問ですね、申し訳ありません。

2016/12/06 07:05

一つの質問が解決してから、次の質問をすべきでした……

2016/12/06 08:58

いえ、ループが肝要な部分であれば私の認識違いかもしれません。申し訳ありません。

行動規範の内容に同意します

回答2件

ベストアンサー

関係ないかもしれませんが、idをインクリメントするなら
オートインクリメントにすればよいのでは？
現行の方法にこだわるとしてもSQLで特殊な文字は別名をつけた方がよいでしょう

PHP
1$recordSet = mysqli_query('SELECT COUNT(id) as count_id FROM dish);
2・・・
3$id = $data['count_id']+1;

フロー的にはfilter処理をすればきれいに収まるとおもいます

PHP
1<?PHP
2$id="hoge";
3$dishId    = filter_input(INPUT_POST,'dishId',FILTER_DEFAULT,["options"=>["default"=>""]]);
4$dishMenu1 = implode('、',filter_input(INPUT_POST,'dishMenu1',FILTER_DEFAULT,["flags"=> FILTER_REQUIRE_ARRAY,"options"=>["default"=>[]]]));
5$dishMenu2 = implode('、',filter_input(INPUT_POST,'dishMenu2',FILTER_DEFAULT,["flags"=> FILTER_REQUIRE_ARRAY,"options"=>["default"=>[]]]));
6$dishMenu3 = implode('、',filter_input(INPUT_POST,'dishMenu3',FILTER_DEFAULT,["flags"=> FILTER_REQUIRE_ARRAY,"options"=>["default"=>[]]]));
7$dishMenu4 = implode('、',filter_input(INPUT_POST,'dishMenu4',FILTER_DEFAULT,["flags"=> FILTER_REQUIRE_ARRAY,"options"=>["default"=>[]]]));
8print 'INSERT INTO dish SET id="'.$id.'",dishId="'.$dishId.'",dishMenu1="'.$dishMenu1.'",dishMenu2="'.$dishMenu2.'",dishMenu3="'.$dishMenu3.'",dishMenu4="'.$dishMenu4.'"';
9?>
10<hr>
11<form method="post">
12dishMenu1<br>
13<input type="checkbox" name="dishMenu1[]" value="1">1
14<input type="checkbox" name="dishMenu1[]" value="2">2
15<input type="checkbox" name="dishMenu1[]" value="3">3<br>
16dishMenu2<br>
17<input type="checkbox" name="dishMenu2[]" value="x">x
18<input type="checkbox" name="dishMenu2[]" value="y">y
19<input type="checkbox" name="dishMenu2[]" value="z">z<br>
20<input type="submit" value="go">
21</form>

修正

まず、新たにご提示されたソースの中でidがユニークでない箇所が散見されます
つかってないidにようにみえるのでなくていいんじゃないでしょうか？
あとはdishMenu2はcheckboxではなさそうなのでfilterの仕方がちょっと変わってきます

PHP
1<form method="post">
2<div class="question" align="center">
3<p align="left" style="display:inline-block;">
4<input type='checkbox' name='dishMenu1[]' value='うああ'>うああ<br>
5<input type='checkbox' name='dishMenu1[]' value='うおおお'>うおおお<br>
6<input type='checkbox' name='dishMenu1[]' value='ふみゅ'>ふみゅ<br>
7<input type='checkbox' name='dishMenu1[]' value='ふ'>ふ<br>
8<input type='checkbox' name='dishMenu1[]' value='ぐ'>ぐ<br>
9sonota<input type='text' name='dishMenu2' value=''>
10<input type='hidden' name='question_id' value='1'></p></div>
11<div align="right">
12<input type="submit" value="送信する" style="width:100px; height:30px;">
13</div>
14</form>
15
16<?PHP
17$dishMenu1 = implode('、',filter_input(INPUT_POST,'dishMenu1',FILTER_DEFAULT,array("flags"=> FILTER_REQUIRE_ARRAY,"options"=>array("default"=>array()))));
18$dishMenu2 = filter_input(INPUT_POST,'dishMenu2',FILTER_DEFAULT,array("options"=>array("default"=>"")));
19print $dishMenu1;
20print "<hr>";
21print $dishMenu2;
22?>

追記

送られてきたものが配列かどうか確認して変数に代入する

$dishMenu1 = getPostData('dishMenu1');
$dishMenu2 = getPostData('dishMenu2');
print $dishMenu1;
print "<hr>";
print $dishMenu2;

function getPostData($key){
  if(isset($_POST[$key])){
    if(is_array($_POST[$key])){
      $ret=implode("、",$_POST[$key]);
    }else{
      $ret=$_POST[$key];
    }
  }else{
    $ret="";
  }
  return $ret;
}

投稿2016/12/08 02:54

編集2016/12/12 06:55

総合スコア114896

2016/12/08 05:40

回答ありがとうございます！サンプルコードのほうもありがとうございます！実行してみましたところ、 $dishId = filter_input(INPUT_POST,'dishId',FILTER_DEFAULT,["options"=>["default"=>""]]); の部分で[]部分がおかしいとのエラーが出力されてしまいました。。。また、質問のコードにあります通り、ラジオボタン、チェックボックスともにデータベースの１つのidで別々の値を取得してきておりますので、 name="dishMenu1[]" value="1" のような表記は少々難しいです。。。宜しくお願いいたします！

2016/12/09 00:38

>[]部分がおかしい PHPのバージョンが古いのでしょうか？具体的なエラーを提示してください >name="dishMenu1[]" value="1" もとのソースで「$_POST[dishMenu1]」をimplodeしようとしているのでてっきり配列で渡しているのかと思いましたが違うのですね PHPで書いている送り元のmenu.php をHTML形式で質問に追記してください

2016/12/12 01:24

[]付近の構文エラーと表示されます。追記しました！

2016/12/12 01:35

話がかみあってないようですが・・・「menu.phpをブラウザで表示し、ソースをHTML形式で提示してください」と書いたつもりですが、そうなっていないようです。 PHPのバージョンの問題であれば $dishMenu1 = implode('、',filter_input(INPUT_POST,'dishMenu1',FILTER_DEFAULT,["flags"=> FILTER_REQUIRE_ARRAY,"options"=>["default"=>[]]])); ↓↓↓ $dishMenu1 = implode('、',filter_input(INPUT_POST,'dishMenu1',FILTER_DEFAULT,array("flags"=> FILTER_REQUIRE_ARRAY,"options"=>array("default"=>array())))); になります。いずれにしろHTMLソース次第なので提示されない限り答えようがありません

2016/12/12 02:12

申し訳ありません、追記修正させていただきました！

2016/12/12 03:16

修正版を追記しました。エラーが出るようならPHPのバージョンも付記してください

2016/12/12 03:17

今回はあくまでテストなのでsend.phpに送らず自分でうけるサンプルにしてあります

2016/12/12 06:30

追記有難うございます！バージョンは5.3.1でした。はじめに質問させていただいた通り、ラジオボタンやチェックボックスはforやifを使い、formatというidに入っている値から判定して取得してきております。（radioという値が入っている場合はラジオボタンを取得、というようなものです）ですので、PHP部分の変数取得部分（$dishMenu1...の部分です）は、ラジオボタンの場合はimplodeで取得せず、チェックボタンの場合はimplodeで取得し、結果をデータベースに渡すというようなことを行いたいです。（質問が下手で申し訳ありません。）

2016/12/12 06:45

dishMenu1が送られてきたものをチェックするのであれば filter_inputにdefaultを指定せず、is_arrayでしらべた上で配列ならimplodeするそうでなければそのままの値を取るということでよいでしょう。むしろ送られてくるものがラジオボタンなのかチェックボックスなのかその他なのかは重要ではありません。配列で送られてくるものとしては、チェックボックスは顕著ですがセレクトボックスだってありえますし、通常のテキストボックスの可能性もあります

2016/12/12 06:56

issetとis_arrayでチェックする例を上げておきます

2016/12/13 05:16

ありがとうございます！issetとis_arrayを使う方法を試させていただきましたところ、無事実行ができました！お忙しいところ大変ありがとうございました。ベストアンサーとさせていただきました！

行動規範の内容に同意します

まずはループで取得している値を確認してみて下さい。

ループしている値を"echo"や"var_dump"で出力してみてください。

for($j=1; $j<=10; $j++){
          echo '$j : '.$j.'=> $col[\'format\'] : '.$col['format'].'<br>';
}

値が意図通りに取れていれば問題は分岐にあり、比較対象、もしくは比較方法そのものに問題がある可能性が高いです。

加筆（認識違いな気がしたので・・・）

implode部分が上手くいかず**”query発行にて問題がある”**とすれば単純に入力値の問題です。
implodeでダメならDBのフィールドの型と値が合っているか、キチンとエスケープしてqueryが壊れていないかなど、query部分を検証するのが良いかと思います。

投稿2016/12/06 09:09

編集2016/12/06 09:21

総合スコア199

2016/12/07 02:26

追記文確認致しました。ではfomat部には問題ないという認識でいきます。次は $dishMenu1 = implode('、',$_POST[dishMenu1]); $dishMenu2 = implode('、',$_POST[dishMenu2]); $dishMenu3 = implode('、',$_POST[dishMenu3]); $dishMenu4 = implode('、',$_POST[dishMenu4]); ですが、dishMenu1~dishMenu4はdefineで定義されているのでしょうか？ここのkeyを生成する部分はどのようになっているのでしょう。

2016/12/08 02:18

ありがとうございます！変数はdefineで定義しておりません。申し訳ありません、keyとはどのようなものでしょうか？