複数のgetパラメータと条件分岐について

Question

###前提：
【課題】ECサイト構築中
商品一覧画面にて、チェックボックスで商品を分けたい
メンズ/レディース/キッズ

##現状：

単体での各チェックボックスを押下した際の挙動は問題なし
複数のチェックボックス押下でうまくいかない。

①↓メンズとレディースを同時に選択した時に、一番最後のGETパラメーターのみ反映されます(ja=ledies)
(例)
http://～～product_all_test2.php?ja=&ja=men&ja=ledies

②↓結果
表示されるのは、レディースの商品情報のみ

###やりたいこと：
②で、表示されるのが、メンズとレディースの商品情報であること。
※複数のパラメーターを正常に処理したい。
if文での条件分岐の記載方法をお教えください。

 if ($_GET["ja"] == 'ledies'){

try{
	$dbh = new PDO($dsn, $user, $password);
	echo "接続成功";
	
	$sql = 'SELECT * FROM product';
	$statement = $dbh -> query($sql);
	
	$sql = 'SELECT * FROM product WHERE genre = 2';    //条件 genre=2(ledies)
	$statement = $dbh -> query($sql);

	//レコード件数取得
	$row_count = $statement->rowCount();
	
	while($row = $statement->fetch()){
		$rows[] = $row;
	}
	
	
	//データベース接続切断
	$dbh = null;
    
}catch (PDOException $e){
	print('Error:'.$e->getMessage());
	die();
}

} elseif ($_GET["ja"] == 'men'){
try{
	$dbh = new PDO($dsn, $user, $password);
	echo "接続成功";
	
	$sql = 'SELECT * FROM product';
	$statement = $dbh -> query($sql);
	
	$sql = 'SELECT * FROM product WHERE genre = 1';       ////条件 genre=1(men)
	$statement = $dbh -> query($sql);

	//レコード件数取得
	$row_count = $statement->rowCount();
	
	while($row = $statement->fetch()){
		$rows[] = $row;
	}
	
	
	//データベース接続切断
	$dbh = null;
    
}catch (PDOException $e){
	print('Error:'.$e->getMessage());
	die();
}

} elseif ($_GET["ja"] == 'kids') {
try{
	$dbh = new PDO($dsn, $user, $password);
	echo "接続成功";
	
	$sql = 'SELECT * FROM product';
	$statement = $dbh -> query($sql);
	
	$sql = 'SELECT * FROM product WHERE genre = 3';       ////条件 genre=3(kids)
	$statement = $dbh -> query($sql);

	//レコード件数取得
	$row_count = $statement->rowCount();
	
	while($row = $statement->fetch()){
		$rows[] = $row;
	}
	
	//データベース接続切断
	$dbh = null;
    
}catch (PDOException $e){
	print('Error:'.$e->getMessage());
	die();
}
} elseif ($_GET["ja"] == 'all') {
try{
	$dbh = new PDO($dsn, $user, $password);
	echo "接続成功";
	
	$sql = 'SELECT * FROM product';
	$statement = $dbh -> query($sql);
	
	$sql = 'SELECT * FROM product';       //すべてのチェックボックス押下
	$statement = $dbh -> query($sql);

	//レコード件数取得
	$row_count = $statement->rowCount();
	
	while($row = $statement->fetch()){
		$rows[] = $row;
	}
	
	
	
	//データベース接続切断
	$dbh = null;
    
}catch (PDOException $e){
	print('Error:'.$e->getMessage());
	die();
}

}

###試したこと

if ($_GET["ja"] == 'men' and 'ledies') {
----------略
$sql = 'SELECT * FROM product WHERE genre=1 or genre=2';  

チェックボックス：
①メンズとレディース
②メンズのみ
③レディースのみ

①の結果：
レディースの商品画像のみが表示された。
②の結果：
メンズとレディースの商品画像が表示された。
③の結果：
レディースの商品画像のみが表示された。

Answer 1

name[]のように配列にして送信してください。
配列を用いずに同名の場合はラジオボタンとほぼ同じ挙動です。1つしか送信されません。
で、SQLではINを利用すると良いです。

※未検証

<input type="checkbox" name="genre[]" value="1">
<input type="checkbox" name="genre[]" value="2">
<input type="checkbox" name="genre[]" value="3">

$where = "";
$genre = filter_input(INPUT_GET, 'genre',FILTER_DEFAULT,FILTER_REQUIRE_ARRAY);
if(!is_null($genre) && count($genre)> 0){
   $param = array_fill(0,count($genre),'?');
   $where = ' where genre in ('.implode(',',$param).')';
}
$sql = "SELECT * FROM product".$where;

$stmt = $pdo->prepare($sql);
$bindValue = [];
if(!is_null($genre) && count($genre)> 0){
  $bindValue = $genre;
}
$stmt->execute($bindValue);

Comments

[Roo]

勉強不足ですみませんが、上記内容一旦調べて試させていただきます。
無邪気にコピーして試したところ、
$stmt -> execute($bindValue);
ここで、おこられました。。

[m.ts10806]

まぁ、未検証なので。コメントも書いてませんしね。
長れの参考にしてもらったらと思って提示しています。
$genreに何が来ているかは確認しておいてください。エラーメッセージ次第ですし。
コードの後半はpdo使うための蛇足です。

[m.ts10806]

本来はtry-catchで囲ってPDOExceptionを捕捉するように書くべきものです。
https://qiita.com/mpyw/items/b00b72c5c95aac573b71

[Roo]

genreの中身は期待値通り、valueが格納されたのですが

$paramは　( [0] => ? [1] => ? )
$whereは　 where genre in (?,?)

となっており、宜しければ、提示していただいた文面に
コメントを当てていただきたいです。

[m.ts10806]

想定通りのSQLのはずです。
「おこられた」だけではわからないのでエラーメッセージを確認してください

[Roo]

現在エラー文言は出ておりません。

$param = array_fill(0,count($genre),'?');

疑問符に値が入っていないことから、
取得する情報がないように思えます。。

[m.ts10806]

疑問符パラメータです。
execute時に値を送ってバインドしてるはずです。

[Roo]

①print_r($param);
②print_r($where);
③print_r($bindValue);

①Array ( [0] => ? [1] => ? )
②where genre in (?,?)
③Array ( [0] => [1] => 2 )

そのため、
$sql = "SELECT * FROM product".$where;
が指定できてない状態です。

[m.ts10806]

今、htmlも含めてコードはどうなってますか？
質問本文に「追記」してください。(提示されたコードを更新するのではなく「試したこと」としての追記をお願いします)

[Roo]

m.ts10806様の提示されたコードに対し
我流でいくつかのコードを記載したことが原因となっておりました。
複数のパラメータの受領と接続
疑問符パラメーターも問題なく作用しました。

また、難航した場合、手助けいただけると幸いです。
ご丁寧にありがとうございました。

Answer 2

if文での条件分岐の記載方法をお教えください。

残念ながら、if文ではどうしようもありません。PHPが$_GETを生成する時点で、キーを上書きしてしまいますので、その後から値を取ろうとどう頑張っても、一番最後の値しか取れません。

対策としては2つ考えられます。

• キーの名前をja[]のようにして、PHP側に配列を送れるようにする
• JavaScriptをフォームに仕掛けて、同じキーに複数の値を送るという形ではない形にデータを変換する

Comments

[yambejp]

> どうしようもありません

厳密に言えば「$_SERVER["QUERY_STRING"]を解釈すればifでもなんとかなるんですけどね

[Roo]

PHPが$_GETを生成する時点で、キーを上書きしてしまいますので、、。
なるほど、故にですね。他の方々含め参考にさせていただきます。

Answer 3

まずはinputのnameを「ja」から「ja[]」に変更してください

その上で

$ja=filter_input(INPUT_GET,"ja",FILTER_DEAFULT,FILTER_REQURE_ARRAY);

で受けます

Comments

[Roo]

filter_input(INPUT_GET,"ja",FILTER_DEAFULT,FILTER_REQURE_ARRAY);
↑から調べます！ご指摘ありがとうございます