PHPでの値の受け取り方について

Question

$dbo01_str 	= isset($this->request->query['dbo01']) && is_string($this->request->query['dbo01']) 	? $this->request->query['dbo01'] 	: FALSE;

isset($dbo01_str) && $dbo01_str != '' ? $query->where(['dbo01 LIKE' => '%'.$dbo01_str.'%']) : FALSE;

$this->request->query['dbo01']には文字列または''が入ります。
それを判定して、$dbo01_strに入れます。

$dbo01_strを判定して処理をするのですが、独学のため書き方が不安です。。
皆さんはどのように受け取り、どのように判定していますか？

Answer 1

三項演算子を何かを実行するために使ってはいけません。
if文を使いましょう。
何のクエリービルダーを使っているか書いたほうがいいですね。

$request = $this->request->query; 
if ($request && isset($request['dbo01']) && $request['dbo01'] != '') {
  $query->where(['dbo01 LIKE' => '%'.$request['dbo01'].'%']);
}
```こんな感じでしょうか。

Comments

[ssk]

回答ありがとうございます。

>>>三項演算子を何かを実行するために使ってはいけません。
ということは

$request = $this->request->query;
$dbo01_str = isset($request['dbo01']) && is_string($request['dbo01']) && $request['dbo01'] !== '' ? $request['dbo01'] : FALSE;

もNGということでしょうか？

[shi_ue]

わたしが言っているのは、質問の二行目です。代入に使う分には問題ありません。

また、is_stringを$this->request->query['dbo01'] に使っていますが、外部から受け取ったものは、大体文字列になっていると思うのですが・・・まあ、違うのかもしれませんね。

[ssk]

なるほど。ありがとうございます。
その助言の通り、書き換えます。