質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

91.23%

  • PHP

    15706questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • SQL

    1759questions

    SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

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

解決済

回答 4

投稿

  • 評価
  • クリップ 1
  • VIEW 99

ark0214

score 37

前提・実現したいこと

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;
  }
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 4

+2

こんな感じですかね?

$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 =? ';

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/01/19 10:10

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

    キャンセル

checkベストアンサー

0

自分好みに

$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 =?';


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

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


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/01/19 10:09

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

    キャンセル

0

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になるので
実質無効化できそうですが。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/01/19 10:11

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

    キャンセル

0

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;
  }

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/01/19 10:16

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

ただいまの回答率

91.23%

関連した質問

同じタグがついた質問を見る

  • PHP

    15706questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • SQL

    1759questions

    SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。