特定の条件時に、一部のwhere句を無効にしたい

Question

###前提・実現したいこと
PHP+MySQLでDBを操作しています。
表示名（例：「夏の野菜（葉物）」）を分解した後にSQLに埋め込んでいるのですが、
一部だけwhere文を省略したい部分があります（例：「夏の果物」）。

ISNULLなどを試してみましたが、
「夏の果物」は表示できても、「夏の野菜（葉物）」が表示できなくなったりしました。
$type_arrの要素数を数えて、if文で分岐させるしかないのでしょうか。

###該当のソースコード

  public function getList($type, $maker)
  {
    $type = $this->typeEdit($type);

    //$typeを分解する
    $type_arr = explode('・', $type);
    array_push($type_arr, $maker);

    $sql = 'SELECT * FROM data
      where data.item_type_1 =?
      and data.item_type_2 =?
      and data.item_type_3 =?　//←これを無効にしたいです
      and data.item_maker_id =?';
    $stmt = $this->pdo->prepare($sql);
    $stmt->execute($type_arr);
    $this->rows = $stmt->fetchAll();
    $this->listEdit();
    return $this->rows;
  }

  protected function typeEdit($type)
  {
    switch($type){
      case '夏の野菜（葉物）':
        $type = '野菜・夏・葉物';
        break;
      case '夏の野菜（根物）':
        $type = '野菜・夏・根物';
        break;
      case '冬の野菜（葉物）':
        $type = '野菜・冬・葉物';
        break;
      case '冬の野菜（根物）':
        $type = '野菜・冬・根物';
        break;
      case '夏の果物':
        $type = '果物・夏';
        break;
      case '冬の果物':
        $type = '果物・冬';
        break;
      default:;
    }
    return $type;
  }

Answer 1

こんな感じですかね？

$sql ='SELECT * FROM data WHERE 1 ';
if(count($type_arr)>1) $sql.='AND item_type_1 =? ';
if(count($type_arr)>2) $sql.='AND item_type_2 =? ';
if(count($type_arr)>3) $sql.='AND item_type_3 =? ';
$sql.='AND item_maker_id =? ';

キーワードを文字列におきかえて分解するのは無駄では？
置き換えの際に配列にすればすこし効率がよくなると思います

Comments

[chibi144]

ご回答ありがとうございました。ご指摘の通りです。最初から配列にすることにしました。

Answer 2

SQL動的に変える方が良いと思いますが・・・coalesceでやってみました。

public function getList($type, $maker)
  {
    $type = $this->typeEdit($type);

    //$typeを分解する
    $type_arr = explode('・', $type);
    if (count($type_arr)==2) array_push($type_arr, NULL);
    array_push($type_arr, $maker);

    $sql = 'SELECT * FROM data
      where data.item_type_1 =?
      and data.item_type_2 =?
      and data.item_type_3 =coalesce(?,data.item_type_3)
      and data.item_maker_id =?';
    $stmt = $this->pdo->prepare($sql);
    $stmt->execute($type_arr);
    $this->rows = $stmt->fetchAll();
    $this->listEdit();
    return $this->rows;
  }

  protected function typeEdit($type)
  {
    switch($type){
      case '夏の野菜（葉物）':
        $type = '野菜・夏・葉物';
        break;
      case '夏の野菜（根物）':
        $type = '野菜・夏・根物';
        break;
      case '冬の野菜（葉物）':
        $type = '野菜・冬・葉物';
        break;
      case '冬の野菜（根物）':
        $type = '野菜・冬・根物';
        break;
      case '夏の果物':
        $type = '果物・夏';
        break;
      case '冬の果物':
        $type = '果物・冬';
        break;
      default:;
    }
    return $type;
  }

Comments

[chibi144]

ご回答ありがとうございます。
coalesce、今まで知りませんでした。
null（根物・葉物がないときの初期値）をdata.item_type_3にひっくり返すイメージなんですね。

Answer 3

ifnullやcaseなどで特定条件下で値そのもの(data.item_type_3)を返せば
全件一致(無効化)できませんか？

SELECT * FROM data
where data.item_type_3=ifnull(?,data.item_type_3)

こんな感じなら？がnullのときにdata.item_type_3=data.item_type_3になるので
実質無効化できそうですが。

Comments

[chibi144]

ご回答ありがとうございます。
今回は他の方法で解決したのでifnullは使わなかったのですが、
無効化自体は今後も使いそうなので、覚えておきます！

Answer 4

自分好みに

$sql = 'SELECT * FROM data WHERE 1 ';
foreach($type_arr as $i=>$type){
    $sql.=' AND data.item_type_'.($i+1).' = ?';
}
$sql.=' AND data.item_maker_id =?';

同じ文字列を３回以上書いてはならない掟

function parseTypeStr($type){
    preg_match('/^([春夏秋冬])の(.+?)(（(.+)）)?$/',$type,$matches);
    $rtn=[$matches[2],$matches[1]];
    if($matches[4])$rtn[]=$matches[4];
    return $rtn;
}

どうせ秋冬が増えたりするんでしょ、知ってるんだから///

Comments

[chibi144]

ご回答ありがとうございました。ベストアンサーをつけるのを忘れていました、すみません。
<< 同じ文字列を３回以上書いてはならない掟
よくやらかしてますので、肝に命じます。
なお、秋冬は増えませんでした///