cakephp2 検索条件の設定方法について

Question

cakePHP2の検索方法についてご教示願います。

会員情報管理システムにおいて、会員の回答有無を検索する検索機能（検索フォーム）を付けようとしておりますが、
コントローラー内での検索条件設定が合ってないようです、、

①会員の回答有りの場合→Viewから「1」をcontrollerへ送って検索
②会員の回答無しの場合→Viewから「２」をcontrollerへ送って検索

したいです。

①の空欄じゃ無い場合の、検索条件部分のみ上手く動かず、困っております。

Viewファイルで以下のように、該当検索フォームの条件を記載しました。

$this->Form->input(
 'person_answer_exists',
  array(
     'type' => 'select',
     'multiple' => 'checkbox',
     'options' => array(1 =>'あり' , 2 => 'なし'),
     'label' => false,
     )
)

対応するコントローラーでの検索条件の記載をしました。
・対象DBテーブル　Persom
・対象DBカラム　pe_contents
・①の場合のViewから送られてくる値　
$this->$request->data('Person.person_answer_exists　＝ array{[0] => 1)

if($this->$request->data('Person.person_answer_exists' === 1)){
       $condition[ 'Person.pe_contents' ] != '';
}

不足情報等ございましたら、お知らせください。

Answer 1

色々と構文が間違っています。

$this->$request だと、$request変数に格納されている名前の$thisのプロパティという意味になります。何もしていなければ $request = nullなので、$this->(null) ということでエラーが発生するはずです。

$this->request が正しいです。

---

$this->request->data()(CakeRequest::dataメソッド)は、$_POST['data']の配列からドット区切りで値を取得します。

POST データにアクセス | リクエストとレスポンスオブジェクト - 2.x

$this->request->data('Person.person_answer_exists' === 1) と書くと、'Person.person_answer_exists' === 1の方が先に評価されますので、$this->request->data(false) と同義です。

これは、(int)$this->request->data('Person.person_answer_exists') === 1 と書きましょう。
(int)を頭に付けているのは整数型に型変換するためです。
POSTデータは通常文字列として得られます。1ではなく'1'が返ってきます。
===は型を含めた比較ですので、型変換をしないと'1' === 1の比較となり false になります。

厳密に判定するなら上記の書き方ですが、値が0,1しかないのであれば、 if ($this->request->data('Person.person_answer_exists')) でも大丈夫です。

---

$condition[ 'Person.pe_contents' ] != '' は、$condition配列の'Person.pe_contents'キーと、空文字列を比較しているだけで代入をしていません。

Person.pe_contentsが空文字のレコードに制限したいのであれば、

$condition['Person.pe_contents !='] = '' とします。

ただし、pe_contentsにNULL制約があり、nullは格納されていないことが保証されていないと意図した結果にならないと思います。

その場合、Person.pe_contents IS NOT NULL も条件に加えると良いでしょう。

また、モデルのfind関数の条件パラメーター名はconditionsですので変数名も合わせておくと色々と楽です。

---

以上をふまえると、以下になります。

if((int)$this->request->data('Person.person_answer_exists') === 1){
       $conditions['Person.pe_contents !='] = '';
       $conditions[] = 'Person.pe_contents IS NOT NULL'; // これはDBスキーマ次第で必要
}

追記

person_answer_existsが複数値(multiple)ということなので、$this->request->data('Person.person_answer_exists') の返り値は、['1'], ['2'], ['1', '2'], null のいずれかになりますよね。

POSTされる値が配列なので in_array を使ってフラグが指定されてるか確認します。

PHP: in_array - Manual

また、条件が複数指定された場合にいずれかに該当するレコードを取得したいのであれば条件を OR で繋ぎます。

複雑な検索条件 - データを取得する - 2.x

$personAnswerExists = $this->request->data('Person.person_answer_exists');

if ($personAnswerExists !== null) { // person_answer_existsが指定されていなければ null が返るので除外しておきます。
   $personAnswerExists = array_map('intval', (array)$personAnswerExists); // 配列化して、個々の値を整数変換しています

   $conditions['OR'] = [];

   if (in_array(1, $personAnswerExists, true)) {
       $conditions['OR'][] = ['Person.pe_contents !=' => ''];
   }

   if (in_array(2, $personAnswerExists, true)) {
       $conditions['OR'][] = ['Person.pe_contents' => ''];
   }
}

Comments

[Tannnnnnnnnnn]

ご回答ありがとうございます。
お陰様でViewから１が送られる時に検索できるようになりました。
しかし、Viewから２が送られる場合も、１が送られて来た時と同様の検索結果となってしまいます。
以下、コントローラの記載部分ですが、何か間違いがございますでしょうか。
お手数おかけしますが、ご確認お願いします。

if((int)$this->request->data('Person.person_answer_exists') === 1){
$conditions['Person.pe_contents !='] = '';
}
if((int)$this->request->data('Person.person_answer_exists') === 2){
$conditions['Person.pe_contents '] = '';
}

[nojimage]

$conditions['Person.pe_contents '] に余分なスペースが含まれていませんか？また、pe_contentsはNOT NULLなフィールドということでよいでしょうか。

[Tannnnnnnnnnn]

余分なスペースは含まれておりませんね、、、。
NOT　NULLなフィールドです。

[nojimage]

$conditions['Person.pe_contents '] $conditions['Person.pe_contents']

[Tannnnnnnnnnn]

語弊がありました。そのスペースは削除し、実行しております。

[Tannnnnnnnnnn]

if((int)$this->request->data('Person.person_answer_exists') === 1){
$conditions['Person.pe_contents !='] = '';
}
if((int)$this->request->data('Person.person_answer_exists') === 2){
$conditions['Person.pe_contents'] = '';
}

スペースを削除し、上記で実行しておりますが「１」が送られてきたパターンしか実行されず、、

[nojimage]

(int)$this->request->data('Person.person_answer_exists')が2でif文の中に入っていることは確認されていますか？

[nojimage]

あ、フォームから person_answer_exists = 2 で送っているつもりなのに、Controller側ではperson_answer_exists = 1 になっているってことでしょうか

[Tannnnnnnnnnn]

ご返信ありがとうございます。

＞(int)$this->request->data('Person.person_answer_exists')が2でif文の中に入っていることは確認されていますか？

今確認し、入っておりませんでした、、。

if((int)$this->request->data('Person.person_answer_exists') === 1){
$conditions['Person.pe_contents !='] = '';
}

の中では、
$this->request->data()
を入れて更新すると、値を確認出来ます。

＞あ、フォームから person_answer_exists = 2 で送っているつもりなのに、Controller側ではperson_answer_exists = 1 になっているってことでしょうか

いえ、controller側で確認しており、
array{
[0] => 1 //あり
[1] => 2 //なし
}
で送られてきております、、

[nojimage]

フォームの方の 'multiple' => 'checkbox', を見逃していましたね。multipleは不要です。

[Tannnnnnnnnnn]

チェックボックスで表示させたいため、multipleの記載は必要そうでした、、

[nojimage]

会員の回答は、あり、なし両方にチェックを入れれるということですね。
あり、なし両方にチェックが入った場合は、両方の検索条件がORで結合されるような形にするとよいでしょうか

[Tannnnnnnnnnn]

その通りです。すみません、言葉足らずで。